[GitHub] rhtyd commented on issue #2098: Fix for CLOUDSTACK-9660

2017-05-22 Thread git
rhtyd commented on issue #2098: Fix for CLOUDSTACK-9660 URL: https://github.com/apache/cloudstack/pull/2098#issuecomment-303298232 @karuturi sure @blueorangutan test centos7 vmware-55u3 This is an automated message from

[GitHub] blueorangutan commented on issue #2098: Fix for CLOUDSTACK-9660

2017-05-22 Thread git
blueorangutan commented on issue #2098: Fix for CLOUDSTACK-9660 URL: https://github.com/apache/cloudstack/pull/2098#issuecomment-303298283 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + vmware-55u3) has been kicked to run smoke tests

[GitHub] yvsubhash commented on a change in pull request #905: BUG-ID: CLOUDSTACK-8922: Unable to delete IP tag

2017-05-22 Thread git
yvsubhash commented on a change in pull request #905: BUG-ID: CLOUDSTACK-8922: Unable to delete IP tag URL: https://github.com/apache/cloudstack/pull/905#discussion_r117904259 ## File path: test/integration/component/test_tags.py ## @@ -2543,3 +2543,248 @@ def

[GitHub] ramkatru commented on issue #1845: CLOUDSTACK-9689: [Hyper-V] Fixed VM console is freezing sometimes and?

2017-05-22 Thread git
ramkatru commented on issue #1845: CLOUDSTACK-9689: [Hyper-V] Fixed VM console is freezing sometimes and? URL: https://github.com/apache/cloudstack/pull/1845#issuecomment-303296229 Code LGTM. This is an automated message

[GitHub] niteshsarda closed pull request #1909: CLOUDSTACK-9740 : Search for secondary IP of NIC that is attached to ?

2017-05-22 Thread git
niteshsarda closed pull request #1909: CLOUDSTACK-9740 : Search for secondary IP of NIC that is attached to ? URL: https://github.com/apache/cloudstack/pull/1909 This is an automated message from the Apache Git Service. To

[GitHub] cloudmonger commented on issue #2113: CLOUDSTACK-9162: Handled vpn user add when vpn is not enabled on the?

2017-05-22 Thread git
cloudmonger commented on issue #2113: CLOUDSTACK-9162: Handled vpn user add when vpn is not enabled on the? URL: https://github.com/apache/cloudstack/pull/2113#issuecomment-303291747 ### ACS CI BVT Run **Sumarry:** Build Number 739 Hypervisor xenserver NetworkType Advanced

[GitHub] yvsubhash commented on issue #1761: CLOUDSTACK-9592 Empty responses from site to site connection status a?

2017-05-22 Thread git
yvsubhash commented on issue #1761: CLOUDSTACK-9592 Empty responses from site to site connection status a? URL: https://github.com/apache/cloudstack/pull/1761#issuecomment-303291370 BVT failures are not related to PR. tag:mergeready

[GitHub] karuturi closed pull request #2121: CLOUDSTACK-9641 In KVM SSVM and CPVM may use the old cmdline data, if?

2017-05-22 Thread git
karuturi closed pull request #2121: CLOUDSTACK-9641 In KVM SSVM and CPVM may use the old cmdline data, if? URL: https://github.com/apache/cloudstack/pull/2121 This is an automated message from the Apache Git Service. To

[GitHub] karuturi commented on issue #2098: Fix for CLOUDSTACK-9660

2017-05-22 Thread git
karuturi commented on issue #2098: Fix for CLOUDSTACK-9660 URL: https://github.com/apache/cloudstack/pull/2098#issuecomment-303288240 Thanks @mike-tutkowski. Let us wait and see if there are same failures in the next run. @rhtyd looks like the last job didn't post results. Can you run

[GitHub] blueorangutan commented on issue #883: CLOUDSTACK-8906: /var/log/cloud/ doesn't get logrotated on xenserver

2017-05-22 Thread git
blueorangutan commented on issue #883: CLOUDSTACK-8906: /var/log/cloud/ doesn't get logrotated on xenserver URL: https://github.com/apache/cloudstack/pull/883#issuecomment-303285280 Trillian test result (tid-1122) Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 7

[GitHub] cloudmonger commented on issue #2081: WIP: CLOUDSTACK-9894 Separate creation and backup operations for a volume snapshot

2017-05-22 Thread git
cloudmonger commented on issue #2081: WIP: CLOUDSTACK-9894 Separate creation and backup operations for a volume snapshot URL: https://github.com/apache/cloudstack/pull/2081#issuecomment-303262771 ### ACS CI BVT Run **Sumarry:** Build Number 738 Hypervisor xenserver

[GitHub] cloudmonger commented on issue #2081: WIP: CLOUDSTACK-9894 Separate creation and backup operations for a volume snapshot

2017-05-22 Thread git
cloudmonger commented on issue #2081: WIP: CLOUDSTACK-9894 Separate creation and backup operations for a volume snapshot URL: https://github.com/apache/cloudstack/pull/2081#issuecomment-302824286 ### ACS CI BVT Run **Sumarry:** Build Number 723 Hypervisor xenserver

[GitHub] blueorangutan commented on issue #926: [4.10] CLOUDSTACK-7984: Collect network statistics for VMs on shared network (KVM implementation)

2017-05-22 Thread git
blueorangutan commented on issue #926: [4.10] CLOUDSTACK-7984: Collect network statistics for VMs on shared network (KVM implementation) URL: https://github.com/apache/cloudstack/pull/926#issuecomment-303250973 Trillian test result (tid-1121) Environment: kvm-centos7 (x2), Advanced

[GitHub] mike-tutkowski commented on issue #2098: Fix for CLOUDSTACK-9660

2017-05-22 Thread git
mike-tutkowski commented on issue #2098: Fix for CLOUDSTACK-9660 URL: https://github.com/apache/cloudstack/pull/2098#issuecomment-303217603 @karuturi For test_vm_life_cycle.test_08_migrate_vm, it looks like the test was run in a bad state for some reason: jobresult :

[GitHub] cloudmonger commented on issue #1969: CLOUDSTACK-9667 Enable resourcecount.check.interval by default

2017-05-22 Thread git
cloudmonger commented on issue #1969: CLOUDSTACK-9667 Enable resourcecount.check.interval by default URL: https://github.com/apache/cloudstack/pull/1969#issuecomment-303208430 ### ACS CI BVT Run **Sumarry:** Build Number 737 Hypervisor xenserver NetworkType Advanced

[GitHub] rafaelweingartner commented on issue #2125: CLOUDSTACK-9927: Root admin user should be forced to change password ?

2017-05-22 Thread git
rafaelweingartner commented on issue #2125: CLOUDSTACK-9927: Root admin user should be forced to change password ? URL: https://github.com/apache/cloudstack/pull/2125#issuecomment-303139973 @DaanHoogland this proposal only affects the UI. The automation you run only use the API, right?

[GitHub] blueorangutan commented on issue #883: CLOUDSTACK-8906: /var/log/cloud/ doesn't get logrotated on xenserver

2017-05-22 Thread git
blueorangutan commented on issue #883: CLOUDSTACK-8906: /var/log/cloud/ doesn't get logrotated on xenserver URL: https://github.com/apache/cloudstack/pull/883#issuecomment-303120463 @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + xenserver-65sp1) has been kicked to run smoke

[GitHub] DaanHoogland commented on issue #883: CLOUDSTACK-8906: /var/log/cloud/ doesn't get logrotated on xenserver

2017-05-22 Thread git
DaanHoogland commented on issue #883: CLOUDSTACK-8906: /var/log/cloud/ doesn't get logrotated on xenserver URL: https://github.com/apache/cloudstack/pull/883#issuecomment-303120134 @blueorangutan test centos7 xenserver-65sp1

[GitHub] DaanHoogland commented on issue #955: CLOUDSTACK-8969 VPN customer gateway can't be registered with hostname

2017-05-22 Thread git
DaanHoogland commented on issue #955: CLOUDSTACK-8969 VPN customer gateway can't be registered with hostname URL: https://github.com/apache/cloudstack/pull/955#issuecomment-303119637 this is simple enough indeed but activity has ceased a year ago. Do we want this @SudharmaJain ?

[GitHub] cloudmonger commented on issue #1965: CLOUDSTACK-9727 Password reset discrepancy in RVR when one of the Rou?

2017-05-22 Thread git
cloudmonger commented on issue #1965: CLOUDSTACK-9727 Password reset discrepancy in RVR when one of the Rou? URL: https://github.com/apache/cloudstack/pull/1965#issuecomment-303119217 ### ACS CI BVT Run **Sumarry:** Build Number 736 Hypervisor xenserver NetworkType Advanced

[GitHub] cloudmonger commented on issue #1965: CLOUDSTACK-9727 Password reset discrepancy in RVR when one of the Rou?

