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
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
-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
, 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
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 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 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 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 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 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 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 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 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
+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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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
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 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 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 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 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 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
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 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 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 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 - 100 of 330 matches
Mail list logo