<div dir="ltr">Hey Paul,<div><br></div><div>This is by design, and reraising a single MechanismDriverError was really to have a nice defined API for the MechanismManager class, avoid blanket try/except calls in the caller. But I do agree that it's really annoying to lose the information about the underlying exception. I like your idea of including the original exception text in the MechanismDriverError message, I think that'd help a lot.</div>

<div><br></div><div>Andre</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jan 24, 2014 at 1:19 PM, Paul Ward <span dir="ltr"><<a href="mailto:wpward@us.ibm.com" target="_blank">wpward@us.ibm.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>
<p><font face="sans-serif">In implementing a mechanism driver for ML2 today, I discovered that any exceptions thrown from your mechanism driver will get swallowed by the ML2 manager (</font><a href="https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/managers.py" target="_blank"><font face="sans-serif">https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/managers.py</font></a><font face="sans-serif"> at line 164).</font><br>


<br>
<font face="sans-serif">Is this by design?  Sure, you can look at the logs, but it seems more user friendly to reraise the exception that got us here.  There could be multiple mechanism drivers being called in a chain, so changing this to reraise an exception that got us in trouble would really only be able to reraise the last exception encountered, but it seems that's better than none at all.  Or maybe even keep a list of exceptions raised and put all their texts into the MechanismDriverError message.</font><br>


<br>
<font face="sans-serif">Thoughts?</font></p></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>