> Op 8 juni 2016 om 3:01 schreef Will Stevens <williamstev...@gmail.com>: > > > Yes. I agree. I need to get some commitment from someone to test it though. > I need to figure out how to test it once it gets in. Any takers for > validating this works after it goes in?
We tested it as it's running in production. In the PR ( https://github.com/apache/cloudstack/pull/1547 ) you can see that Daan als did tests which came out positive. A additional pair of eyes on this one would be great so that it makes it in to 4.9 Wido > On Jun 7, 2016 8:54 PM, "ilya" <ilya.mailing.li...@gmail.com> wrote: > > Sounds like a bug fix to me.. Bug fixes should be allowed in my opinion. > > On 6/7/16 4:07 AM, Wido den Hollander wrote: > >> Op 28 mei 2016 om 0:34 schreef Will Stevens <williamstev...@gmail.com>: > >> > >> > >> Hey All, > >> I think I have done what I can do at this point. I am sorry if you have > a > >> PR that you wanted to get in that didn't make it. I pushed my deadline > for > >> the freeze a bit because I had a lot of PRs that were close and I was > able > >> to get a bunch of them in. > >> > >> I plan to wait about a week before I cut the first RC to give people a > >> chance to test master and get me the details of their testing. This will > >> reduce the number of RCs we will need to have in order to get this > release > >> out the door. > >> > >> Please start testing master and let me know if you run into any issues. > >> There are a couple periodic issues that show up in my CI environments, > so I > >> will probably spend some time to see if I can get those sorted out > before I > >> cut the first release. > >> > >> I plan to create a Github PR that will never be merged where I can post > CI > >> results against master for this release so we can troubleshoot anything > we > >> find. This approach is mainly because my workflow with `upr` lets me > post > >> results easily and include the logs for the run. > >> > > > > I would like to get some attention for this PR: > https://github.com/apache/cloudstack/pull/1547 > > > > Currently in Basic Networking multiple ranges per VLAN/POD is broken, the > VR doesn't take the additional IP addresses. > > > > That PR fixes that (and is running in production at us), but this has > been broken since 4.7 > > > > I would like 4.9 to have this fixed as it's a known issue right now. Can > we do that? > > > > Wido > > > >> Cheers, > >> > >> Will