<div dir="ltr">Dmitry,<div>thank you for clarification of a status. In a meeting we had [1], I misinterpreted a status of a feature is being fully completed, and that is what I reflected in my FFEs post [2].</div><div><br></div><div>I'm not sure if we have definition of done written for Fuel specifically, but my expectation from the spec [3] was that Ubuntu bootstrap has to be default option. That means that all related code has to be merged.</div><div>One of the patches you've mentioned goes on since November, 27 and I'm not sure how long would it take to complete it. It is not a simple switch in a config file as I'd expect to see.</div><div>It's not a 100+ LOC changeset in very core either, so I call for Matt as a maintainer [4] and core reviewers to assess a situation.</div><div><br></div><div>My perspective is that we can test it right now without this patch and steps to do so are known [6], so feature is actually nearly complete except of additional action in fuel-menu. I hope that we could go an extra mile here and maintainer / cores could help with the patch.</div><div><br></div><div>I don't personally think that we could go on without having a clear deadline for when it will be enabled. Even if [4] is merged, but ubuntu is not a default - I see a serious risks here, as majority of our automated tests and lots of manual tests done by random people will still be using another image, thus significantly decreasing our chances to see regressions on time.</div><div><br></div><div>I'd give it an exception till end of Tuesday (as of after CentOS 7 expected merges) for full enablement, and that should have a fix to critical issue [7].<br></div><div><br></div><div>[1] <a href="https://etherpad.openstack.org/p/fuel-8.0-FF-meeting">https://etherpad.openstack.org/p/fuel-8.0-FF-meeting</a></div><div>[2] <a href="http://lists.openstack.org/pipermail/openstack-dev/2015-December/081131.html">http://lists.openstack.org/pipermail/openstack-dev/2015-December/081131.html</a></div><div>[3] <a href="http://specs.fuel-infra.org/fuel-specs-master/specs/8.0/dynamically-build-bootstrap.html">http://specs.fuel-infra.org/fuel-specs-master/specs/8.0/dynamically-build-bootstrap.html</a></div><div>[4] <a href="https://review.openstack.org/#/c/250662/">https://review.openstack.org/#/c/250662/</a></div><div>[5] <a href="https://github.com/openstack/fuel-menu/blob/master/MAINTAINERS">https://github.com/openstack/fuel-menu/blob/master/MAINTAINERS</a></div><div>[6] <a href="https://bugs.launchpad.net/fuel/+bug/1522406/comments/3">https://bugs.launchpad.net/fuel/+bug/1522406/comments/3</a></div><div>[7] <a href="https://bugs.launchpad.net/fuel/+bug/1522406/comments/3" style="z-index: 0;">https://bugs.launchpad.net/fuel/+bug/1522406</a></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Dec 3, 2015 at 10:59 AM Dmitry Klenov <<a href="mailto:dklenov@mirantis.com" target="_blank">dklenov@mirantis.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div><div><div><div><div><div><div>Hi folks,<br><br></div>Let me clarify the situation with Ubuntu bootstrap feature.<br><br></div>First of all, I would like to highlight that all functional commits for this feature were merged. This means that starting from yesterday everyone has an ability to switch to Ubuntu-based bootstrap manually and start using it. So I do not see the risk in loosing testing cycles in the community.<br><br></div>The item which brought concerns on today status meeting was the enablement of the feature by default. To do it, 2 patches have to be merged together:<br>* <a href="https://review.openstack.org/#/c/250662/" target="_blank">https://review.openstack.org/#/c/250662/</a> - main switch.<br>* <a href="https://review.openstack.org/#/c/251908/" target="_blank">https://review.openstack.org/#/c/251908/</a> - compatibility commit to QA suite to comply with new bootstrap config format.<br><br></div>I would like to raise the question if we can have a feature freeze exception for these 2 patches?<br><br></div>There are a couple of reasons why I consider it safer to merge these patches several days later:<br></div>* There is a bug caught today which will block the tests to pass if we switch to Ubuntu bootstrap by default: <a href="https://bugs.launchpad.net/fuel/+bug/1522406" target="_blank">https://bugs.launchpad.net/fuel/+bug/1522406</a><br></div>* There were concerns that a lot of FF commit merges can bring instability to QA suite. So it might be reasonable not to bring one more variable right now and to enable ubuntu bootstrap by default when all automated tests are stabilized.<br><br></div>I would like to ask engineering and QA leads to express their ideas on this.<br><br></div>Thanks,<br></div>Dmitry.<br></div>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div></div><div dir="ltr">-- <br></div><div dir="ltr">Mike Scherbakov<br>#mihgen</div>