[GitHub] cloudstack issue #1929: CLOUDSTACK-9770: fix missing ip routes in VR

2017-02-05 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1929 LGTM. thanks @ustcweizhou --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

Re: [PROPOSAL] add native container orchestration service

2017-02-02 Thread Murali Reddy
k8s >specific knowledge into cloudstack. > >Regards, > > >-- >Lianghwa Jou >VP Engineering, Accelerite >email: lianghwa@accelerite.com > > > > > >On 1/29/17, 11:54 PM, "Murali Reddy" wrote: > > >I agree with some good views Will has shar

Re: [PROPOSAL] add native container orchestration service

2017-01-29 Thread Murali Reddy
-config scripts that automatically setup k8s cluster master and node VM’s. All CloudStack is doing in passing the cloud-config to the core OS VM’s as user data. Regards Murali Reddy On 29/01/17, 8:54 AM, "Will Stevens" wrote: >I agree that we need to be careful what we take on a

Re: [PROPOSAL] add native container orchestration service

2017-01-27 Thread Murali Reddy
, but there is value why AWS provides native functionality with AWS ECS. Same goes with google's GKE. On 27/01/17, 3:20 PM, "Wido den Hollander" wrote: > >> Op 27 januari 2017 om 7:20 schreef Murali Reddy : >> >> >> All, >> >> I would like p

[PROPOSAL] add native container orchestration service

2017-01-26 Thread Murali Reddy
All, I would like propose native functionality into CloudStack to provide a container service through which users out-of-the box can use to launch container based application. Idea is to support ability to orchestrate the resources and automate aspects of setting up container orchestrator throu

[GitHub] cloudstack issue #1856: CLOUDSTACK-9569: propagate global configuration rout...

2016-12-22 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1856 @ustcweizhou this will be useful for 4.9 as well, would you mind rebasing the patch to 4.9? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request #1856: CLOUDSTACK-9569: propagate global configurati...

2016-12-22 Thread murali-reddy
Github user murali-reddy commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1856#discussion_r93599467 --- Diff: core/src/com/cloud/agent/resource/virtualnetwork/VirtualRoutingResource.java --- @@ -259,8 +260,18 @@ private Answer execute

[GitHub] cloudstack issue #1804: CLOUDSTACK-9639: Unable to create shared network wit...

2016-12-21 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1804 LGTM. a small side note. User can create a guest VM with nic's in two networks with same CIDR but different VLAN isolations and mess-up routing. --- If your project is set u

[GitHub] cloudstack issue #1837: [4.9] Smoketest Health

2016-12-21 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1837 @blueorangutan test centos7 xenserver-65sp1 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] cloudstack issue #1837: [4.9] Smoketest Health

2016-12-21 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1837 @blueorangutan test centos7 vmware-55u3 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] cloudstack issue #1519: Cloudstack 9339: Virtual Routers do not handle Multi...

2016-12-14 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1519 @dsclose #1659 is merged, so you can close this PR if you wish. Though issue related to redundant VR fixed by commit d582358, bf285e1 in this PR are not addressed by #1659, and still a

[GitHub] cloudstack issue #1828: CLOUDSTACK-9676 Start instance fails after reverting...

2016-12-14 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1828 @blueorangutan test centos7 vmware-55u3 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] cloudstack issue #1828: CLOUDSTACK-9676 Start instance fails after reverting...

2016-12-14 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1828 @blueorangutan package --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

Re: [VOTE] Apache Cloudstack 4.9.1.0 (RC1)

2016-12-14 Thread Murali Reddy
+1 Based on smoke test results [1] and below tests run manually on VMware deployment - VM life cycle operations - network life cycle operations - verified source NAT, static NAT, PF, LB, VPC internal LB, network ACL, firewall - upgrade from 4.9 and 4.5.2 IMO issue raised by Milamber s

[GitHub] cloudstack issue #1783: CLOUDSTACK-9615: Fixd applying ingress rules without...

2016-12-12 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1783 LGTM. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or

[GitHub] cloudstack issue #1823: CLOUDSTACK-9594: Fix regression in test_templates

2016-12-08 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1823 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if

[GitHub] cloudstack pull request #1821: CLOUDSTACK-9659: mismatch in traffic type in ...

