diff options
author | Lee Duncan <lduncan@suse.com> | 2019-01-23 11:19:06 -0800 |
---|---|---|
committer | Lee Duncan <lduncan@suse.com> | 2019-01-23 11:19:06 -0800 |
commit | 59806c99fb3f86277a0d596c6f4c9a58446edc91 (patch) | |
tree | 1f704f283c066567c45d1602dc13a37af1034a7e /etc | |
parent | 364d757606271ff34e08e11a223b252cf40db7a8 (diff) | |
download | open-iscsi-59806c99fb3f86277a0d596c6f4c9a58446edc91.tar.gz |
Fix output for iscsiadm node/iface print level P1
The output from 'iscsiadm -m node -P1' and 'iscsiadm -m iface -P1'
has regressed with the introduction of libopeniscsiusr.
The pre-lib output for 'iscsiadm -m node -P1' before was:
>Target: <TARGET-IQN>
> Portal: <IP-ADDRESS>:<PORT>,1
> Iface Name: default
repeated for each Node in the node datbase, and the output
of 'iscsiadm -m iface -P1' before the change was:
>Iface: default
> Target: <TARGET-IQN>
> Portal: <IP-ADDRESS>:<PORT>,1
repeated once for each Node in the database that uses the
'default' interface (i.e. usually all nodes).
In both of these cases, the output contained details for each
and every node in the database. But this evidently wasn't the
intention if one looks at the code. The code appears to try
to only print the details for each 'Target' if and only if
those details are different than the previous line. In other
words, the code looks like it wants to suppress the 'Portal'
and 'Iface Name' lines if they are the same as the previously-
listed target. This explains why libopeniscsiusr implemented
the output of iscsiadm to suppress this info on all but the
first target, assuming the Portal and IfaceName were the
same.
The fix is to simplify print_nodes_tree() to print details
for every Node, since that is how the output worked
before the libopeniscsiusr update.
Diffstat (limited to 'etc')
0 files changed, 0 insertions, 0 deletions