[GitHub] DaanHoogland commented on issue #2071: CLOUDSTACK-9815 move CertService to more generic location

2017-04-28 Thread git
DaanHoogland commented on issue #2071: CLOUDSTACK-9815 move CertService to more generic location URL: https://github.com/apache/cloudstack/pull/2071#issuecomment-297927230 @rafaelweingartner I don't like to. Is this :-1: worth? Can of worms involved. There is a lot more service defi

[GitHub] DaanHoogland commented on issue #2071: CLOUDSTACK-9815 move CertService to more generic location

2017-04-28 Thread git
DaanHoogland commented on issue #2071: CLOUDSTACK-9815 move CertService to more generic location URL: https://github.com/apache/cloudstack/pull/2071#issuecomment-297927230 @rafaelweingartner I don't like to. Is this :-1: worth? Can of worms involved. There is a lot more service defi

[GitHub] DaanHoogland commented on issue #2071: CLOUDSTACK-9815 move CertService to more generic location

2017-04-28 Thread git
DaanHoogland commented on issue #2071: CLOUDSTACK-9815 move CertService to more generic location URL: https://github.com/apache/cloudstack/pull/2071#issuecomment-297930758 the virtio failures are addressed in #2066. still needs a 'this reads like poetry to me" ---

Re: Github messages on dev list

2017-04-28 Thread Rene Moser
Hi Wido There have been messages from github in the past (you probably had a filter?). I already wrote I don't like it and suggested to use comm...@cloudstack.apache.org for automated and bot messages. But a few devs insisted they like to keep them here that filtering would be an option (for all

[GitHub] rhtyd commented on a change in pull request #1302: CLOUDSTACK-9104: VM naming convention in case vmware is used

2017-04-28 Thread git
rhtyd commented on a change in pull request #1302: CLOUDSTACK-9104: VM naming convention in case vmware is used URL: https://github.com/apache/cloudstack/pull/1302#discussion_r113885728 ## File path: plugins/hypervisors/vmware/test/com/cloud/hypervisor/vmware/resource/VmwareResourc

[GitHub] rhtyd commented on a change in pull request #1302: CLOUDSTACK-9104: VM naming convention in case vmware is used

2017-04-28 Thread git
rhtyd commented on a change in pull request #1302: CLOUDSTACK-9104: VM naming convention in case vmware is used URL: https://github.com/apache/cloudstack/pull/1302#discussion_r113885986 ## File path: server/src/com/cloud/vm/UserVmManagerImpl.java ## @@ -3383,12 +3383,6 @@

[GitHub] rhtyd commented on a change in pull request #1302: CLOUDSTACK-9104: VM naming convention in case vmware is used

2017-04-28 Thread git
rhtyd commented on a change in pull request #1302: CLOUDSTACK-9104: VM naming convention in case vmware is used URL: https://github.com/apache/cloudstack/pull/1302#discussion_r113886159 ## File path: plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource

[GitHub] rhtyd commented on issue #1302: CLOUDSTACK-9104: VM naming convention in case vmware is used

2017-04-28 Thread git
rhtyd commented on issue #1302: CLOUDSTACK-9104: VM naming convention in case vmware is used URL: https://github.com/apache/cloudstack/pull/1302#issuecomment-297949612 @abhinandanprateek @DaanHoogland -- ping for review? @PaulAngus would you like to share your thoughts in changing VM nam

[GitHub] rhtyd commented on issue #2018: CLOUDSTACK-9848: Added exit status checking for the iptables commands

2017-04-28 Thread git
rhtyd commented on issue #2018: CLOUDSTACK-9848: Added exit status checking for the iptables commands URL: https://github.com/apache/cloudstack/pull/2018#issuecomment-297949754 It might be regressions due to the change, I can help rekick them. @blueorangutan package --

[GitHub] blueorangutan commented on issue #2018: CLOUDSTACK-9848: Added exit status checking for the iptables commands

