[GitHub] sudhansu7 commented on issue #1849: CLOUDSTACK-9690: Scale CentOS7 VM fails with error

2017-05-04 Thread git
sudhansu7 commented on issue #1849: CLOUDSTACK-9690: Scale CentOS7 VM fails with error URL: https://github.com/apache/cloudstack/pull/1849#issuecomment-299381952 tag:mergeready This is an automated message from the Apache

[GitHub] sudhansu7 commented on issue #1849: CLOUDSTACK-9690: Scale CentOS7 VM fails with error

2017-05-04 Thread git
sudhansu7 commented on issue #1849: CLOUDSTACK-9690: Scale CentOS7 VM fails with error URL: https://github.com/apache/cloudstack/pull/1849#issuecomment-299381856 @jayantpatil1234, I have removed Upgrade4910to4920.java from commit.

[GitHub] mrunalinikankariya commented on issue #1940: CLOUDSTACK-9781:ACS records ID in events tables instead of UUID.

2017-05-04 Thread git
mrunalinikankariya commented on issue #1940: CLOUDSTACK-9781:ACS records ID in events tables instead of UUID. URL: https://github.com/apache/cloudstack/pull/1940#issuecomment-299374950 Code changes looks good, Code LGTM

[GitHub] mrunalinikankariya commented on issue #1940: CLOUDSTACK-9781:ACS records ID in events tables instead of UUID.

2017-05-04 Thread git
mrunalinikankariya commented on issue #1940: CLOUDSTACK-9781:ACS records ID in events tables instead of UUID. URL: https://github.com/apache/cloudstack/pull/1940#issuecomment-299374950 Code changes looks good, Code LGTM

[GitHub] jayapalu commented on issue #1996: CLOUDSTACK-9099: SecretKey is returned from the APIs

2017-05-04 Thread git
jayapalu commented on issue #1996: CLOUDSTACK-9099: SecretKey is returned from the APIs URL: https://github.com/apache/cloudstack/pull/1996#issuecomment-299368310 Previously Travis CI is failed. Now Travis CI is passed.

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

2017-05-04 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-299364306 ### ACS CI BVT Run **Sumarry:** Build Number 623 Hypervisor xenserver

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

2017-05-04 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-299320546 ### ACS CI BVT Run **Sumarry:** Build Number 622 Hypervisor xenserver

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

2017-05-04 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-299191802 ### ACS CI BVT Run **Sumarry:** Build Number 620 Hypervisor xenserver

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

2017-05-04 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-299320546 ### ACS CI BVT Run **Sumarry:** Build Number 622 Hypervisor xenserver

[GitHub] leprechau commented on issue #1606: Allow CGN (RFC6598) to be used within a VPC

2017-05-04 Thread git
leprechau commented on issue #1606: Allow CGN (RFC6598) to be used within a VPC URL: https://github.com/apache/cloudstack/pull/1606#issuecomment-299309549 @kiwiflyer @karuturi Tests passed! This is an automated message from

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

2017-05-04 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-299279867 Trillian test result (tid-1056) Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7

[GitHub] serg38 commented on issue #927: CLOUDSTACK-9901 secure and hidden config values are returned as plaintext string

2017-05-04 Thread git
serg38 commented on issue #927: CLOUDSTACK-9901 secure and hidden config values are returned as plaintext string URL: https://github.com/apache/cloudstack/pull/927#issuecomment-299225065 LGTM @rhtyd @borisstoyanov @DaanHoogland Can we run B.O. for this PR?

[GitHub] nathanejohnson opened a new pull request #2078: CLOUDSTACK-9902: add consoleproxy.sslEnable toggle

2017-05-04 Thread git
nathanejohnson opened a new pull request #2078: CLOUDSTACK-9902: add consoleproxy.sslEnable toggle URL: https://github.com/apache/cloudstack/pull/2078 This adds the missing global configuration parameter consoleproxy.sslEnable required for enabling SSL on the console proxy VM.

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

2017-05-04 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-299191802 ### ACS CI BVT Run **Sumarry:** Build Number 620 Hypervisor xenserver

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

2017-05-04 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-298819708 ### ACS CI BVT Run **Sumarry:** Build Number 614 Hypervisor xenserver

[GitHub] cloudmonger commented on issue #927: CLOUDSTACK-9901 secure and hidden config values are returned as plaintext string

2017-05-04 Thread git
cloudmonger commented on issue #927: CLOUDSTACK-9901 secure and hidden config values are returned as plaintext string URL: https://github.com/apache/cloudstack/pull/927#issuecomment-299137118 ### ACS CI BVT Run **Sumarry:** Build Number 619 Hypervisor xenserver NetworkType

[GitHub] cloudmonger commented on issue #927: CLOUDSTACK-9901 secure and hidden config values are returned as plaintext string

