Re: [PATCH 1/6] docs: Remove unnecessary <auth> example for iscsi disk type='volume'

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Sep 14, 2017 at 14:03:05 -0400, John Ferlan wrote:
> Alter the example to remove the <auth> from:
> 
>   <disk type='volume' device='disk'>
>     <driver name='qemu' type='raw'/>
>     <source pool='iscsi-pool' volume='unit:0:0:1' mode='host'/>
>     <auth username='myuser'>
>       <secret type='iscsi' usage='libvirtiscsi'/>
>     </auth>
>     <target dev='vdb' bus='virtio'/>
>   </disk>
> 
> and
> 
>   <disk type='volume' device='disk'>
>     <driver name='qemu' type='raw'/>
>     <source pool='iscsi-pool' volume='unit:0:0:2' mode='direct'/>
>     <auth username='myuser'>
>       <secret type='iscsi' usage='libvirtiscsi'/>
>     </auth>
>     <target dev='vdc' bus='virtio'/>
>   </disk>
> 
> The reality is, it's not even used. For a <source pool> the authdef
> from the storage source pool will supercede whatever is in the <disk>
> definition during virStorageTranslateDiskSourcePool processing. In fact,
> if the pool doesn't have/need authentication, then the authdef would
> be removed anyway as the storage pool would be handling things.
> 
> The "proof" for this is in the adjustment to the test to add an
> <auth> for a disk. The resulting .args file won't add what normally
> would be added "myname:encodedpassword@" prior to the hostname in
> the IQN (e.g. iscsi://myname:encodedpassword@xxxxxxxxxxxxxxxxx:3260/...
> 
> Signed-off-by: John Ferlan <jferlan@xxxxxxxxxx>
> ---
>  docs/formatdomain.html.in                                   |  6 ------
>  .../qemuxml2argv-disk-source-pool-mode.args                 |  3 +++
>  .../qemuxml2argvdata/qemuxml2argv-disk-source-pool-mode.xml | 13 +++++++++++++
>  .../qemuxml2xmlout-disk-source-pool-mode.xml                | 13 +++++++++++++
>  4 files changed, 29 insertions(+), 6 deletions(-)

ACK

Attachment: signature.asc
Description: PGP signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux