<div dir="ltr">Hi Barmawer,<div><br></div><div>Currently the Ironic Nova driver is blocked from merging. The Ironic team is working on getting all the pieces in place for our C.I. testing. At this point I would say your best path is to create your patch with 51328 as a dependency. Please note that the nova driver will most likely be going through several more revisions as we get closer. This will mean that your dependent patch will need to rebased as new Nova driver patches are pushed up. This is very common, I am just pointing it out so that you can keep an eye out for the "[OUTDATED]" tag on the review. Also please tag your dependent patch with "<span style="color:rgb(0,0,0);font-family:'Lucida Console','Lucida Sans Typewriter',Monaco,monospace;font-size:11px;white-space:pre">implements </span>bp:deprecate-baremetal-driver" this will ensure your patch is added to the Blue Print, and make it clear that is part of the deprecate-baremetal-driver patch set.</div>
<div><br></div><div><br></div><div>Chris Krelle</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Feb 24, 2014 at 6:05 AM, Faizan Barmawer <span dir="ltr"><<a href="mailto:faizan.barmawer@gmail.com" target="_blank">faizan.barmawer@gmail.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">Hi All,<br><br>I am currently working on ilo driver for ironic project.<br>As part of this implementation and to integrate with nova ironic driver (<a href="https://review.openstack.org/#/c/51328/" target="_blank">https://review.openstack.org/#/c/51328/</a>) we need to make changes to "driver.py" and "ironic_driver_fields.py" files, to pass down ilo driver specific fields to the ironic node. Since nova ironic driver code review still in progress and not yet integrated into openstack, we have not included this piece of code in the ilo driver code review patch (<a href="https://review.openstack.org/#/c/73787/" target="_blank">https://review.openstack.org/#/c/73787/</a>).<div>

<br></div><div>We need your suggestion on delivering this part of ilo driver code change in nova ironic driver. </div><div>- Should we wait for the completion of nova ironic driver and then raise a defect to submit these changes? or</div>

<div>- should we raise a defect now and submit for review, giving the dependency on the nova ironic driver review? or</div><div>- Can we use the existing blueprint for ilo driver to raise a separate review for this code change giving nova ironic driver as dependency?</div>

<div><br></div><div>Please suggest a better way of delivering these changes. </div><div><br></div><div>Thanks & Regards,</div><div>Barmawer</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></div>