Re: [PATCH v1 12/12] hotplug/Linux: add iscsi block hotplug script

["Roger Pau Monné" <roger.pau@citrix.com>]

On 13/03/13 17:31, Ian Jackson wrote:
> Roger Pau Monne writes ("[Xen-devel] [PATCH v1 12/12] hotplug/Linux: add iscsi block hotplug script"):
>> This hotplug script has been tested with IET and NetBSD iSCSI targets,
>> without authentication.
>>
>> Authentication parameters, including the password are passed as
>> parameters to iscsiadm, which is not recommended because other users
>> of the system can see them. This parameters could also be set by
>> editing a corresponding file directly, but the location of this file
>> seems to be different depending on the distribution used.
> 
> Would it be possible for the user to specify the file that the
> password should be read from ?

AFAIK no, this is read from a system defined file, that can be different
depending on the distro used. Maybe be could expand this script latter
if needed, in order to be able to detect where this file resides and use
that instead of passing the user/password in the command line.

> 
> Did you write this script from scratch ?  (TBH I don't intend to
> review it in detail...)

I've found a previous version of iSCSI and Xen here:

http://backdrift.org/xen-block-iscsi-script-with-multipath-support

but I didn't use it, so yes.

> Why no set -e ?

Because some of the executed commands might fail, but that's expected
(and shouldn't abort the execution) and we manually check for the return
value of executed commands.

> Ian.
>