Re: Can't create a zone using master

2017-02-19 Thread Rajani Karuturi
Thanks Sigert. ~ Rajani http://cloudplatform.accelerite.com/ On February 17, 2017 at 5:53 PM, Sigert GOEMINNE (sigert.goemi...@nuagenetworks.net) wrote: Hi all, We have a new PR that fixes this issue: https://github.com/apache/cloudstack/pull/1952 Kind regards, *Sigert Goeminne* Software

Re: Can't create a zone using master

2017-02-17 Thread Sigert GOEMINNE
Hi all, We have a new PR that fixes this issue: https://github.com/apache/cloudstack/pull/1952 Kind regards, *Sigert Goeminne* Software Development Engineer *nuage*networks.net Copernicuslaan 50 2018 Antwerp Belgium On Fri, Feb 17, 2017 at 8:55 AM, Kris Sterckx <

Re: Can't create a zone using master

2017-02-16 Thread Kris Sterckx
Thanks Mike Investigating. Kris On 17 February 2017 at 08:18, Tutkowski, Mike wrote: > I added a comment on the code of the relevant PR, but the PR is in the > Merged state, so I don't know if people receive e-mail notifications then. > > > On Feb 17, 2017, at 12:16

Re: Can't create a zone using master

2017-02-16 Thread Tutkowski, Mike
I added a comment on the code of the relevant PR, but the PR is in the Merged state, so I don't know if people receive e-mail notifications then. > On Feb 17, 2017, at 12:16 AM, Wei ZHOU wrote: > > Maybe we need ask Nuage why the lines are added. > > > 2017-02-17 7:54

Re: Can't create a zone using master

2017-02-16 Thread Wei ZHOU
Maybe we need ask Nuage why the lines are added. 2017-02-17 7:54 GMT+01:00 Tutkowski, Mike : > Hopefully we can now find someone with experience in CloudStack's > networking code to tell us if that fix is OK or if that was just a Band-Aid > and we should do something

Re: Can't create a zone using master

2017-02-16 Thread Tutkowski, Mike
Hopefully we can now find someone with experience in CloudStack's networking code to tell us if that fix is OK or if that was just a Band-Aid and we should do something more thorough to correct the issue. > On Feb 16, 2017, at 11:48 PM, Rajani Karuturi wrote: > > Thanks for

Re: Can't create a zone using master

2017-02-16 Thread Rajani Karuturi
Thanks for digging it Mike. ~ Rajani http://cloudplatform.accelerite.com/ On February 17, 2017 at 12:18 AM, Tutkowski, Mike (mike.tutkow...@netapp.com) wrote: Hi, I opened a blocker for 4.10 regarding this issue: https://issues.apache.org/jira/browse/CLOUDSTACK-9790 Thanks, Mike On

Re: Can't create a zone using master

2017-02-16 Thread Tutkowski, Mike
Hi, I opened a blocker for 4.10 regarding this issue: https://issues.apache.org/jira/browse/CLOUDSTACK-9790 Thanks, Mike On 2/16/17, 8:35 AM, "Tutkowski, Mike" wrote: I believe the issue is here: https://github.com/apache/cloudstack/pull/1579

Re: Can't create a zone using master

2017-02-16 Thread Tutkowski, Mike
I believe the issue is here: https://github.com/apache/cloudstack/pull/1579 In ConfigurationManagerImpl.java, it appears the lines that led to NPEs being thrown were added on lines 2924 – 2926: https://github.com/apache/cloudstack/pull/1579/files On 2/16/17, 2:42 AM, "Wido den Hollander"

Re: Can't create a zone using master

2017-02-16 Thread Wido den Hollander
> Op 16 februari 2017 om 6:21 schreef Rajani Karuturi : > > > unfortunately, running one set of tests itself is very time > consuming(multiplied by hypervisors, network modes etc. its > impossible to run all) > > If I remember correctly, wido tested it with PR 1700 (IPV6 in

Re: Can't create a zone using master

2017-02-15 Thread Tutkowski, Mike
For the CloudStack networking person who might investigate this, here is the code I changed per Wei’s comments to get this to work: https://github.com/mike-tutkowski/cloudstack/commit/fdb4fd1e4b4fbb61d8e30006fceca86b8664bb5e On 2/15/17, 11:22 PM, "Tutkowski, Mike"

Re: Can't create a zone using master

2017-02-15 Thread Tutkowski, Mike
Hi Rajani, It looks like a combination of actions has fixed this for me: 1) From Wei: You may remove the "else" part which does not exist in previous versions in ConfigurationManagerImpl.java. 2) I made use of the new system VM template. It looks like we have a bug in ConfigurationManagerImpl.

Re: Can't create a zone using master

2017-02-15 Thread Rajani Karuturi
This is the order in which they are merged. * 7233ac3 N Merge pull request #977 from ustcweizhou/vm-snapshot 2 weeks ago [Rajani Karuturi] * f10c8bf N Merge pull request #1700 from wido/ipv6-basic-networking 3 weeks ago [Rajani Karuturi] * 4721c53 N Merge pull request #1749 from

Re: Can't create a zone using master

2017-02-15 Thread Tutkowski, Mike
Actually Rajani, I just saw the email you sent out about a new system VM template for master. I can try to make use of that new template and see if that fixes my agent issues. > On Feb 15, 2017, at 10:21 PM, Rajani Karuturi wrote: > > unfortunately, running one set of

Re: Can't create a zone using master

