<div dir="ltr">I'm fine with getting this patch in...<div>though a few things should be fixed first, in my opinion:</div><div><ol><li>I don't see a header in the blueprint, who is responsible for what. See example in the blueprint: <a href="https://blueprints.launchpad.net/fuel/+spec/send-anon-usage">https://blueprints.launchpad.net/fuel/+spec/send-anon-usage</a> (Feature Lead, QA, etc.)</li><li>There is no fuel-spec associated. If this is very minor thing to address, which doesn't affect anything beyond, then we probably better to track this as bug.</li><li>Currently, it's not very clear to me from description what this is for. Can you please provide more information, ideally what are the use cases, and acceptance criteria for proposed functionality?</li></ol><div>We have FF deadline not just because features can affect other features stability, but also because we will need time to assure they are of production quality by themselves. Slipping this to the end of the release naturally increases risks for quality.</div></div><div><br></div><div>Thanks,</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Nov 15, 2014 at 3:51 AM, Dmitry Mescheryakov <span dir="ltr"><<a href="mailto:dmescheryakov@mirantis.com" target="_blank">dmescheryakov@mirantis.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Oops, the last line should be read as<div>"<span style="font-family:arial,sans-serif;font-size:13px">On the other side, it is a nice UX feature we really want to have 6.0"</span></div><span class="HOEnZb"><font color="#888888"><div><span style="font-family:arial,sans-serif;font-size:13px"><br></span></div><div><span style="font-family:arial,sans-serif;font-size:13px">Dmitry</span></div></font></span></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">2014-11-15 3:50 GMT+03:00 Dmitry Mescheryakov <span dir="ltr"><<a href="mailto:dmescheryakov@mirantis.com" target="_blank">dmescheryakov@mirantis.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><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><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><br>
On Wed, Nov 12, 2014 at 7:37 AM, Dmitry Mescheryakov<br>
<<a href="mailto:dmescheryakov@mirantis.com" target="_blank">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" target="_blank">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><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" target="_blank">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>
</div></div></blockquote></div><br></div>
</div></div><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>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">Mike Scherbakov<br>#mihgen<br><br></div></div>
</div>