2016-12-08 Thread murali-reddy
GitHub user murali-reddy opened a pull request: https://github.com/apache/cloudstack/pull/1821 CLOUDSTACK-9659: mismatch in traffic type in ip_associations.json and ips.json As part of the bug 'CLOUDSTACK-9339 Virtual Routers don't handle Multiple Public Interfaces

[GitHub] cloudstack issue #1579: CLOUDSTACK-9403 : Support for shared networks in Nua...

2016-12-07 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1579 code changes LGTM Travis run has been failing due to test 'test_create_volume_under_domain' for all the PR's. Lets wait till we fix the test or disable this

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-12-06 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1659 component test results. If any one wish to run component test test_multiple_public_interfaces.py, add a new public ip range in the test_daya.py, in the 'publiciprange' d

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-12-06 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1659 @rhtyd please proceed with merge. I have not added the test suite(test_multiple_public_interfaces.py) part of this PR as smoke tests, because it takes around an hour for execution so they

[GitHub] cloudstack issue #1799: CLOUDSTACK-9632: Upgrade bouncy castle to version 1....

2016-12-06 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1799 code change LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-12-05 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1659 Can this merged now? test failures are specific to redundent VR/VPC VR which have been failing in 4.8/4.9/master. --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack issue #1604: improvement: systemvm slow operations

2016-12-05 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1604 @romain-dartigues we fixed issue where ips.json has wrong gateway due to error in parsing the cmd_line.json. its committed in 4.8 and above [1]. Do you still see (or can you please verify

[GitHub] cloudstack issue #1784: CS-505: Marvin test to check VR internal DNS Service

2016-12-05 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1784 @abhinandanprateek Multi-nic will be different scenario, we can skip that in this test case. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-04 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1802 This is good to be merged now. test_privategw_acl.py is passing mostly. there are still intermittent failures (not addressed by this PR) in the test_privategw_acl.py like create and delete

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-01 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1802 @blueorangutan package --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-01 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1802 @jburwell This PR is not rebased to fix from #1801. This PR is confined to test_privategw_acl.py, so no scope of any regression out side of it. --- If your project is set up for it, you

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-01 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1802 @jburwell this PR is ready for merge. PR fixes intermittent failures of suite of tests in test_privategw_acl.py. test_04_rvpc_privategw_static_routesSuccess 553.97

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-11-30 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1802 @blueorangutan test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-11-30 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1802 @blueorangutan package --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] cloudstack issue #1801: fix marvin test failure test_router_dhcp_opts

2016-11-30 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1801 test_router_dhcp_opts is passing, and we have necessary LGTM's. can we merge this? --- If your project is set up for it, you can reply to this email and have your reply appear on G

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-11-30 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1802 @blueorangutan test centos6 kvm-centos6 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-11-30 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1802 @blueorangutan test centos7 kvm-centos6 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-11-30 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1802 @blueorangutan package --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] cloudstack pull request #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-11-30 Thread murali-reddy
GitHub user murali-reddy opened a pull request: https://github.com/apache/cloudstack/pull/1802 CLOUDSTACK-9635: fix test_privategw_acl.py ensure VLAN used for createPrivateGateway is determined after the guest networks in the VPC is created, so that we skip VLAN allocated for

[GitHub] cloudstack pull request #1801: fix marvin test failure test_router_dhcp_opts

2016-11-30 Thread murali-reddy
GitHub user murali-reddy opened a pull request: https://github.com/apache/cloudstack/pull/1801 fix marvin test failure test_router_dhcp_opts marvin, VirtualMachine objects are not in any particualr order in the array. so check isdefault attribute to the get non-default nic

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-11-29 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1659 @blueorangutan test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-11-29 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1659 @blueorangutan test centos7 kvm-centos7 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] cloudstack issue #1787: fix syntax error introduced in CsDhcp.py

2016-11-28 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1787 closing, as its addressed by #1788 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] cloudstack pull request #1787: fix syntax error introduced in CsDhcp.py

2016-11-28 Thread murali-reddy
Github user murali-reddy closed the pull request at: https://github.com/apache/cloudstack/pull/1787 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] cloudstack issue #1787: fix syntax error introduced in CsDhcp.py