2017-02-15 Thread Tutkowski, Mike
Hi Rajani, Just to make sure I'm running with the right code, can you provide me the SHA that you'd like me to test with? Thanks! Mike > On Feb 15, 2017, at 10:21 PM, Rajani Karuturi wrote: > > unfortunately, running one set of tests itself is very time >

Re: Can't create a zone using master

2017-02-15 Thread Rajani Karuturi
unfortunately, running one set of tests itself is very time consuming(multiplied by hypervisors, network modes etc. its impossible to run all) If I remember correctly, wido tested it with PR 1700 (IPV6 in basic networking) Can you try a commit before and a commit after that PR? Thanks, ~

Re: Can't create a zone using master

2017-02-15 Thread Tutkowski, Mike
Hi Wei, OK, the situation is partially improved now: I can create a zone/pod/cluster. However, my SSVM and CPVM don’t report the agent as Running. From cloud.log in SSVM: Wed Feb 15 21:12:59 UTC 2017 Incompleted parameters STORAGE_IP:, STORAGE_NETMASK:, STORAGE_CIDR:. Cannot setup storage

Re: Can't create a zone using master

2017-02-15 Thread Tutkowski, Mike
Thanks, Wei. I will give that a shot and report back. On 2/15/17, 1:26 PM, "Wei ZHOU" wrote: Mike, You may remove the "else" part which does not exist in previous versions in ConfigurationManagerImpl.java. -Wei 2017-02-15 20:55 GMT+01:00

Re: Can't create a zone using master

2017-02-15 Thread Wei ZHOU
Mike, You may remove the "else" part which does not exist in previous versions in ConfigurationManagerImpl.java. -Wei 2017-02-15 20:55 GMT+01:00 Tutkowski, Mike : > OK, so, what I plan to do is jump back a bunch of commits and see if it > works. If it does, I can

Re: Can't create a zone using master

2017-02-15 Thread Tutkowski, Mike
OK, so, what I plan to do is jump back a bunch of commits and see if it works. If it does, I can binary search my way to the commit that fails it. If the commit I pick doesn’t work, I can jump back a bunch more. If that one works, I I can binary search my way to the commit that fails it, etc.

Re: Can't create a zone using master

2017-02-15 Thread Will Stevens
That does seem to be the case. :( *Will STEVENS* Lead Developer On Wed, Feb 15, 2017 at 2:32 PM, Tutkowski, Mike wrote: > Thanks for noting that PR, Rene. I tried building a Basic Zone making use > of code from that PR, but – unfortunately –

Re: Can't create a zone using master

2017-02-15 Thread Tutkowski, Mike
Thanks for noting that PR, Rene. I tried building a Basic Zone making use of code from that PR, but – unfortunately – I still get an exception (below). On a related note, does this mean we don’t run any integration tests that build a Basic Zone before we OK a PR to be checked in? ERROR

Re: Can't create a zone using master

2017-02-14 Thread Tutkowski, Mike
Thanks for pointing this out. I can incorporate those changes in the morning and see if it does the trick. > On Feb 15, 2017, at 12:50 AM, Rene Moser wrote: > > Probably related to https://github.com/apache/cloudstack/pull/1927 > > > >> On 02/14/2017 10:44 PM, Tutkowski,

Re: Can't create a zone using master

2017-02-14 Thread Rene Moser
Probably related to https://github.com/apache/cloudstack/pull/1927 On 02/14/2017 10:44 PM, Tutkowski, Mike wrote: > Hi, > > I’m getting a NullPointerException when trying to create a zone using master. > > Below is the relevant code in ConfigurationManagerImpl. > > In the else block,

Re: Can't create a zone using master

2017-02-14 Thread Tutkowski, Mike
If it helps in debugging, I typically assign 10.117.40.20 - 10.117.40.25 for Management IPs and 10.117.40.26 - 10.117.40.99 for Guest IPs. On the network in my lab, I am assigned the following range (by our IT people): 10.117.40.0 /25 (10.117.40.0 - 10.117.40.127) Subnet Mask: 255.255.255.128

Re: Can't create a zone using master

2017-02-14 Thread Tutkowski, Mike
Also, for any CloudStack networking person out there who is curious about this, I see the following in cloud.log for my SSVM: Wed Feb 15 01:07:50 UTC 2017 Incompleted parameters STORAGE_IP:, STORAGE_NETMASK:, STORAGE_CIDR:. Cannot setup storage network Wed Feb 15 01:07:50 UTC 2017 Not setting

Re: Can't create a zone using master

2017-02-14 Thread Tutkowski, Mike
I’m not very familiar with the networking code that’s failing, but I worked around it with these changes: https://github.com/mike-tutkowski/cloudstack/commit/834b7ea7df18f278cbc9eb1bfaa9578fad2debeb I’m not sure if those are viable changes in this case, but perhaps someone who works in the

Re: Can't create a zone using master

2017-02-14 Thread Tutkowski, Mike
By the way, this is for a Basic Zone. On 2/14/17, 2:44 PM, "Tutkowski, Mike" wrote: Hi, I’m getting a NullPointerException when trying to create a zone using master. Below is the relevant code in ConfigurationManagerImpl. In the else

Can't create a zone using master

2017-02-14 Thread Tutkowski, Mike
Hi, I’m getting a NullPointerException when trying to create a zone using master. Below is the relevant code in ConfigurationManagerImpl. In the else block, network.getCidr() returns null and NetUtil.getCidrNetmask then throws a NullPointerException. I noticed that network.getGateway() also