2017-04-28 Thread git
blueorangutan commented on issue #2018: CLOUDSTACK-9848: Added exit status checking for the iptables commands URL: https://github.com/apache/cloudstack/pull/2018#issuecomment-297949836 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. -

[GitHub] rhtyd commented on issue #2066: CLOUDSTACK-9893 / CLOUDSTACK-8239 fix test_deploy_virtio_scsi_vm.py

2017-04-28 Thread git
rhtyd commented on issue #2066: CLOUDSTACK-9893 / CLOUDSTACK-8239 fix test_deploy_virtio_scsi_vm.py URL: https://github.com/apache/cloudstack/pull/2066#issuecomment-297950110 LGTM. @blueorangutan package This is an automa

[GitHub] pavanaravapalli opened a new pull request #2072: WIP: CLOUDSTACK-9895 : Added support for parallel volume(s) creation from a volume snapshot

2017-04-28 Thread git
pavanaravapalli opened a new pull request #2072: WIP: CLOUDSTACK-9895 : Added support for parallel volume(s) creation from a volume snapshot URL: https://github.com/apache/cloudstack/pull/2072 This is an automated message f

Re: Github messages on dev list

2017-04-28 Thread Will Stevens
I would be +1 for moving them to commits@. In the current implementation, I find it is not nearly as useful at a glance. The info I care about (PR number and user) are not visible when I scroll through the list. Ideally we could change the subject format to move the important info to the start of

[GitHub] borisstoyanov commented on issue #2062: CLOUDSTACK-9878: remove unnecessary VPN drops

2017-04-28 Thread git
borisstoyanov commented on issue #2062: CLOUDSTACK-9878: remove unnecessary VPN drops URL: https://github.com/apache/cloudstack/pull/2062#issuecomment-297972556 @blueorangutan package This is an automated message from the Ap

[GitHub] blueorangutan commented on issue #2062: CLOUDSTACK-9878: remove unnecessary VPN drops

2017-04-28 Thread git
blueorangutan commented on issue #2062: CLOUDSTACK-9878: remove unnecessary VPN drops URL: https://github.com/apache/cloudstack/pull/2062#issuecomment-297972707 @borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --

[GitHub] DaanHoogland commented on a change in pull request #1302: CLOUDSTACK-9104: VM naming convention in case vmware is used

2017-04-28 Thread git
DaanHoogland commented on a change in pull request #1302: CLOUDSTACK-9104: VM naming convention in case vmware is used URL: https://github.com/apache/cloudstack/pull/1302#discussion_r113907545 ## File path: plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareR

[GitHub] DaanHoogland commented on a change in pull request #1302: CLOUDSTACK-9104: VM naming convention in case vmware is used

2017-04-28 Thread git
DaanHoogland commented on a change in pull request #1302: CLOUDSTACK-9104: VM naming convention in case vmware is used URL: https://github.com/apache/cloudstack/pull/1302#discussion_r113907617 ## File path: plugins/hypervisors/vmware/test/com/cloud/hypervisor/vmware/resource/Vmware

[GitHub] DaanHoogland commented on a change in pull request #1302: CLOUDSTACK-9104: VM naming convention in case vmware is used

2017-04-28 Thread git
DaanHoogland commented on a change in pull request #1302: CLOUDSTACK-9104: VM naming convention in case vmware is used URL: https://github.com/apache/cloudstack/pull/1302#discussion_r113908318 ## File path: server/src/com/cloud/vm/UserVmManagerImpl.java ## @@ -3383,12 +338

[GitHub] blueorangutan commented on issue #2062: CLOUDSTACK-9878: remove unnecessary VPN drops

2017-04-28 Thread git
blueorangutan commented on issue #2062: CLOUDSTACK-9878: remove unnecessary VPN drops URL: https://github.com/apache/cloudstack/pull/2062#issuecomment-297977466 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] borisstoyanov commented on issue #2062: CLOUDSTACK-9878: remove unnecessary VPN drops

2017-04-28 Thread git
borisstoyanov commented on issue #2062: CLOUDSTACK-9878: remove unnecessary VPN drops URL: https://github.com/apache/cloudstack/pull/2062#issuecomment-297977456 @blueorangutan test This is an automated message from the Apach

