<div dir="ltr">Yup,<div><br></div><div>I'm expecting all FFEs to be merged by the EOW, otherwise we'll be revisiting case by case with reject by default.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 10, 2016 at 1:38 PM, Doug Hellmann <span dir="ltr"><<a href="mailto:doug@doughellmann.com" target="_blank">doug@doughellmann.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Excerpts from lu jander's message of 2016-03-07 14:28:21 +0800:<br>
<span class="">> Hi folks,<br>
><br>
> I would like to request a FFE for the feature “Resume EDP job”:<br>
><br>
><br>
><br>
> BP:<br>
> <a href="https://blueprints.launchpad.net/sahara/+spec/add-suspend-resume-ability-for-edp-jobs" rel="noreferrer" target="_blank">https://blueprints.launchpad.net/sahara/+spec/add-suspend-resume-ability-for-edp-jobs</a><br>
</span>> <<a href="https://blueprints.launchpad.net/sahara/+spec/improving-anti-affinity" rel="noreferrer" target="_blank">https://blueprints.launchpad.net/sahara/+spec/improving-anti-affinity</a>><br>
><br>
><br>
> <<a href="https://blueprints.launchpad.net/sahara/+spec/improving-anti-affinity" rel="noreferrer" target="_blank">https://blueprints.launchpad.net/sahara/+spec/improving-anti-affinity</a>><br>
<span class="">><br>
> Spec has been merged. <a href="https://review.openstack.org/#/c/198264/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/198264/</a><br>
><br>
><br>
> Suspend EDP patch has been merged.<br>
> <a href="https://review.openstack.org/#/c/201448/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/201448/</a><br>
</span>> <<a href="https://review.openstack.org/#/c/201448/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/201448/</a>><br>
><br>
><br>
> <<a href="https://review.openstack.org/#/c/210839/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/210839/</a>><br>
><br>
> Code Review: <a href="https://review.openstack.org/#/c/285839/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/285839/</a><br>
> <<a href="https://review.openstack.org/#/c/218638/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/218638/</a>><br>
<span class="">><br>
><br>
><br>
> code is ready for review.<br>
><br>
><br>
><br>
> The Benefits for this change: after suspend job, we can resume this job.<br>
><br>
><br>
><br>
> The Risk: The risk would be low for this patch, since the code of suspend<br>
> patch has been long time reviewed.<br>
><br>
><br>
><br>
> Thanks,<br>
><br>
> luhuichun<br>
<br>
</span>Both <a href="https://review.openstack.org/#/c/285839/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/285839/</a> and<br>
<a href="https://review.openstack.org/#/c/218638/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/218638/</a> have -1 votes on them. We will<br>
be tagging RC1 next week, so if the team wants to include this feature<br>
you should work on getting the patches into shape tomorrow to give you<br>
time to test them before the release candidate is tagged.<br>
<span class="HOEnZb"><font color="#888888"><br>
Doug<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
__________________________________________________________________________<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>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div>Sincerely yours,<br>Sergey Lukjanov<br>Principal Software Engineer<br>Mirantis Inc.</div></div></div>
</div>