2017-05-04 Thread git
cloudmonger commented on issue #927: CLOUDSTACK-9901 secure and hidden config values are returned as plaintext string URL: https://github.com/apache/cloudstack/pull/927#issuecomment-297779185 ### ACS CI BVT Run **Sumarry:** Build Number 591 Hypervisor xenserver NetworkType

[GitHub] SowjanyaPatha commented on issue #1940: CLOUDSTACK-9781:ACS records ID in events tables instead of UUID.

2017-05-04 Thread git
SowjanyaPatha commented on issue #1940: CLOUDSTACK-9781:ACS records ID in events tables instead of UUID. URL: https://github.com/apache/cloudstack/pull/1940#issuecomment-299131855 LGTM for Testing. Checked the Events Tab for the ID it is referring to UUID. Attached screenshot

[GitHub] blueorangutan commented on issue #2070: CLOUDSTACK-9904: Fix log4j to have @AGENTLOG@ replaced

2017-05-04 Thread git
blueorangutan commented on issue #2070: CLOUDSTACK-9904: Fix log4j to have @AGENTLOG@ replaced URL: https://github.com/apache/cloudstack/pull/2070#issuecomment-299123363 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #2070: CLOUDSTACK-9904: Fix log4j to have @AGENTLOG@ replaced

2017-05-04 Thread git
rhtyd commented on issue #2070: CLOUDSTACK-9904: Fix log4j to have @AGENTLOG@ replaced URL: https://github.com/apache/cloudstack/pull/2070#issuecomment-299123277 @blueorangutan package This is an automated message from the

[GitHub] rhtyd commented on issue #2070: [4.9+] hyperv: Fix log4j to have @AGENTLOG@ replaced

2017-05-04 Thread git
rhtyd commented on issue #2070: [4.9+] hyperv: Fix log4j to have @AGENTLOG@ replaced URL: https://github.com/apache/cloudstack/pull/2070#issuecomment-299122904 @kiwiflyer thanks, done https://issues.apache.org/jira/browse/CLOUDSTACK-9904

[GitHub] shwetaag closed pull request #1949: Automated Cloudstack bugs 9277 9276 9275 9274 9273 9179 9178 9177

2017-05-04 Thread git
shwetaag closed pull request #1949: Automated Cloudstack bugs 9277 9276 9275 9274 9273 9179 9178 9177 URL: https://github.com/apache/cloudstack/pull/1949 This is an automated message from the Apache Git Service. To

[GitHub] shwetaag opened a new pull request #1949: Automated Cloudstack bugs 9277 9276 9275 9274 9273 9179 9178 9177

2017-05-04 Thread git
shwetaag opened a new pull request #1949: Automated Cloudstack bugs 9277 9276 9275 9274 9273 9179 9178 9177 URL: https://github.com/apache/cloudstack/pull/1949 Addressed all review comments. Changed all URL to pubic available url where ever possible . For few tests needing rhel7

[GitHub] rhtyd commented on a change in pull request #1960: [4.11/Future] CLOUDSTACK-9782: Host HA and KVM HA provider

2017-05-04 Thread git
rhtyd commented on a change in pull request #1960: [4.11/Future] CLOUDSTACK-9782: Host HA and KVM HA provider URL: https://github.com/apache/cloudstack/pull/1960#discussion_r114709634 ## File path: plugins/hypervisors/kvm/src/com/cloud/ha/KVMInvestigator.java ## @@ -40,17

[GitHub] rhtyd commented on a change in pull request #1960: [4.11/Future] CLOUDSTACK-9782: Host HA and KVM HA provider

2017-05-04 Thread git
rhtyd commented on a change in pull request #1960: [4.11/Future] CLOUDSTACK-9782: Host HA and KVM HA provider URL: https://github.com/apache/cloudstack/pull/1960#discussion_r114709646 ## File path: plugins/hypervisors/kvm/src/com/cloud/ha/KVMInvestigator.java ## @@ -67,6

[GitHub] rhtyd commented on a change in pull request #1960: [4.11/Future] CLOUDSTACK-9782: Host HA and KVM HA provider

2017-05-04 Thread git
rhtyd commented on a change in pull request #1960: [4.11/Future] CLOUDSTACK-9782: Host HA and KVM HA provider URL: https://github.com/apache/cloudstack/pull/1960#discussion_r114709491 ## File path: plugins/hypervisors/kvm/src/com/cloud/ha/KVMInvestigator.java ## @@ -67,6

[GitHub] rhtyd commented on a change in pull request #1960: [4.11/Future] CLOUDSTACK-9782: Host HA and KVM HA provider

2017-05-04 Thread git
rhtyd commented on a change in pull request #1960: [4.11/Future] CLOUDSTACK-9782: Host HA and KVM HA provider URL: https://github.com/apache/cloudstack/pull/1960#discussion_r114709474 ## File path: plugins/hypervisors/kvm/src/com/cloud/ha/KVMInvestigator.java ## @@ -40,17

