<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 16 June 2015 at 03:12, Dmitry Tantsur <span dir="ltr"><<a href="mailto:dtantsur@redhat.com" target="_blank">dtantsur@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 06/16/2015 08:58 AM, Ramakrishnan G wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Hi All,<br>
<br>
Some time back we created a new repository[1] to move all the reusable<br>
code components of Ironic to a separate library.  The branched out code<br>
has changed and there has been a review out to sync it [2].  But<br>
unfortunately, it has got stale again as some more changes have gone in<br>
to the branched out code.  To avoid repeated efforts of such syncing, I<br>
suggest we sync the latest code from Ironic to ironic-lib (in<br>
appropriate files) and immediately change Ironic to start using it.<br>
<br>
I suggest we can do the following:<br>
1) Decide on a timeline for the change (1 or 2 days)<br>
</blockquote>
<br></span>
Now is a good time, IMO, I don't think we're in pressing need to change this code.<span class=""><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
2) Stop +Aing changes in Ironic to the files/code being moved to ironic-lib<br>
3) Sync the latest code in ironic-lib and merge it<br>
4) Make a new release of ironic-lib<br>
5) Make changes in Ironic to use ironic-lib and make sure gate is back<br>
up and running again (I can't think of anything that will break gate on<br>
switching to ironic-lib as it's just a pip install)<br>
</blockquote>
<br></span>
Note that this will need adding ironic-lib to global-requirements, which will take time, unless you grab a couple of g-r cores to do it asap.<span class=""><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
6) Make new reviews in ironic-lib for any pending reviews in Ironic<br>
<br>
If we come to an agreement on #1 and #2 above, Syed Ismail Faizan<br>
Barmawer can continue to work on #3 - #5<br>
<br>
Let me know if it will work out or if there are any better plans (or I<br>
am missing something)<br>
</blockquote>
<br></span>
Otherwise plan LGTM<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
<br>
Thanks.<br>
<br>
[1] <a href="https://github.com/openstack/ironic-lib" rel="noreferrer" target="_blank">https://github.com/openstack/ironic-lib</a><br>
[2] <a href="https://review.openstack.org/#/c/162162/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/162162/</a><br>
<br>
Regards,<br>
Ramesh<br></span></blockquote></blockquote><div><br></div><div>If we haven't yet released a version of ironic-lib, I suggest taking a more conservative (but more work) approach:</div><div>0.1. sync the latest code in ironic-lib (this is optional)</div><div>0.2. make a first release of ironic-lib</div><div>0.3. add ironic-lib to global-requirements</div><div><br></div><div>Then the steps you suggested Ramesh. (Changes need to be made to IPA too? Not sure what code is being copied.)</div><div><br></div><div>Hopefully that will get any kinks out of the process, and will give us an idea of how long that process might take. (Eg, there are only certain people that can do releases, and if we can get things set up in global-requirements sooner rather than later, that is one less thing to do). </div><div><br></div><div>--ruby</div><div><br></div></div></div></div>