2017-05-22 Thread git
cloudmonger commented on issue #1965: CLOUDSTACK-9727 Password reset discrepancy in RVR when one of the Rou? URL: https://github.com/apache/cloudstack/pull/1965#issuecomment-302677455 ### ACS CI BVT Run **Sumarry:** Build Number 718 Hypervisor xenserver NetworkType Advanced

[GitHub] DaanHoogland closed pull request #938: CLOUDSTACK-8959: Option to attach the config drive.

2017-05-22 Thread git
DaanHoogland closed pull request #938: CLOUDSTACK-8959: Option to attach the config drive. URL: https://github.com/apache/cloudstack/pull/938 This is an automated message from the Apache Git Service. To respond to the

[GitHub] DaanHoogland commented on issue #938: CLOUDSTACK-8959: Option to attach the config drive.

2017-05-22 Thread git
DaanHoogland commented on issue #938: CLOUDSTACK-8959: Option to attach the config drive. URL: https://github.com/apache/cloudstack/pull/938#issuecomment-303117779 as stated, closing in favour of #2116 and #2097! This is an

[GitHub] DaanHoogland commented on issue #2116: CLOUDSTACK-8959: Option to attach the config drive

2017-05-22 Thread git
DaanHoogland commented on issue #2116: CLOUDSTACK-8959: Option to attach the config drive URL: https://github.com/apache/cloudstack/pull/2116#issuecomment-303117376 reviewer please see #938 as well when reviewing this This

[GitHub] blueorangutan commented on issue #926: [4.10] CLOUDSTACK-7984: Collect network statistics for VMs on shared network (KVM implementation)

2017-05-22 Thread git
blueorangutan commented on issue #926: [4.10] CLOUDSTACK-7984: Collect network statistics for VMs on shared network (KVM implementation) URL: https://github.com/apache/cloudstack/pull/926#issuecomment-303116374 @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has

[GitHub] blueorangutan commented on issue #926: [4.10] CLOUDSTACK-7984: Collect network statistics for VMs on shared network (KVM implementation)

2017-05-22 Thread git
blueorangutan commented on issue #926: [4.10] CLOUDSTACK-7984: Collect network statistics for VMs on shared network (KVM implementation) URL: https://github.com/apache/cloudstack/pull/926#issuecomment-303115346 @DaanHoogland unsupported parameters provided. Supported mgmt server os are:

[GitHub] DaanHoogland commented on issue #926: [4.10] CLOUDSTACK-7984: Collect network statistics for VMs on shared network (KVM implementation)

2017-05-22 Thread git
DaanHoogland commented on issue #926: [4.10] CLOUDSTACK-7984: Collect network statistics for VMs on shared network (KVM implementation) URL: https://github.com/apache/cloudstack/pull/926#issuecomment-303116173 @blueorangutan test centos7 kvm-centos7

[GitHub] DaanHoogland commented on issue #926: [4.10] CLOUDSTACK-7984: Collect network statistics for VMs on shared network (KVM implementation)

2017-05-22 Thread git
DaanHoogland commented on issue #926: [4.10] CLOUDSTACK-7984: Collect network statistics for VMs on shared network (KVM implementation) URL: https://github.com/apache/cloudstack/pull/926#issuecomment-303115080 @blueorangutan test kvm-centos7

[GitHub] blueorangutan commented on issue #926: [4.10] CLOUDSTACK-7984: Collect network statistics for VMs on shared network (KVM implementation)

2017-05-22 Thread git
blueorangutan commented on issue #926: [4.10] CLOUDSTACK-7984: Collect network statistics for VMs on shared network (KVM implementation) URL: https://github.com/apache/cloudstack/pull/926#issuecomment-303115346 @DaanHoogland unsupported parameters provided. Supported mgmt server os are:

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

2017-05-22 Thread git
DaanHoogland 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-303115561 @karuturi , you want to merge this?

[GitHub] DaanHoogland commented on issue #926: [4.10] CLOUDSTACK-7984: Collect network statistics for VMs on shared network (KVM implementation)

2017-05-22 Thread git
DaanHoogland commented on issue #926: [4.10] CLOUDSTACK-7984: Collect network statistics for VMs on shared network (KVM implementation) URL: https://github.com/apache/cloudstack/pull/926#issuecomment-303115080 @blueorangutan test kvm-centos7

[GitHub] DaanHoogland commented on issue #925: Fix for CLOUDSTACK-8937 - XenServer migrations with storage failing i?

2017-05-22 Thread git
DaanHoogland commented on issue #925: Fix for CLOUDSTACK-8937 - XenServer migrations with storage failing i? URL: https://github.com/apache/cloudstack/pull/925#issuecomment-303111468 as stated by the author, this can be closed.

[GitHub] DaanHoogland closed pull request #925: Fix for CLOUDSTACK-8937 - XenServer migrations with storage failing i?

2017-05-22 Thread git
DaanHoogland closed pull request #925: Fix for CLOUDSTACK-8937 - XenServer migrations with storage failing i? URL: https://github.com/apache/cloudstack/pull/925 This is an automated message from the Apache Git Service. To

[GitHub] DaanHoogland commented on issue #914: CLOUDSTACK-8939 VM Snapshot size with memory correctly calculated in cloud.usage_event (XenServer)

2017-05-22 Thread git
DaanHoogland commented on issue #914: CLOUDSTACK-8939 VM Snapshot size with memory correctly calculated in cloud.usage_event (XenServer) URL: https://github.com/apache/cloudstack/pull/914#issuecomment-303110931 @yvsubhash this is not ready for merge. It has only one lgts and there are

[GitHub] blueorangutan commented on issue #883: CLOUDSTACK-8906: /var/log/cloud/ doesn't get logrotated on xenserver

2017-05-22 Thread git
blueorangutan commented on issue #883: CLOUDSTACK-8906: /var/log/cloud/ doesn't get logrotated on xenserver URL: https://github.com/apache/cloudstack/pull/883#issuecomment-303110652 Packaging result: ?centos6 ?centos7 ?debian. JID-738

[GitHub] DaanHoogland commented on a change in pull request #905: BUG-ID: CLOUDSTACK-8922: Unable to delete IP tag

2017-05-22 Thread git
DaanHoogland commented on a change in pull request #905: BUG-ID: CLOUDSTACK-8922: Unable to delete IP tag URL: https://github.com/apache/cloudstack/pull/905#discussion_r117751953 ## File path: server/src/com/cloud/tags/TaggedResourceManagerImpl.java ## @@ -297,16 +297,46

[GitHub] DaanHoogland commented on a change in pull request #905: BUG-ID: CLOUDSTACK-8922: Unable to delete IP tag

2017-05-22 Thread git
DaanHoogland commented on a change in pull request #905: BUG-ID: CLOUDSTACK-8922: Unable to delete IP tag URL: https://github.com/apache/cloudstack/pull/905#discussion_r117751554 ## File path: test/integration/component/test_tags.py ## @@ -2543,3 +2543,248 @@ def

[GitHub] DaanHoogland commented on issue #899: BUG-ID:CLOUDSTACK-8921 snapshot_store_ref table should store actual size of backup snapshot in secondary storage

2017-05-22 Thread git
DaanHoogland commented on issue #899: BUG-ID:CLOUDSTACK-8921 snapshot_store_ref table should store actual size of backup snapshot in secondary storage URL: https://github.com/apache/cloudstack/pull/899#issuecomment-303105509 trusting the process took place on this one

[GitHub] blueorangutan commented on issue #883: CLOUDSTACK-8906: /var/log/cloud/ doesn't get logrotated on xenserver

2017-05-22 Thread git
blueorangutan commented on issue #883: CLOUDSTACK-8906: /var/log/cloud/ doesn't get logrotated on xenserver URL: https://github.com/apache/cloudstack/pull/883#issuecomment-303104274 @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] DaanHoogland commented on issue #883: CLOUDSTACK-8906: /var/log/cloud/ doesn't get logrotated on xenserver

2017-05-22 Thread git
DaanHoogland commented on issue #883: CLOUDSTACK-8906: /var/log/cloud/ doesn't get logrotated on xenserver URL: https://github.com/apache/cloudstack/pull/883#issuecomment-303104192 @blueorangutan package This is an

[GitHub] DaanHoogland commented on a change in pull request #876: CLOUDSTACK-8865: Adding SR doesn't create Storage_pool_host_ref entry?

2017-05-22 Thread git
DaanHoogland commented on a change in pull request #876: CLOUDSTACK-8865: Adding SR doesn't create Storage_pool_host_ref entry? URL: https://github.com/apache/cloudstack/pull/876#discussion_r117747025 ## File path: server/src/com/cloud/resource/ResourceManagerImpl.java ##

[GitHub] DaanHoogland commented on a change in pull request #859: Bug-ID: CLOUDSTACK-8882: calculate network offering usage per nic instead of per network offering

2017-05-22 Thread git
DaanHoogland commented on a change in pull request #859: Bug-ID: CLOUDSTACK-8882: calculate network offering usage per nic instead of per network offering URL: https://github.com/apache/cloudstack/pull/859#discussion_r117745815 ## File path:

