<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 17 September 2015 at 21:50, Jim Rollenhagen <span dir="ltr"><<a href="mailto:jim@jimrollenhagen.com" target="_blank">jim@jimrollenhagen.com</a>></span> wrote:<snip></div><div class="gmail_quote"><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
This may also mean in-band RAID configuration may not land; the<br>
interface in general did land, and drivers may do out-of-band<br>
configuration. We assumed that in-band RAID would be done through<br>
zapping. However, if folks can agree on how to do it during automated<br>
cleaning, I'd be happy to get that in Liberty if the code is not too<br>
risky. If it is risky, we'll need to punt it to Mitaka as well.<br></blockquote><div><br></div><div>Ramesh had worked on this but removed the part that hooks into automated cleaning [0]. One reason being that if a target RAID config wasn't specified, does this mean the create-raid-config clean step is skipped (considered successful), or does it mean that the cleaning operation fails. After a bit of discussion on IRC[1], some of us think that having the clean operation fail makes sense.</div><div><br></div><div>There are two patches that will help with this [2] & [3]. The code was extracted from [0]. I think it may need more work but I won't be able to look into it until next week. In the meantime, hopefully someone else (Ramesh? anyone?) will pick it up :)</div><div><br></div><div>--ruby</div><div><br></div><div>[0] <a href="https://review.openstack.org/#/c/198238/">https://review.openstack.org/#/c/198238/</a>, removed from revision 21</div><div>[1] 2015-09-17T21:21:54 ish, <a href="http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2015-09-17.log">http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2015-09-17.log</a></div><div>[2] <a href="https://review.openstack.org/#/c/222264/">https://review.openstack.org/#/c/222264/</a></div><div>[3] <a href="https://review.openstack.org/#/c/224938/">https://review.openstack.org/#/c/224938/</a></div></div></div></div>