Re: Github messages on dev list

2017-04-28 Thread Nicolas Vazquez
I personally prefer the way it worked before, where Github username was shown in From field and not in Subject. That way mails could be grouped by PR 2017-04-28 8:08 GMT-03:00 Will Stevens : > I would be +1 for moving them to commits@. > > In the current implementation, I find it is not nearly as

[GitHub] koushik-das commented on a change in pull request #2072: WIP: CLOUDSTACK-9895 : Added support for parallel volume(s) creation from a volume snapshot

2017-04-28 Thread git
koushik-das commented on a change in pull request #2072: WIP: CLOUDSTACK-9895 : Added support for parallel volume(s) creation from a volume snapshot URL: https://github.com/apache/cloudstack/pull/2072#discussion_r113936161 ## File path: engine/storage/volume/src/org/apache/cloudsta

[GitHub] koushik-das commented on a change in pull request #2072: WIP: CLOUDSTACK-9895 : Added support for parallel volume(s) creation from a volume snapshot

2017-04-28 Thread git
koushik-das commented on a change in pull request #2072: WIP: CLOUDSTACK-9895 : Added support for parallel volume(s) creation from a volume snapshot URL: https://github.com/apache/cloudstack/pull/2072#discussion_r113935048 ## File path: engine/storage/volume/src/org/apache/cloudsta

[GitHub] koushik-das commented on a change in pull request #2072: WIP: CLOUDSTACK-9895 : Added support for parallel volume(s) creation from a volume snapshot

2017-04-28 Thread git
koushik-das commented on a change in pull request #2072: WIP: CLOUDSTACK-9895 : Added support for parallel volume(s) creation from a volume snapshot URL: https://github.com/apache/cloudstack/pull/2072#discussion_r113936886 ## File path: framework/jobs/src/org/apache/cloudstack/fram

[GitHub] koushik-das commented on a change in pull request #2072: WIP: CLOUDSTACK-9895 : Added support for parallel volume(s) creation from a volume snapshot

2017-04-28 Thread git
koushik-das commented on a change in pull request #2072: WIP: CLOUDSTACK-9895 : Added support for parallel volume(s) creation from a volume snapshot URL: https://github.com/apache/cloudstack/pull/2072#discussion_r113931425 ## File path: engine/storage/volume/src/org/apache/cloudsta

[GitHub] koushik-das commented on a change in pull request #2072: WIP: CLOUDSTACK-9895 : Added support for parallel volume(s) creation from a volume snapshot

2017-04-28 Thread git
koushik-das commented on a change in pull request #2072: WIP: CLOUDSTACK-9895 : Added support for parallel volume(s) creation from a volume snapshot URL: https://github.com/apache/cloudstack/pull/2072#discussion_r113930154 ## File path: engine/schema/src/com/cloud/storage/VolumeDeta

[GitHub] koushik-das commented on a change in pull request #2072: WIP: CLOUDSTACK-9895 : Added support for parallel volume(s) creation from a volume snapshot

2017-04-28 Thread git
koushik-das commented on a change in pull request #2072: WIP: CLOUDSTACK-9895 : Added support for parallel volume(s) creation from a volume snapshot URL: https://github.com/apache/cloudstack/pull/2072#discussion_r113932053 ## File path: engine/schema/src/org/apache/cloudstack/resou

[GitHub] koushik-das commented on a change in pull request #2072: WIP: CLOUDSTACK-9895 : Added support for parallel volume(s) creation from a volume snapshot

2017-04-28 Thread git
koushik-das commented on a change in pull request #2072: WIP: CLOUDSTACK-9895 : Added support for parallel volume(s) creation from a volume snapshot URL: https://github.com/apache/cloudstack/pull/2072#discussion_r113932159 ## File path: engine/schema/src/org/apache/cloudstack/resou

[GitHub] koushik-das commented on a change in pull request #2072: WIP: CLOUDSTACK-9895 : Added support for parallel volume(s) creation from a volume snapshot

