Nova volume should support tgt rather than iscsitarget.

[Chuck Short]

Nova volume should use tgt rather than iscsitarget. tgt is the iscsi software supproted by Ubuntu.

[Mark McLoughlin (markmc)]

I've done some more work on Chuck's patch and it should be ready to merge now. See the lp:~markmc/nova/iscsi-tgtadm-choice branch

[Openstack Gerrit (openstack-gerrit)]

Reviewed: https://review.openstack.org/555
Committed: http://github.com/openstack/nova/commit/871141d4d3cc0ac739de72ca010aef3e5c13fe1f
Submitter: Jenkins
Branch: master

 status fixcommitted
 done

commit 871141d4d3cc0ac739de72ca010aef3e5c13fe1f
Author: Mark McLoughlin <email address hidden>
Date: Thu Aug 11 07:44:38 2011 -0400

    Allow the user to choose either ietadm or tgtadm (lp:819997)

    Also, refactor ietadm/tgtadm calls out into helper classes. Add a new
    TargetAdmin abstract base class and implement it using ietadm and
    tgtadm. This cleans up the code greatly and gets us some code reuse.

    (Based on a patch by Chuck Short <email address hidden>)

    Change-Id: I1c0064e5d35483a6c4059cfc61a484f5f576b2da

[Openstack Gerrit (openstack-gerrit)]

Reviewed: https://review.openstack.org/554
Committed: http://github.com/openstack/nova/commit/2522a4438b1bc6799845f733894bc1d64b2a755f
Submitter: Jenkins
Branch: master

 status fixcommitted
 done

commit 2522a4438b1bc6799845f733894bc1d64b2a755f
Author: Mark McLoughlin <email address hidden>
Date: Sun Sep 18 12:04:46 2011 +0100

    Remove VolumeDriver.sync_exec method (lp:819997)

    We always use the same functions for sync_exec and execute.

    The execute method is always synchronous, so the distinction doesn't
    appear to make sense.

    Finally, it looks like it would make sense for execute to ever be
    async, so the distinction isn't even serving a useful documentation
    purpose.

    Change-Id: I86d491cfbf8be73672df7cfdf22e465627a86034