[PATCH pve-stoage 3/4] iscsiPlugin: add wwid

[PATCH pve-stoage 3/4] iscsiPlugin: add wwid

[Felix Drießler]

From: Felix Driessler <fdriessler@inett.de>

Signed-off-by: Felix Driessler <fdriessler@inett.de>
---
 src/PVE/Storage/ISCSIPlugin.pm | 20 ++++++++++++++------
 1 file changed, 14 insertions(+), 6 deletions(-)

diff --git a/src/PVE/Storage/ISCSIPlugin.pm b/src/PVE/Storage/ISCSIPlugin.pm
index 30f4178..bb5576b 100644
--- a/src/PVE/Storage/ISCSIPlugin.pm
+++ b/src/PVE/Storage/ISCSIPlugin.pm
@@ -305,13 +305,21 @@ sub iscsi_device_list {
 
                     my $volid = "$channel.$id.$lun.$blockdev";
 
+                    my $wwid = '';
+                    my $dev = "/dev/$bdev";
+                    $dev =~ m|^(/dev/[A-Za-z0-9_\-./]+)$|
+                        or die "Invalid device path";
+                    $dev = $1;
+                    run_command(['/lib/udev/scsi_id', '-g', '-u', '-d', $dev], outfunc => sub { $wwid .= "$_[0]" });
+
                     $res->{$target}->{$volid} = {
                         'format' => 'raw',
                         'size' => int($size * 512),
                         'vmid' => 0, # not assigned to any vm
                         'channel' => int($channel),
                         'id' => int($id),
                         'lun' => int($lun),
+                        'wwid' => $wwid,
                     };
 
                     #print "TEST: $target $session $host,$bus,$tg,$lun $blockdev\n";
-- 
2.52.0



Felix Driessler
Junior Systems Engineer // #GernPerDU

Linux - Open Source - IT Solutions

T: +49-681-410993-0
E: fdriessler@inett.de
https://www.inett.de



inett GmbH | Altenkesseler Strasse 17 / B8 | 66115 Saarbrücken | Germany

Vertrieb: vertrieb@inett.de // +49-681-410993-33
Support: support@inett.de // +49-681-410993-42





Geschäftsführung: Marco Gabriel | Amtsgericht Saarbrücken HRB 16588

[PATCH pve-manager/pve-storage 0/4] extend iscsi configuration

[Felix Driessler]

This patch series adds a WWID field and CHAP Authentication boxes for iScsi.

pve-manager:

The WWID field is added to:
 - storage view
 - file selector

The WWID field is made to be only visible if a iScsi Storage is being viewed.

For CHAP authentication a checkbox as well as username/password field are added, 
to allow for togglin CHAP authentication on/off.

pve-storage:

Adds wwid return value to iscsi_device_list to make volumes of a iScsi Storage
easier to identify

Adds required functions to allow the configuration of CHAP through the proxmox web ui.

Felix Driessler (4):
  ui: Add Display of iScsi WWID
  ui: Add iScsi Chap Auth
  iscsiPlugin: add WWID
  iscsiPlugin: add chap auth

pve-manager:
 www/manager6/Utils.js               |  8 +++
 www/manager6/form/FileSelector.js   | 81 +++++++++++++++++++++--------
 www/manager6/storage/ContentView.js | 24 ++++++++-
 www/manager6/storage/IScsiEdit.js   | 43 +++++++++++++++
 4 files changed, 132 insertions(+), 24 deletions(-)
pve-storage:
 src/PVE/Storage/ISCSIPlugin.pm      | 142 ++++++++++++++++++++++++++++++---
 1 file changed, 133 insertions(+), 9 deletions(-)


-- 
2.52.0



Felix Driessler
Junior Systems Engineer // #GernPerDU

Linux - Open Source - IT Solutions

T: +49-681-410993-0
E: fdriessler@inett.de
https://www.inett.de



inett GmbH | Altenkesseler Strasse 17 / B8 | 66115 Saarbrücken | Germany

Vertrieb: vertrieb@inett.de // +49-681-410993-33
Support: support@inett.de // +49-681-410993-42





Geschäftsführung: Marco Gabriel | Amtsgericht Saarbrücken HRB 16588

[PATCH pve-stoage 3/4] iscsiPlugin: add wwid

[Felix Driessler]

Signed-off-by: Felix Driessler <fdriessler@inett.de>
---
 src/PVE/Storage/ISCSIPlugin.pm | 20 ++++++++++++++------
 1 file changed, 14 insertions(+), 6 deletions(-)

diff --git a/src/PVE/Storage/ISCSIPlugin.pm b/src/PVE/Storage/ISCSIPlugin.pm
index 30f4178..bb5576b 100644
--- a/src/PVE/Storage/ISCSIPlugin.pm
+++ b/src/PVE/Storage/ISCSIPlugin.pm
@@ -305,13 +305,21 @@ sub iscsi_device_list {
 
                     my $volid = "$channel.$id.$lun.$blockdev";
 
+                    my $wwid = '';
+                    my $dev = "/dev/$bdev";
+                    $dev =~ m|^(/dev/[A-Za-z0-9_\-./]+)$|
+                        or die "Invalid device path";
+                    $dev = $1;
+                    run_command(['/lib/udev/scsi_id', '-g', '-u', '-d', $dev], outfunc => sub { $wwid .= "$_[0]" });
+
                     $res->{$target}->{$volid} = {
-                        'format' => 'raw',
-                        'size' => int($size * 512),
-                        'vmid' => 0, # not assigned to any vm
-                        'channel' => int($channel),
-                        'id' => int($id),
-                        'lun' => int($lun),
+                        'format'    => 'raw',
+                        'size'      => int($size * 512),
+                        'vmid'      => 0, # not assigned to any vm
+                        'channel'   => int($channel),
+                        'id'        => int($id),
+                        'lun'       => int($lun),
+                        'wwid'      => $wwid
                     };
 
                     #print "TEST: $target $session $host,$bus,$tg,$lun $blockdev\n";
-- 
2.52.0



Felix Driessler
Junior Systems Engineer // #GernPerDU

Linux - Open Source - IT Solutions

T: +49-681-410993-0
E: fdriessler@inett.de
https://www.inett.de



inett GmbH | Altenkesseler Strasse 17 / B8 | 66115 Saarbrücken | Germany

Vertrieb: vertrieb@inett.de // +49-681-410993-33
Support: support@inett.de // +49-681-410993-42





Geschäftsführung: Marco Gabriel | Amtsgericht Saarbrücken HRB 16588

Re: [PATCH pve-stoage 3/4] iscsiPlugin: add wwid

[Daniel Kral]

Thanks for the patch!

Some notes inline

On Fri Mar 27, 2026 at 11:14 AM CET, Felix Driessler wrote:
> Signed-off-by: Felix Driessler <fdriessler@inett.de>
> ---
>  src/PVE/Storage/ISCSIPlugin.pm | 20 ++++++++++++++------
>  1 file changed, 14 insertions(+), 6 deletions(-)
>
> diff --git a/src/PVE/Storage/ISCSIPlugin.pm b/src/PVE/Storage/ISCSIPlugin.pm
> index 30f4178..bb5576b 100644
> --- a/src/PVE/Storage/ISCSIPlugin.pm
> +++ b/src/PVE/Storage/ISCSIPlugin.pm
> @@ -305,13 +305,21 @@ sub iscsi_device_list {
>  
>                      my $volid = "$channel.$id.$lun.$blockdev";
>  
> +                    my $wwid = '';
> +                    my $dev = "/dev/$bdev";
> +                    $dev =~ m|^(/dev/[A-Za-z0-9_\-./]+)$|
> +                        or die "Invalid device path";
> +                    $dev = $1;
> +                    run_command(['/lib/udev/scsi_id', '-g', '-u', '-d', $dev], outfunc => sub { $wwid .= "$_[0]" });

Only a suggestion and haven't tested this extensively yet: Since this
code block already enumerates the iSCSI devices through the relevant
sysfs, we can also fetch the wwid with:

    my $wwid = file_read_firstline("/sys/bus/scsi/devices/$tmp/wwid");

Though this does have a different output depending on the WWN type [0]
and this should probably also check whether the device actually has a
WWID (just a check whether the file 'wwid' exists).

[0] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/scsi/scsi_lib.c?h=v7.0#n3348

> +
>                      $res->{$target}->{$volid} = {
> -                        'format' => 'raw',
> -                        'size' => int($size * 512),
> -                        'vmid' => 0, # not assigned to any vm
> -                        'channel' => int($channel),
> -                        'id' => int($id),
> -                        'lun' => int($lun),
> +                        'format'    => 'raw',
> +                        'size'      => int($size * 512),
> +                        'vmid'      => 0, # not assigned to any vm
> +                        'channel'   => int($channel),
> +                        'id'        => int($id),
> +                        'lun'       => int($lun),

Aligning the indentation of the other existing properties should be done
in a separate patch before this.

Though our `make tidy` target will revert this back to the previous
formatting.

> +                        'wwid'      => $wwid

nit: add a , at the end of the new property

>                      };
>  
>                      #print "TEST: $target $session $host,$bus,$tg,$lun $blockdev\n";

Re: [PATCH pve-stoage 3/4] iscsiPlugin: add wwid

[Felix Driessler - inett GmbH]

> Aligning the indentation of the other existing properties should be done
> in a separate patch before this.

> Though our `make tidy` target will revert this back to the previous
> formatting.

> nit: add a , at the end of the new property

I've sent patch #3 again with these 2 fixed.
I missed reverting the automatic style changes from my IDE it seems.

> Only a suggestion and haven't tested this extensively yet: Since this
> code block already enumerates the iSCSI devices through the relevant
> sysfs, we can also fetch the wwid with:

I originally made use of this exact method but then swapped to using this CLI tool,
as, at least in my test environment, the WWID would be shown as 3600*, rather than naa.600*.
I would have opted for a regex replace, but wasn't sure if naa., t10. and eui.
are the only prefixes, or if there are more.
If the prefix, rather than its ID is prefered then I'd change the CLI call back to be what you suggested.

> Thanks for the patch!
>
> Some notes inline
>
> On Fri Mar 27, 2026 at 11:14 AM CET, Felix Driessler wrote:
>> Signed-off-by: Felix Driessler <fdriessler@inett.de>
>> ---
>>   src/PVE/Storage/ISCSIPlugin.pm | 20 ++++++++++++++------
>>   1 file changed, 14 insertions(+), 6 deletions(-)
>>
>> diff --git a/src/PVE/Storage/ISCSIPlugin.pm b/src/PVE/Storage/ISCSIPlugin.pm
>> index 30f4178..bb5576b 100644
>> --- a/src/PVE/Storage/ISCSIPlugin.pm
>> +++ b/src/PVE/Storage/ISCSIPlugin.pm
>> @@ -305,13 +305,21 @@ sub iscsi_device_list {
>>   
>>                       my $volid = "$channel.$id.$lun.$blockdev";
>>   
>> +                    my $wwid = '';
>> +                    my $dev = "/dev/$bdev";
>> +                    $dev =~ m|^(/dev/[A-Za-z0-9_\-./]+)$|
>> +                        or die "Invalid device path";
>> +                    $dev = $1;
>> +                    run_command(['/lib/udev/scsi_id', '-g', '-u', '-d', $dev], outfunc => sub { $wwid .= "$_[0]" });
> Only a suggestion and haven't tested this extensively yet: Since this
> code block already enumerates the iSCSI devices through the relevant
> sysfs, we can also fetch the wwid with:
>
>      my $wwid = file_read_firstline("/sys/bus/scsi/devices/$tmp/wwid");
>
> Though this does have a different output depending on the WWN type [0]
> and this should probably also check whether the device actually has a
> WWID (just a check whether the file 'wwid' exists).
>
> [0] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/scsi/scsi_lib.c?h=v7.0#n3348
>
>> +
>>                       $res->{$target}->{$volid} = {
>> -                        'format' => 'raw',
>> -                        'size' => int($size * 512),
>> -                        'vmid' => 0, # not assigned to any vm
>> -                        'channel' => int($channel),
>> -                        'id' => int($id),
>> -                        'lun' => int($lun),
>> +                        'format'    => 'raw',
>> +                        'size'      => int($size * 512),
>> +                        'vmid'      => 0, # not assigned to any vm
>> +                        'channel'   => int($channel),
>> +                        'id'        => int($id),
>> +                        'lun'       => int($lun),
> Aligning the indentation of the other existing properties should be done
> in a separate patch before this.
>
> Though our `make tidy` target will revert this back to the previous
> formatting.
>
>> +                        'wwid'      => $wwid
> nit: add a , at the end of the new property
>
>>                       };
>>   
>>                       #print "TEST: $target $session $host,$bus,$tg,$lun $blockdev\n";




Felix Driessler
Junior Systems Engineer // #GernPerDU

Linux - Open Source - IT Solutions

T: +49-681-410993-0
E: fdriessler@inett.de
https://www.inett.de



inett GmbH | Altenkesseler Strasse 17 / B8 | 66115 Saarbrücken | Germany

Vertrieb: vertrieb@inett.de // +49-681-410993-33
Support: support@inett.de // +49-681-410993-42





Geschäftsführung: Marco Gabriel | Amtsgericht Saarbrücken HRB 16588