2017-04-28 Thread git
koushik-das commented on a change in pull request #2072: WIP: CLOUDSTACK-9895 : Added support for parallel volume(s) creation from a volume snapshot URL: https://github.com/apache/cloudstack/pull/2072#discussion_r113936474 ## File path: framework/jobs/src/org/apache/cloudstack/fram

[GitHub] cloudmonger commented on issue #1894: CLOUDSTACK-9700 Allow user to Register/Copy templates to multiple zones at the same time

2017-04-28 Thread git
cloudmonger commented on issue #1894: CLOUDSTACK-9700 Allow user to Register/Copy templates to multiple zones at the same time URL: https://github.com/apache/cloudstack/pull/1894#issuecomment-298036704 ### ACS CI BVT Run **Sumarry:** Build Number 595 Hypervisor xenserver Net

[GitHub] DaanHoogland commented on issue #2066: CLOUDSTACK-9893 / CLOUDSTACK-8239 fix test_deploy_virtio_scsi_vm.py

2017-04-28 Thread git
DaanHoogland commented on issue #2066: CLOUDSTACK-9893 / CLOUDSTACK-8239 fix test_deploy_virtio_scsi_vm.py URL: https://github.com/apache/cloudstack/pull/2066#issuecomment-298045082 @karuturi can you merge this? This is an a

Re: Github messages on dev list

2017-04-28 Thread Tutkowski, Mike
I don’t know…I like waking up to about 200 e-mails that I need to delete. ;) On 4/28/17, 7:52 AM, "Nicolas Vazquez" wrote: I personally prefer the way it worked before, where Github username was shown in From field and not in Subject. That way mails could be grouped by PR 20

[GitHub] blueorangutan commented on issue #2062: CLOUDSTACK-9878: remove unnecessary VPN drops

2017-04-28 Thread git
blueorangutan commented on issue #2062: CLOUDSTACK-9878: remove unnecessary VPN drops URL: https://github.com/apache/cloudstack/pull/2062#issuecomment-298100691 Trillian test result (tid-1048) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 3194

[GitHub] cloudmonger commented on issue #1894: CLOUDSTACK-9700 Allow user to Register/Copy templates to multiple zones at the same time

2017-04-28 Thread git
cloudmonger commented on issue #1894: CLOUDSTACK-9700 Allow user to Register/Copy templates to multiple zones at the same time URL: https://github.com/apache/cloudstack/pull/1894#issuecomment-298036704 ### ACS CI BVT Run **Sumarry:** Build Number 595 Hypervisor xenserver Net

[GitHub] cloudmonger commented on issue #1894: CLOUDSTACK-9700 Allow user to Register/Copy templates to multiple zones at the same time

2017-04-28 Thread git
cloudmonger commented on issue #1894: CLOUDSTACK-9700 Allow user to Register/Copy templates to multiple zones at the same time URL: https://github.com/apache/cloudstack/pull/1894#issuecomment-298117640 ### ACS CI BVT Run **Sumarry:** Build Number 596 Hypervisor xenserver Net

[GitHub] cloudmonger commented on issue #1894: CLOUDSTACK-9700 Allow user to Register/Copy templates to multiple zones at the same time

2017-04-28 Thread git
cloudmonger commented on issue #1894: CLOUDSTACK-9700 Allow user to Register/Copy templates to multiple zones at the same time URL: https://github.com/apache/cloudstack/pull/1894#issuecomment-298145735 ### ACS CI BVT Run **Sumarry:** Build Number 597 Hypervisor xenserver Net

[GitHub] cloudmonger commented on issue #1894: CLOUDSTACK-9700 Allow user to Register/Copy templates to multiple zones at the same time

2017-04-28 Thread git
cloudmonger commented on issue #1894: CLOUDSTACK-9700 Allow user to Register/Copy templates to multiple zones at the same time URL: https://github.com/apache/cloudstack/pull/1894#issuecomment-298117640 ### ACS CI BVT Run **Sumarry:** Build Number 596 Hypervisor xenserver Net