<div dir="ltr">Dmitry,<div><br></div><div>Lets review the CR from the point of danger to current deployment process: in the essence it is 43 lines of change in puppet module. The module calls a shell script which always returns 0. So whatever happens inside, the deployment will not fail.</div><div><br></div><div>The only changes (non-get requests) the script does, it does to Sahara. It tries to upload cluster and node-group templates. That is not dangerous operation for Sahara - in the worst case the templates will just not be created and that is all. It will not affect Sahara correctness in any way.</div><div><br></div><div>On the other side, it is a nice UX feature we really want to have 5.1.1.</div><div><br></div><div>Thanks,</div><div><br></div><div>Dmitry</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">2014-11-15 3:04 GMT+03:00 Dmitry Borodaenko <span dir="ltr"><<a href="mailto:dborodaenko@mirantis.com" target="_blank">dborodaenko@mirantis.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">+286 lines a week after Feature Freeze, IMHO it's too late to make an<br>
exception for this one.<br>
<div><div class="h5"><br>
On Wed, Nov 12, 2014 at 7:37 AM, Dmitry Mescheryakov<br>
<<a href="mailto:dmescheryakov@mirantis.com">dmescheryakov@mirantis.com</a>> wrote:<br>
> Hello fuelers,<br>
><br>
> I would like to request you merging CR [1] which implements blueprint [2].<br>
> It is a nice UX feature we really would like to have in 6.0. On the other<br>
> side, the implementation is really small: it is a small piece of puppet<br>
> which runs a shell script. The script always exits with 0, so the change<br>
> should not be dangerous. Other files in the change are used in the shell<br>
> script only. Please consider reviewing and merging this though we've already<br>
> reached FF.<br>
><br>
> Thanks,<br>
><br>
> Dmitry<br>
><br>
> [1] <a href="https://review.openstack.org/#/c/132196/" target="_blank">https://review.openstack.org/#/c/132196/</a><br>
> [2]<br>
> <a href="https://blueprints.launchpad.net/mos/+spec/sahara-create-default-templates" target="_blank">https://blueprints.launchpad.net/mos/+spec/sahara-create-default-templates</a><br>
><br>
</div></div>> _______________________________________________<br>
> OpenStack-dev mailing list<br>
> <a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
><br>
<span class="HOEnZb"><font color="#888888"><br>
<br>
<br>
--<br>
Dmitry Borodaenko<br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</font></span></blockquote></div><br></div>