<div dir="ltr"><font face="georgia, serif">Hi Michal,</font><div><font face="georgia, serif"><br></font></div><div><font face="georgia, serif">Apart from reviewing your changes, few more efforts done from TaskFlow side that I find useful for Cinder's case (and in general).</font><div>
<p class=""><span class=""><a href="https://review.openstack.org/#/c/186524/"><font face="georgia, serif">https://review.openstack.org/#/c/186524/</font></a></span></p><p class=""><span class=""><font face="georgia, serif"><a href="https://review.openstack.org/#/c/184814/"><span class="">https://review.openstack.org/#/c/184814/</span></a> </font></span></p><p class=""><span class=""><font face="georgia, serif">-Vilobh</font></span></p></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jun 2, 2015 at 2:45 AM, Dulko, Michal <span dir="ltr"><<a href="mailto:michal.dulko@intel.com" target="_blank">michal.dulko@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div lang="EN-US" link="blue" vlink="purple">
<div>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d">Right now we’re working on refactoring current TaskFlow implementations in Cinder to make them more readable and clean. Then we’ll be able to decide if we want
to get more TaskFlow into Cinder or step back from using it. Deadline for refactoring work is around 1 of July.<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d">Here’s related patch for scheduler’s create_volume workflow:
<a href="https://review.openstack.org/#/c/186439/" target="_blank">https://review.openstack.org/#/c/186439/</a><u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d">Currently I’m working on a patch for API’s create_volume and John Griffith agreed to work on manager’s one (I don’t know current status). If you want to help
with these efforts – reviews are always welcomed. You may also take a shot at refactoring manage_existing flow in manager. It seem simple enough but maybe there are some improvement that we can do to make it more readable.<u></u><u></u></span></p>
<p class="MsoNormal"><a name="14db3ac81a4360e3__MailEndCompose"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d"><u></u> <u></u></span></a></p>
<div style="border:none;border-left:solid blue 1.5pt;padding:0in 0in 0in 4.0pt">
<div>
<div style="border:none;border-top:solid #e1e1e1 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">From:</span></b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"> hao wang [mailto:<a href="mailto:sxmatch1986@gmail.com" target="_blank">sxmatch1986@gmail.com</a>]
<br>
<b>Sent:</b> Tuesday, June 2, 2015 11:30 AM<br>
<b>To:</b> <a href="mailto:openstack-dev@lists.openstack.org" target="_blank">openstack-dev@lists.openstack.org</a><br>
<b>Subject:</b> [openstack-dev] [Cinder] [Taskflow]Review help to cinder bp:Implement function to manage/unmanage snapshots<u></u><u></u></span></p>
</div>
</div><div><div class="h5">
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<p class="MsoNormal">Hi, folks, <u></u><u></u></p>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">There is a cinder bp:<span style="font-size:9.0pt;color:black">Implement function to manage/unmanage snapshots(<a href="https://review.openstack.org/#/c/144590/" target="_blank">https://review.openstack.org/#/c/144590/</a>), that we use taskflow to implement
this feature. </span><u></u><u></u></p>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:9.0pt;color:black">So I need your guys' help(cinder & taskflow) to push this forward.</span><u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:9.0pt;color:black">Thanks.</span><u></u><u></u></p>
</div>
<div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<p class="MsoNormal">-- <u></u><u></u></p>
<div>
<pre>Best Wishes For You!<u></u><u></u></pre>
</div>
</div>
</div>
</div>
</div>
</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" 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></blockquote></div><br></div>