On Mon, Jan 30, 2012 at 1:05 PM, Brian Waldon <span dir="ltr"><<a href="mailto:brian.waldon@rackspace.com">brian.waldon@rackspace.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word">After implementing a working version of file injection on Libvirt, a good question was brought up on the merge prop: how should we handle a file injection failure? Injection could fail for several reasons: missing necessary libraries, unsupported image formats and bad permissions are just a few. There seem to be two clear paths forward:<div>
<br></div><div>1) Log an error, set the instance to ERROR, add an asynchronous fault to the instance in the db</div><div>2) Log a warning, move on with the boot process</div></div></blockquote><div><br></div><div><br></div>
<div><div>My preference would be to log a warning and move on with the boot process (#2). Or perhaps we could address this with some sort of async error messages concept?</div><div><br></div><div>Also, Armando just file this ticket to change how XenServer currently handles admin password failures:</div>
<div><br></div><div> <a href="https://bugs.launchpad.net/nova/+bug/923865">https://bugs.launchpad.net/nova/+bug/923865</a></div><div><br></div><div>I understand file injection is slightly different than admin passwords but it would seem the preference is to treat these types of failures as warnings and not errors.</div>
</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div>It's not obvious which of these is the best route to take from a user's point of view. I'm currently leaning towards option 1 as I wouldn't want to have an instance come up (and be billed for it) while it wasn't what I explicitly requested. </div>
<div><br></div><div>I would love to get some help with this problem. You can either reply directly to this email, or head over to the merge prop: <a href="https://review.openstack.org/#change,3526" target="_blank">https://review.openstack.org/#change,3526</a></div>
<span class="HOEnZb"><font color="#888888"><div><br></div><div><br></div><div>Brian Waldon</div><div><br></div></font></span></div><br>_______________________________________________<br>
Mailing list: <a href="https://launchpad.net/~openstack" target="_blank">https://launchpad.net/~openstack</a><br>
Post to     : <a href="mailto:openstack@lists.launchpad.net">openstack@lists.launchpad.net</a><br>
Unsubscribe : <a href="https://launchpad.net/~openstack" target="_blank">https://launchpad.net/~openstack</a><br>
More help   : <a href="https://help.launchpad.net/ListHelp" target="_blank">https://help.launchpad.net/ListHelp</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br>Dan Prince<div><a href="mailto:princepd@alumni.jmu.edu" target="_blank">princepd@alumni.jmu.edu</a></div><br>