<br><br><div class="gmail_quote">On Wed, Nov 28, 2012 at 11:52 AM, Vishvananda Ishaya <span dir="ltr"><<a href="mailto:vishvananda@gmail.com" target="_blank">vishvananda@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<div><br>
On Nov 28, 2012, at 9:50 AM, Andy Grover <<a href="mailto:agrover@redhat.com" target="_blank">agrover@redhat.com</a>> wrote:<br>
<br>
><br>
> It is designed entirely for interactive use. Local programmatic<br>
> configuration should be done via rtslib. Remote programmatic<br>
> configuration should be done via targetd.<br>
><br>
> If neither of these is really an option, I'd actually prefer you write a<br>
> custom executable wrapper around rtslib that does what you need. This<br>
> will also avoid pulling in the dependencies needed for targetcli's<br>
> interactive capability.<br>
><br>
> I am of the opinion that using libraries is infinitely preferable to<br>
> shelling out to execute cmdline tools, see:<br>
><br>
> <a href="http://www.linuxplumbersconf.org/2012/wp-content/uploads/2012/09/Grover-APIs.odp" target="_blank">http://www.linuxplumbersconf.org/2012/wp-content/uploads/2012/09/Grover-APIs.odp</a><br>
><br>
> I certainly understand needing to do so for the many existing tools that<br>
> don't have libraries, but to force this model when proper APIs are<br>
> available (in Python, even) seems less than ideal.<br>
<br>
</div>An implementation of:<br>
<br>
class LioAdm(TargetAdmin):<br>
<br>
using rtslib seems fairly trivial.<br>
<br>
I would suggest going that route to start. We can also have a targetd version but<br>
that is definitely a bigger task.<br>
<br>
Vish<br>
<div><div><br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org" target="_blank">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>Talking to Eric H and folks in the Cinder meeting this morning I think we all agreed on this, so long as there's no objections here from the broader ML?  The only question that remains is dealing with the root-access requirement by Cinder for rtslib.  Not sure about changes to the lib to fix this, or the possibility of a wrapper etc. or maybe it's not a major concern?