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

2017-05-23 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-303622021 @rafaelweingartner The router_proxy route happens for Xen and KVM (similar to the way we manually access

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

2017-05-23 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-303622021 @rafaelweingartner The router_proxy route happens for Xen and KVM (similar to the way we manually access

[GitHub] cloudmonger commented on issue #1911: CLOUDSTACK-9742:Simultaneous snapshots for detached volume

2017-05-23 Thread git
cloudmonger commented on issue #1911: CLOUDSTACK-9742:Simultaneous snapshots for detached volume URL: https://github.com/apache/cloudstack/pull/1911#issuecomment-303615045 ### ACS CI BVT Run **Sumarry:** Build Number 743 Hypervisor xenserver NetworkType Advanced Passed=1

[GitHub] niteshsarda commented on issue #2082: CLOUDSTACK-9017 : VPC VR DHCP broken for multihomed guest VMs

2017-05-23 Thread git
niteshsarda commented on issue #2082: CLOUDSTACK-9017 : VPC VR DHCP broken for multihomed guest VMs URL: https://github.com/apache/cloudstack/pull/2082#issuecomment-303607308 @ustcweizhou : Can this fix be merged now ? This

[GitHub] niteshsarda commented on issue #2082: CLOUDSTACK-9017 : VPC VR DHCP broken for multihomed guest VMs

2017-05-23 Thread git
niteshsarda commented on issue #2082: CLOUDSTACK-9017 : VPC VR DHCP broken for multihomed guest VMs URL: https://github.com/apache/cloudstack/pull/2082#issuecomment-303606987 @borisstoyanov : Following are the steps for reproducing this issue : 1. Deploy a vm to a VPC having more th

[GitHub] cloudmonger commented on issue #1906: CLOUDSTACK-9743 - ODL plugin responds to deleteHost causing other plugin in the chain to be ignored

2017-05-23 Thread git
cloudmonger commented on issue #1906: CLOUDSTACK-9743 - ODL plugin responds to deleteHost causing other plugin in the chain to be ignored URL: https://github.com/apache/cloudstack/pull/1906#issuecomment-303552514 ### ACS CI BVT Run **Sumarry:** Build Number 742 Hypervisor xense

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

2017-05-23 Thread git
blueorangutan commented on issue #2098: Fix for CLOUDSTACK-9660 URL: https://github.com/apache/cloudstack/pull/2098#issuecomment-303503166 Trillian test result (tid-1123) Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7 Total time taken: 47819 seconds Marvin log

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

2017-05-23 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 NetworkTy

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

2017-05-23 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-303458905 ### ACS CI BVT Run **Sumarry:** Build Number 741 Hypervisor xenserver NetworkTy

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

2017-05-23 Thread git
rafaelweingartner commented on issue #1995: CLOUDSTACK-9828: GetDomRVersionCmd fails to get the correct versi? URL: https://github.com/apache/cloudstack/pull/1995#issuecomment-303434312 @abhinandanprateek even with this change the error still happened in XenServer hosts. I noticed the u

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

2017-05-23 Thread git
blueorangutan commented on issue #2098: Fix for CLOUDSTACK-9660 URL: https://github.com/apache/cloudstack/pull/2098#issuecomment-303408170 Trillian test result (tid-1124) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 28720 seconds Marvin log

[GitHub] ustcweizhou commented on issue #2082: CLOUDSTACK-9017 : VPC VR DHCP broken for multihomed guest VMs

2017-05-23 Thread git
ustcweizhou commented on issue #2082: CLOUDSTACK-9017 : VPC VR DHCP broken for multihomed guest VMs URL: https://github.com/apache/cloudstack/pull/2082#issuecomment-303399911 @borisstoyanov are you testing with 4.9.2 release ? can you post some configuration files in the VR, for example

[GitHub] SudharmaJain commented on issue #876: CLOUDSTACK-8865: Adding SR doesn't create Storage_pool_host_ref entry?

2017-05-23 Thread git
SudharmaJain commented on issue #876: CLOUDSTACK-8865: Adding SR doesn't create Storage_pool_host_ref entry? URL: https://github.com/apache/cloudstack/pull/876#issuecomment-303392400 @DaanHoogland Nothing to be done from my side. The failures are not related to my changes. -

[GitHub] sateesh-chodapuneedi commented on a change in pull request #2105: Vyos router plugin

2017-05-23 Thread git
sateesh-chodapuneedi commented on a change in pull request #2105: Vyos router plugin URL: https://github.com/apache/cloudstack/pull/2105#discussion_r117983849 ## File path: plugins/network-elements/vyos/src/com/cloud/api/commands/ConfigureVyosRouterFirewallCmd.java ## @@

[GitHub] sateesh-chodapuneedi commented on a change in pull request #2105: Vyos router plugin

2017-05-23 Thread git
sateesh-chodapuneedi commented on a change in pull request #2105: Vyos router plugin URL: https://github.com/apache/cloudstack/pull/2105#discussion_r117983648 ## File path: plugins/network-elements/vyos/src/com/cloud/api/commands/AddVyosRouterFirewallCmd.java ## @@ -0,0 +

[GitHub] borisstoyanov commented on a change in pull request #2082: CLOUDSTACK-9017 : VPC VR DHCP broken for multihomed guest VMs

2017-05-23 Thread git
borisstoyanov commented on a change in pull request #2082: CLOUDSTACK-9017 : VPC VR DHCP broken for multihomed guest VMs URL: https://github.com/apache/cloudstack/pull/2082#discussion_r117981173 ## File path: systemvm/patches/debian/config/opt/cloud/bin/cs_dhcp.py ## @@ -3

[GitHub] ustcweizhou commented on issue #2082: CLOUDSTACK-9017 : VPC VR DHCP broken for multihomed guest VMs

2017-05-23 Thread git
ustcweizhou commented on issue #2082: CLOUDSTACK-9017 : VPC VR DHCP broken for multihomed guest VMs URL: https://github.com/apache/cloudstack/pull/2082#issuecomment-303383626 LGTM however, please note there will be multiple lines in /etc/hosts for the vm. if customer uses vm name

[GitHub] niteshsarda commented on issue #2082: CLOUDSTACK-9017 : VPC VR DHCP broken for multihomed guest VMs

2017-05-23 Thread git
niteshsarda commented on issue #2082: CLOUDSTACK-9017 : VPC VR DHCP broken for multihomed guest VMs URL: https://github.com/apache/cloudstack/pull/2082#issuecomment-303379177 @ustcweizhou : I have removed search method from cs_dhcp.py class and tested the code again. Changes seems to be wo

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

2017-05-23 Thread git
rhtyd 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-303340427 LGTM. This is an auto

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

2017-05-23 Thread git
harikrishna-patnala commented on issue #2125: CLOUDSTACK-9927: Root admin user should be forced to change password ? URL: https://github.com/apache/cloudstack/pull/2125#issuecomment-303338380 Automation runs use 8096 port where we don't need any password. This should not effect test runs.

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

2017-05-23 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_r11791 ## File path: test/integration/component/test_tags.py ## @@ -2543,3 +2543,248 @@ def test_24_

[GitHub] niteshsarda commented on issue #2126: CLOUDSTACK-9740 : Search for secondary IP of NIC that is attached to an instance is not working

2017-05-23 Thread git
niteshsarda commented on issue #2126: CLOUDSTACK-9740 : Search for secondary IP of NIC that is attached to an instance is not working URL: https://github.com/apache/cloudstack/pull/2126#issuecomment-303317873 @ustcweizhou @SudharmaJain : Old PR https://github.com/apache/cloudstack/pull/190

[GitHub] niteshsarda commented on issue #1909: CLOUDSTACK-9740 : Search for secondary IP of NIC that is attached to ?

2017-05-23 Thread git
niteshsarda commented on issue #1909: CLOUDSTACK-9740 : Search for secondary IP of NIC that is attached to ? URL: https://github.com/apache/cloudstack/pull/1909#issuecomment-303317270 @ustcweizhou @SudharmaJain : This PR got automatically closed while pushing the updated code. So created a

[GitHub] niteshsarda opened a new pull request #2126: CLOUDSTACK-9740 : Search for secondary IP of NIC that is attached to an instance is not working

2017-05-23 Thread git
niteshsarda opened a new pull request #2126: CLOUDSTACK-9740 : Search for secondary IP of NIC that is attached to an instance is not working URL: https://github.com/apache/cloudstack/pull/2126 Search for secondary IP of NIC that is attached to an instance is not working. If instance

[GitHub] Divyanimmagadda commented on issue #2091: CLOUDSTACK-8609: [VMware] VM is not accessible after migration across clusters

2017-05-23 Thread git
Divyanimmagadda commented on issue #2091: CLOUDSTACK-8609: [VMware] VM is not accessible after migration across clusters URL: https://github.com/apache/cloudstack/pull/2091#issuecomment-303309509 Fix verified. LGTM. This is

[GitHub] Divyanimmagadda commented on issue #2091: CLOUDSTACK-8609: [VMware] VM is not accessible after migration across clusters

2017-05-23 Thread git
Divyanimmagadda commented on issue #2091: CLOUDSTACK-8609: [VMware] VM is not accessible after migration across clusters URL: https://github.com/apache/cloudstack/pull/2091#issuecomment-303305702 Test verified. LGTM. This is

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

2017-05-23 Thread git
jayapalu 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-303308318 Failed tests are not related to this PR. Marking merge ready. --