<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Sep 26, 2013 at 3:12 PM, Michael Davies <span dir="ltr"><<a href="mailto:michael@the-davies.net" target="_blank">michael@the-davies.net</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Mon, Sep 23, 2013 at 6:20 PM, Thierry Carrez <<a href="mailto:thierry@openstack.org">thierry@openstack.org</a>> wrote:<br>


> Monty Taylor wrote:<br>
> > On 09/20/2013 02:47 PM, Michael Still wrote:<br>
> >> Before <a href="https://review.openstack.org/#/c/46867/" target="_blank">https://review.openstack.org/#/c/46867/</a> if file injection of a<br>
> >> mandatory file fails, nova just silently ignores the failure, which is<br>
> >> clearly wrong. However, that review now can't land because its<br>
> >> revealed another failure in the file injection code via tempest, which<br>
> >> is...<br>
> >><br>
> >> Should file injection work for instances which are boot from volume?<br>
> >> Now that we actually notice injection failures we're now failing to<br>
> >> boot such instances as file injection for them doesn't work.<br>
> >><br>
> >> I'm undecided though -- should file injection work for boot from<br>
> >> volume at all? Or should we just skip file injection for instances<br>
> >> like this? I'd prefer to see us just support config drive and metadata<br>
> >> server for these instances, but perhaps I am missing something really<br>
> >> important.<br>
> ><br>
> > Well, first of all, I think file injection should DIAF everywhere.<br>
><br>
> +1<br>
><br>
> > That said, it may be no surprise that I think boot-from-volume should<br>
> > just do config drive and metadata.<br>
><br>
> That sounds like the simplest way to preserve behavior. From what you<br>
> said the current behavior is "try, fail and ignore failure" -- having<br>
> noop instead is probably the right thing to do for havana.<br>
<br>
</div>This behaviour is what is causing <a href="https://bugs.launchpad.net/nova/+bug/1188543" target="_blank">https://bugs.launchpad.net/nova/+bug/1188543</a><br>
<br>
I've submitted a patch (<a href="https://review.openstack.org/#/c/48533/" target="_blank">https://review.openstack.org/#/c/48533/</a>) that<br>
addresses the issue.<br>
<br>
It appears that:<br>
1) File injection for instances which are boot from volume doesn't<br>
appear to have ever worked.<br>
2) Attempting file injection just fails quietlyish and causes instance<br>
spawning slowdown<br>
3) The code needed to do this properly isn't trivial and probably<br>
wouldn't land in Havana so late in the cycle.<br>
<br>
Instead of attempting file injection on a boot volume, my patch simply<br>
LOG.warns the user. I think that's the best solution for now. However<br>
I think we should address file injection in Icehouse as discussed in<br>
this thread.<br></blockquote><div><br></div><div>++</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Thanks in advance,<br>
<br>
Michael...<br>
<span class="HOEnZb"><font color="#888888">--<br>
Michael Davies   <a href="mailto:michael@the-davies.net">michael@the-davies.net</a><br>
Rackspace Australia<br>
</font></span><div class="HOEnZb"><div class="h5"><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>
</div></div></blockquote></div><br></div></div>