[GitHub] DaanHoogland commented on issue #837: CLOUDSTACK-8855 Improve Error Message for Host Alert State and reconnect host API.

2017-05-22 Thread git
DaanHoogland commented on issue #837: CLOUDSTACK-8855 Improve Error Message for Host Alert State and reconnect host API. URL: https://github.com/apache/cloudstack/pull/837#issuecomment-303100490 @bvbharatk this is waiting for Author and @rafaelweingartner to come to agreement on the code.

[GitHub] sureshanaparti commented on a change in pull request #2042: CLOUDSTACK-9875: Unable to re-apply Explicit dedication to VM

2017-05-22 Thread git
sureshanaparti commented on a change in pull request #2042: CLOUDSTACK-9875: Unable to re-apply Explicit dedication to VM URL: https://github.com/apache/cloudstack/pull/2042#discussion_r117737324 ## File path: server/src/org/apache/cloudstack/affinity/AffinityGroupServiceImpl.java

[GitHub] sureshanaparti commented on a change in pull request #2042: CLOUDSTACK-9875: Unable to re-apply Explicit dedication to VM

2017-05-22 Thread git
sureshanaparti commented on a change in pull request #2042: CLOUDSTACK-9875: Unable to re-apply Explicit dedication to VM URL: https://github.com/apache/cloudstack/pull/2042#discussion_r117734140 ## File path: server/src/org/apache/cloudstack/affinity/AffinityGroupServiceImpl.java

[GitHub] borisstoyanov commented on issue #862: CLOUDSTACK-7151: Support multiple physical networks at cluster level for guest traffic over VMware deployments.

2017-05-22 Thread git
borisstoyanov commented on issue #862: CLOUDSTACK-7151: Support multiple physical networks at cluster level for guest traffic over VMware deployments. URL: https://github.com/apache/cloudstack/pull/862#issuecomment-303064611 @sureshanaparti could you please resolve the merge conflicts

[GitHub] abhinandanprateek commented on issue #1995: CLOUDSTACK-9828: GetDomRVersionCmd fails to get the correct versi?

2017-05-22 Thread git
abhinandanprateek commented on issue #1995: CLOUDSTACK-9828: GetDomRVersionCmd fails to get the correct versi? URL: https://github.com/apache/cloudstack/pull/1995#issuecomment-303062705 @rafaelweingartner it is not easily reproducible. We were seeing in one of our customer env and then in

[GitHub] cloudmonger commented on issue #2116: CLOUDSTACK-8959: Option to attach the config drive

2017-05-22 Thread git
cloudmonger commented on issue #2116: CLOUDSTACK-8959: Option to attach the config drive URL: https://github.com/apache/cloudstack/pull/2116#issuecomment-303042849 ### ACS CI BVT Run **Sumarry:** Build Number 733 Hypervisor xenserver NetworkType Advanced Passed=57

[GitHub] anshul1886 commented on a change in pull request #1845: CLOUDSTACK-9689: [Hyper-V] Fixed VM console is freezing sometimes and?

2017-05-22 Thread git
anshul1886 commented on a change in pull request #1845: CLOUDSTACK-9689: [Hyper-V] Fixed VM console is freezing sometimes and? URL: https://github.com/apache/cloudstack/pull/1845#discussion_r117695756 ## File path:

[GitHub] vedulasantosh commented on issue #2100: CLOUDSTACK-9907 Physical size of snapshot is considered for usage

2017-05-22 Thread git
vedulasantosh commented on issue #2100: CLOUDSTACK-9907 Physical size of snapshot is considered for usage URL: https://github.com/apache/cloudstack/pull/2100#issuecomment-303023562 Test LGTM In my observation when we give the global configuration (usage.snapshot.virtualsize.select)

[GitHub] DaanHoogland commented on issue #2125: CLOUDSTACK-9927: Root admin user should be forced to change password ?

2017-05-22 Thread git
DaanHoogland commented on issue #2125: CLOUDSTACK-9927: Root admin user should be forced to change password ? URL: https://github.com/apache/cloudstack/pull/2125#issuecomment-303010989 Though this might seem as a reasonable security feature, I have doubts. The integration automation

[GitHub] harikrishna-patnala opened a new pull request #2125: CLOUDSTACK-9927: Root admin user should be forced to change password ?

2017-05-22 Thread git
harikrishna-patnala opened a new pull request #2125: CLOUDSTACK-9927: Root admin user should be forced to change password ? URL: https://github.com/apache/cloudstack/pull/2125 The default password for the root admin in CloudStack is "password". The user is not required to change this