<div dir="ltr">Hi,<div><br></div><div>Not approved.</div><div><br></div><div>Unfortunately it seems like dangerous patch, API changing and Liberty client will not support it as well. Let's have it merged early in M cycle (after Liberty RC published, master will be open for M).</div><div><br></div><div>Thanks.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 11, 2015 at 12:06 AM, Ethan Gafford <span dir="ltr"><<a href="mailto:egafford@redhat.com" target="_blank">egafford@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div style="font-family:times new roman,new york,times,serif;font-size:12pt;color:#000000"><div>Sadly, in reviewing the client code, Vitaly is right; the current client will not support this feature, which would make it direct REST call only. Given that, I am uncertain it is worth the risk. I'd really love to have seen this go in; it's a great feature and a lot of work has gone into it, but perhaps it should be a great feature in M.</div><div><br></div><div>If we agree to cut a new client point release, I could see this, but for now I fear I'm -1.</div><div><br></div><div>Thanks,</div><div>Ethan<br><br>>From: "lu jander" <<a href="mailto:juvenboy1987@gmail.com" target="_blank">juvenboy1987@gmail.com</a>><br>>To: "OpenStack Development Mailing List (not for usage questions)" <<a href="mailto:openstack-dev@lists.openstack.org" target="_blank">openstack-dev@lists.openstack.org</a>><br>>Sent: Monday, September 7, 2015 1:26:49 PM<br>>Subject: Re: [openstack-dev] [sahara] FFE request for scheduler and suspend EDP job for sahara<div><div class="h5"><br>><br>>Hi Vitaly,<br>>enable-scheduled-edp-jobs patch has 34 patch sets review. <a href="https://review.openstack.org/#/c/182310/" target="_blank">https://review.openstack.org/#/c/182310/</a> , it has no impact with another working in process patch.<br>><br>>2015-09-07 18:48 GMT+08:00 Vitaly Gridnev <<a href="mailto:vgridnev@mirantis.com" target="_blank">vgridnev@mirantis.com</a>>:<br>><br>> Hey!<br>><br>> From my point of view, we definetly should not give FFE for add-suspend-resume-ability-for-edp-jobs spec, because client side for this change is not included in official liberty release.<br>><br>> By the way, I am not sure about FFE for enable-scheduled-edp-jobs, because it's not clear which progress of these blueprint. Implementation of that consists with 2 patch-sets, and one of that marked as Work In Progress.<br>> <br>><br>> On Sun, Sep 6, 2015 at 7:18 PM, lu jander <<a href="mailto:juvenboy1987@gmail.com" target="_blank">juvenboy1987@gmail.com</a>> wrote:<br>><br>> Hi, Guys<br>><br>> I would like to request FFE for scheduler EDP job and suspend EDP job for sahara. these patches has been reviewed for a long time with lots of patch sets.<br>><br>> Blueprint:<br>><br>> (1) <a href="https://blueprints.launchpad.net/sahara/+spec/enable-scheduled-edp-jobs" target="_blank">https://blueprints.launchpad.net/sahara/+spec/enable-scheduled-edp-jobs</a> <br>> (2) <a href="https://blueprints.launchpad.net/sahara/+spec/add-suspend-resume-ability-for-edp-jobs" target="_blank">https://blueprints.launchpad.net/sahara/+spec/add-suspend-resume-ability-for-edp-jobs</a> <br>><br>> Spec:<br>><br>> (1) <a href="https://review.openstack.org/#/c/175719/" target="_blank">https://review.openstack.org/#/c/175719/</a><br>> (2) <a href="https://review.openstack.org/#/c/198264/" target="_blank">https://review.openstack.org/#/c/198264/</a><br>><br>><br>> Patch:<br>><br>> (1) <a href="https://review.openstack.org/#/c/182310/" target="_blank">https://review.openstack.org/#/c/182310/</a><br>> (2) <a href="https://review.openstack.org/#/c/201448/" target="_blank">https://review.openstack.org/#/c/201448/</a> <br>><br>> __________________________________________________________________________<br>> OpenStack Development Mailing List (not for usage questions)<br>> Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</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>><br>><br>><br>> -- <br>> Best Regards,<br>> Vitaly Gridnev<br>> Mirantis, Inc<br>><br>> __________________________________________________________________________<br>> OpenStack Development Mailing List (not for usage questions)<br>> Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</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>><br>>__________________________________________________________________________<br>>OpenStack Development Mailing List (not for usage questions)<br>>Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</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><br></div></div></div></div></div><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>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">Sincerely yours,<br>Sergey Lukjanov<br>Sahara Technical Lead<br>(OpenStack Data Processing)<br>Principal Software Engineer<br>Mirantis Inc.</div>
</div>