2016-11-27 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1787 @ blueorangutan package --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] cloudstack pull request #1787: fix syntax error introduced in CsDhcp.py

2016-11-27 Thread murali-reddy
GitHub user murali-reddy opened a pull request: https://github.com/apache/cloudstack/pull/1787 fix syntax error introduced in CsDhcp.py fix syntax error introduced in CsDhcp.py while forward merging fix for CS-9598 on to 4.9, 4.10 due to signature mismatch You can merge this pull

[GitHub] cloudstack issue #1766: CLOUDSTACK-9598: wrong defaut gateway for the nic in...

2016-11-23 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1766 @rhtyd it was packaging issue, in failed run my change was not picked --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] cloudstack issue #1766: CLOUDSTACK-9598: wrong defaut gateway for the nic in...

2016-11-23 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1766 @rhtyd @abhinandanprateek @ustcweizhou can you please review, we have clean test pass and new test test_router_dhcp_opts passing too. --- If your project is set up for it, you can reply

[GitHub] cloudstack issue #1766: CLOUDSTACK-9598: wrong defaut gateway for the nic in...

2016-11-23 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1766 @blueorangutan test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] cloudstack issue #1766: CLOUDSTACK-9598: wrong defaut gateway for the nic in...

2016-11-22 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1766 @blueorangutan test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] cloudstack issue #1766: CLOUDSTACK-9598: wrong defaut gateway for the nic in...

2016-11-22 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1766 Added test case TestRouterDHCPOpts, that will test the VR behaviour of not sending gateway and DNS server information while server DHCP response on non-default network NIC&#

[GitHub] cloudstack issue #1766: CLOUDSTACK-9598: wrong defaut gateway for the nic in...

2016-11-22 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1766 @blueorangutan package --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] cloudstack issue #1766: CLOUDSTACK-9598: wrong defaut gateway for the nic in...

2016-11-17 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1766 @blueorangutan test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] cloudstack issue #1766: CLOUDSTACK-9598: wrong defaut gateway for the nic in...

2016-11-17 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1766 @blueorangutan package --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] cloudstack pull request #1766: CLOUDSTACK-9598: wrong defaut gateway for the...

2016-11-17 Thread murali-reddy
GitHub user murali-reddy opened a pull request: https://github.com/apache/cloudstack/pull/1766 CLOUDSTACK-9598: wrong defaut gateway for the nic in non-default network when guest VM has nic's in more than one guest network set the tag for each host in /etc/dhcphosts.txt, an

[GitHub] cloudstack issue #1757: CLOUDSTACK-9583: VR: In CsDhcp.py preseed both hosta...

2016-11-10 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1757 @blueorangutan test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] cloudstack issue #1757: CLOUDSTACK-9583: VR: In CsDhcp.py preseed both hosta...

2016-11-10 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1757 @blueorangutan package --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] cloudstack issue #1750: CLOUDSTACK-9583: VR: In CsDhcp.py preseed both hosta...

2016-11-10 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1750 opened #1757 targeting 4.8 so closed this PR --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] cloudstack pull request #1757: CLOUDSTACK-9583: VR: In CsDhcp.py preseed bot...

2016-11-10 Thread murali-reddy
GitHub user murali-reddy opened a pull request: https://github.com/apache/cloudstack/pull/1757 CLOUDSTACK-9583: VR: In CsDhcp.py preseed both hostaname and localhost to resolve to 127.0.0.1 The VR executes a ip route flush command as part of configurations. This command

[GitHub] cloudstack pull request #1750: CLOUDSTACK-9583: VR: In CsDhcp.py preseed bot...

2016-11-09 Thread murali-reddy
Github user murali-reddy closed the pull request at: https://github.com/apache/cloudstack/pull/1750 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] cloudstack issue #1751: systemd: Fix semicolon missing in b75e69

2016-11-09 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1751 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if

[GitHub] cloudstack issue #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

2016-11-09 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1706 @jburwell #1659 has a fix which complements this patch.Basically when first IP is associated with an interface routing table is created and should be deleted when last IP associated is

[GitHub] cloudstack issue #1750: CLOUDSTACK-9583: VR: In CsDhcp.py preseed both hosta...

2016-11-08 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1750 @blueorangutan test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] cloudstack issue #1750: CLOUDSTACK-9583: VR: In CsDhcp.py preseed both hosta...

2016-11-08 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1750 @blueorangutan test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] cloudstack issue #1673: CLOUDSTACK-9071: Properly parse stats.output.uri in ...

2016-11-08 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1673 @jburwell failures are redundant VPC related being tracked CLOUDSTACK-9541 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] cloudstack issue #1750: CLOUDSTACK-9583: VR: In CsDhcp.py preseed both hosta...

2016-11-08 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1750 @blueorangutan package --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] cloudstack pull request #1750: CLOUDSTACK-9583: VR: In CsDhcp.py preseed bot...

2016-11-08 Thread murali-reddy
GitHub user murali-reddy opened a pull request: https://github.com/apache/cloudstack/pull/1750 CLOUDSTACK-9583: VR: In CsDhcp.py preseed both hostaname and localhost to resolve to 127.0.0.1 ensuring for both vpc and non vpc vr, both localhost and hostname is resolved to

[GitHub] cloudstack issue #1713: CLOUDSTACK-9552: Allow egress TCP/53 implicitly in B...

2016-11-07 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1713 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if

[GitHub] cloudstack issue #1676: CLOUDSTACK-9502: DS template copies don’t get dele...

2016-11-02 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1676 @jburwell VR failures seen are intermittent failures we are seeing w.r.t VPC static routes, RVPC VR in other PR's as well. Not related to the patch. --- If your project is set up f

[GitHub] cloudstack issue #1741: Updated StrongSwan VPN Implementation

2016-11-02 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1741 @jburwell No, I have fixed issue with static nat on any public IP associated in case of multiple public interfaces. @swill Is there any possibility of public IP going out of order

[GitHub] cloudstack issue #1729: CLOUDSTACK-9564: Fix memory leaks in VmwareContextPo...

2016-11-01 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1729 LGTM from code change. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

2016-10-30 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1681 @blueorangutan test centos7 vmware55u3 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] cloudstack issue #1680: CLOUDSTACK-9498: VR CsFile search utility methods fa...

2016-10-30 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1680 @blueorangutan test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] cloudstack issue #1680: CLOUDSTACK-9498: VR CsFile search utility methods fa...

2016-10-28 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1680 @blueorangutan test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

2016-10-28 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1681 @blueorangutan test centos7 vmware55u3 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

2016-10-28 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1681 @blueorangutan package --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] cloudstack issue #1680: CLOUDSTACK-9498: VR CsFile search utility methods fa...

2016-10-28 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1680 @blueorangutan package --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-10-26 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1659 @blueorangutan test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-10-26 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1659 @blueorangutan package --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] cloudstack issue #1731: CLOUDSTACK-9565: Fix intermittent failure in test_oo...

2016-10-26 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1731 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if

[GitHub] cloudstack pull request #1724: CLOUDSTACK-9511: fix test_privategw_acl.py to...

2016-10-25 Thread murali-reddy
GitHub user murali-reddy reopened a pull request: https://github.com/apache/cloudstack/pull/1724 CLOUDSTACK-9511: fix test_privategw_acl.py to handle multiple physical network fix to ensure only physical network with guest traffic is picked up for creating a private

[GitHub] cloudstack pull request #1724: CLOUDSTACK-9511: fix test_privategw_acl.py to...

2016-10-25 Thread murali-reddy
Github user murali-reddy closed the pull request at: https://github.com/apache/cloudstack/pull/1724 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] cloudstack issue #1724: CLOUDSTACK-9511: fix test_privategw_acl.py to handle...

2016-10-24 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1724 all tests in test_privategw_acl.py passed on KVM & XEN except for test_02_vpc_privategw_static_routes test fail in XenServer, which was due to private gateway not getting cre

[GitHub] cloudstack pull request #1724: CLOUDSTACK-9511: fix test_privategw_acl.py to...

2016-10-24 Thread murali-reddy
GitHub user murali-reddy opened a pull request: https://github.com/apache/cloudstack/pull/1724 CLOUDSTACK-9511: fix test_privategw_acl.py to handle multiple physical network fix to ensure only physical network with guest traffic is picked up for creating a private network

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-10-20 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1659 @blueorangutan package --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] cloudstack issue #1692: Fix Smoke Test Failures

