[libvirt] [PATCH v2 3/4] util: Remove disabling of autologin for iscsi-targets

[Peter Krempa]

On Mon, May 16, 2016 at 11:03:10 -0400, John Ferlan wrote:
> From: Fritz Elfert <fritz at fritz-elfert.de>
> 
> https://bugzilla.redhat.com/show_bug.cgi?id=1331552
> 
> Instead of disabling auto-login of all scsi targets (even those
> that do not "belong" to libvirt), use iscsiadm's "--op nonpersistent"
> during discovery of iSCSI targets (e.g. "iscsiadm --mode discovery
> --type sendtargets") in order to avoid the node database being altered
> which led to the need for the "large hammer" approach taken by
> commit id '3c12b654'.

commit 3c12b6542cfc994f6b9cf3aef623767bf441c3ea
Author: Daniel P. Berrange <berrange at redhat.com>
Date:   Fri Nov 12 15:21:48 2010 +0000

> This commit removes the virISCSITargetAutologin adjustment (eg. the setting
> of node.startup to "manual"). The iscsiadm command has supported this mode
> of operation as of commit id 'ad873767' to open-iscsi.

commit ad873767436f1cc242f0d4a522a2fce7133795c1
Author: Mike Christie <michaelc at cs.wisc.edu>
Date:   Mon Mar 22 17:28:41 2010 -0500

    iscsiadm: add nonpersistent mode to discovery mode


Looks like this is already around for a while so I'm okay with using the
new approach now.

> Signed-off-by: John Ferlan <jferlan at redhat.com>
> ---
>  src/util/viriscsi.c  | 30 ++----------------------------
>  tests/viriscsitest.c |  4 +++-
>  2 files changed, 5 insertions(+), 29 deletions(-)

ACK
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20160518/00c4e8c2/attachment-0001.sig>