<div dir="ltr">Hi!<div>Regarding 1. I'd start with giving a generic message, e.g. "error while attaching volume" that does not point to a specific reason.</div><div>Regarding 2., if the reason is identifiable throwing the corresponding exception would come in handy. Only use the generic failure if there's no specific error knowledge.</div><div>I'm hitting that bug, too, great to see it addressed! :)</div><div>Best regards</div><div>Silvan</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">2016-05-18 21:57 GMT+02:00 Radhakrishnan, Siva <span dir="ltr"><<a href="mailto:siva.radhakrishnan@intel.com" target="_blank">siva.radhakrishnan@intel.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi All!<br>
Currently I am working on this bug <a href="https://bugs.launchpad.net/nova/+bug/1168011" rel="noreferrer" target="_blank">https://bugs.launchpad.net/nova/+bug/1168011</a> which says we have to change  error message displayed  when attaching a volume fails. Currently it catches all operation errors that libvirt can raise and assume that all of them are the source of device being busy. You can find the source of this code here <a href="https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L1160" rel="noreferrer" target="_blank">https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L1160</a>. I have few questions on this bug<br>
<br>
1.      what kind of error message and other required info should we include in the exception to make it look more      generalized instead of the current one ?<br>
<br>
2.      Should we raise separate exception for "Device is Busy" or a single general exception would work fine ?<br>
<br>
3.      If we need separate exception for device being busy what would be the equivalent libvirt error code  for that<br>
<br>
I would like to have your feedback/suggestions to proceed further with this bug.<br>
<br>
Regards,<br>
Siva<br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div><span style="color:rgb(0,0,0);font-size:small">Dr. Silvan Kaiser</span></div><div dir="ltr"><span style="color:rgb(0,0,0);font-size:small">Quobyte GmbH</span><br style="color:rgb(0,0,0);font-size:small"><span style="color:rgb(0,0,0);font-size:small">Hardenbergplatz 2, 10623 Berlin - Germany</span><br style="color:rgb(0,0,0);font-size:small"><span style="color:rgb(0,0,0);font-size:small">+49-30-814 591 800 - </span><a href="http://www.quobyte.com/" style="color:rgb(17,85,204);text-decoration:none;font-size:small" target="_blank">www.quobyte.com</a><span style="color:rgb(0,0,0);font-size:small"><</span><a href="http://www.quobyte.com/" style="color:rgb(17,85,204);text-decoration:none;font-size:small" target="_blank">http://www.quobyte.com/</a><span style="color:rgb(0,0,0);font-size:small">></span><br style="color:rgb(0,0,0);font-size:small"><span style="color:rgb(0,0,0);font-size:small">Amtsgericht Berlin-Charlottenburg, HRB 149012B</span><br style="color:rgb(0,0,0);font-size:small"><span style="color:rgb(0,0,0);font-size:small">Management board: Dr. Felix Hupfeld, Dr. Björn Kolbeck, Dr. Jan Stender</span><br></div></div></div></div></div></div></div></div>
</div>