2016-10-19 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1692 @syed Thanks for testing out. Tests related VPC private gateway, redundant VPC are still failing as noted [1]. I am working on fix. We want to proceed with this PR with those failing tests

[GitHub] cloudstack issue #1695: CLOUDSTACK-9511: fix test_privategw_acl.py to handle...

2016-09-29 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1695 I am closing this pull request, and pushed the changes as part of PR #1692, which will cover overall marvin test failures --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request #1695: CLOUDSTACK-9511: fix test_privategw_acl.py to...

2016-09-29 Thread murali-reddy
Github user murali-reddy closed the pull request at: https://github.com/apache/cloudstack/pull/1695 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] cloudstack pull request #1695: CLOUDSTACK-9511: fix test_privategw_acl.py to...

2016-09-29 Thread murali-reddy
GitHub user murali-reddy opened a pull request: https://github.com/apache/cloudstack/pull/1695 CLOUDSTACK-9511: fix test_privategw_acl.py to handle multiple physical networks modified script to handle case where there can be multple physical networks in the zone

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

2016-09-26 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1681 @blueorangutan test centos7 vmware55u3 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

2016-09-26 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1681 @jburwell function 'findRouterEthDeviceIndex' which is modified, is invoked when ever a new tier to VPC is added (virtual NIC is hot-plugged). Many VPC tests run againest

[GitHub] cloudstack issue #1680: CLOUDSTACK-9498: VR CsFile search utility methods fa...

2016-09-26 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1680 @jburwell changed the existing marvin test to use password as "p@a#s=s.w_o-r+d", which will cover the all chars permitted for VPN user password --- If your project is set up f

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

2016-09-26 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1681 @sateesh-chodapuneedi added optimization for now, but its better we eliminate the whole logic of finding device index from the VR. --- If your project is set up for it, you can reply to

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-09-26 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1659 @ustcweizhou thanks, i have fixed it earlier today and tested, am trying to fix other test failures and re-run the CI --- If your project is set up for it, you can reply to this email and

Re: ACS 4.9 + VMware: Failed to deploy VPC VR due to IPAssoc failure

2016-09-26 Thread Murali Reddy
findRouterEthDeviceIndex() in VmwareResource returned -1. Looks like CLOUDSTACK-9491: incorrect parsing of device list to find ethernet index of plugged NIC There is a PR https://github.com/apache/cloudstack/pull/1681 On 26/09/16, 5:13 PM, "Prashanth Manthena" wrote: >Hi All, > >We are h

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-09-26 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1659 @jburwell i am investigating them. Just so you know, blueorangutan is running full smoke test suite. Similar failures are seen in other PR as well. So some of the tests may have been

[GitHub] cloudstack issue #1683: fix ping tests to properly recognise successful ping...

2016-09-25 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1683 Just ran in to this issue. Fix LGTM. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] cloudstack issue #1680: CLOUDSTACK-9498: VR CsFile search utility methods fa...

2016-09-23 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1680 @cloudmonger error seems to be due to 'physicalnetwork 200 has allocated vnets in the range 731-732' This is same error seen in test run. After test run in done, please see i

[GitHub] cloudstack issue #1674: CLOUDSTACK-9460: For long running transactions, if t...

2016-09-22 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1674 LGTM from the code change. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

2016-09-22 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1681 @serg38 Though same code path, i ran into different issue. When i was adding new tier to VPC, eth3 was plugged into VR from vCenter. As commented in the function there is delay for

Re: [DISCUSS] Replacing the VR

2016-09-22 Thread Murali Reddy
rchitecture and, >short of tracing code, was not successful at finding the information I >needed to satisfy myself that I know how it is designed. > >Thanks, > >Matthew Smart >President >Smart Software Solutions Inc. >108 S Pierre St. >Pierre, SD 57501 > >Phone: (605)

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-09-22 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1659 @blueorangutan test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-09-21 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1659 @blueorangutan package --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] cloudstack issue #1666: CLOUDSTACK-9480, CLOUDSTACK-9495 fix egress rule inc...

2016-09-21 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1666 @cloudmonger thanks test_extendPhysicalNetworkVlan failed due to 'physicalnetwork 200 has allocated vnets in the range 734-735'. Possibly due to stale network that was n

  1   2   3   4   >