[GitHub] rhtyd commented on issue #1960: [4.11/Future] CLOUDSTACK-9782: Host HA and KVM HA provider

2017-05-04 Thread git
rhtyd commented on issue #1960: [4.11/Future] CLOUDSTACK-9782: Host HA and KVM HA provider URL: https://github.com/apache/cloudstack/pull/1960#issuecomment-299110561 @koushik-das Host-HA is disabled by default, it needs explicit configuration and to be enabled in order to be used. It

[GitHub] koushik-das commented on issue #2007: CLOUDSTACK-9834: prepareTemplate API call doesn't work well with XenServer & Local SR (Db_exn.Uniqueness_constraint_violation)

2017-05-04 Thread git
koushik-das commented on issue #2007: CLOUDSTACK-9834: prepareTemplate API call doesn't work well with XenServer & Local SR (Db_exn.Uniqueness_constraint_violation) URL: https://github.com/apache/cloudstack/pull/2007#issuecomment-299107199 Code changes LGTM

[GitHub] sureshanaparti commented on a change in pull request #1897: CLOUDSTACK-9733: Concurrent volume snapshots of a VM are not allowed and are not limited per host as per the global configuration p

2017-05-04 Thread git
sureshanaparti commented on a change in pull request #1897: CLOUDSTACK-9733: Concurrent volume snapshots of a VM are not allowed and are not limited per host as per the global configuration parameter "concurrent.snapshots.threshold.perhost". URL:

[GitHub] sureshanaparti commented on a change in pull request #1897: CLOUDSTACK-9733: Concurrent volume snapshots of a VM are not allowed and are not limited per host as per the global configuration p

2017-05-04 Thread git
sureshanaparti commented on a change in pull request #1897: CLOUDSTACK-9733: Concurrent volume snapshots of a VM are not allowed and are not limited per host as per the global configuration parameter "concurrent.snapshots.threshold.perhost". URL:

[GitHub] sureshanaparti commented on issue #1253: CLOUDSTACK-9184: [VMware] vmware.ports.per.dvportgroup global setting is not useful from vCenter 5.0 onwards

2017-05-04 Thread git
sureshanaparti commented on issue #1253: CLOUDSTACK-9184: [VMware] vmware.ports.per.dvportgroup global setting is not useful from vCenter 5.0 onwards URL: https://github.com/apache/cloudstack/pull/1253#issuecomment-299103280 @sateesh-chodapuneedi Thanks for confirming this. I'll update

[GitHub] sureshanaparti commented on issue #1878: CLOUDSTACK-9717: [VMware] RVRs have mismatching MAC addresses for extra public NICs.

2017-05-04 Thread git
sureshanaparti commented on issue #1878: CLOUDSTACK-9717: [VMware] RVRs have mismatching MAC addresses for extra public NICs. URL: https://github.com/apache/cloudstack/pull/1878#issuecomment-299103325 @sateesh-chodapuneedi Sure. Thanks for the review.

[GitHub] jayapalu commented on issue #1996: CLOUDSTACK-9099: SecretKey is returned from the APIs

2017-05-04 Thread git
jayapalu commented on issue #1996: CLOUDSTACK-9099: SecretKey is returned from the APIs URL: https://github.com/apache/cloudstack/pull/1996#issuecomment-299103191 @borisstoyanov With 8096, I did not get see any test failures in my local setup. But in Trillian it seems the test setup

[GitHub] sureshanaparti commented on a change in pull request #1872: CLOUDSTACK-3223 Exception observed while creating CPVM in VMware Setup with DVS

2017-05-04 Thread git
sureshanaparti commented on a change in pull request #1872: CLOUDSTACK-3223 Exception observed while creating CPVM in VMware Setup with DVS URL: https://github.com/apache/cloudstack/pull/1872#discussion_r114280010 ## File path:

[GitHub] kishankavala commented on a change in pull request #1897: CLOUDSTACK-9733: Concurrent volume snapshots of a VM are not allowed and are not limited per host as per the global configuration par

2017-05-04 Thread git
kishankavala commented on a change in pull request #1897: CLOUDSTACK-9733: Concurrent volume snapshots of a VM are not allowed and are not limited per host as per the global configuration parameter "concurrent.snapshots.threshold.perhost". URL:

[GitHub] kishankavala commented on a change in pull request #1897: CLOUDSTACK-9733: Concurrent volume snapshots of a VM are not allowed and are not limited per host as per the global configuration par

2017-05-04 Thread git
kishankavala commented on a change in pull request #1897: CLOUDSTACK-9733: Concurrent volume snapshots of a VM are not allowed and are not limited per host as per the global configuration parameter "concurrent.snapshots.threshold.perhost". URL: