Comment 1 for bug 1819577

Revision history for this message
Sahid Orentino (sahid-ferdjaoui) wrote :

It would have been great to share a bit of more of details related to the issue. My thinking is that your patch can't work in case of multipath usage since the devices are not registered anymore in sysfs. But it's just one potential issue, it may have others.

As you have noticed, reverting this commit 14be08d0b53836963a530e2990a6db73901cbdac may be a good start but if we can have share details we could also try to fix the change.

commit 14be08d0b53836963a530e2990a6db73901cbdac
Author: Avishay Traeger <email address hidden>
Date: Tue Jan 29 10:10:50 2019 +0200

    Verify WWN of connected iSCSI devices if passed

    If a WWN has been provided in connection_properties, verify that the
    found devices have that expected WWN. Fail the attachment if not. This
    prevents cases where a device already exists on the host from an old
    attachment that wasn't fully cleaned up, and the new attachment got
    the same LUN. Using this old device could lead to data corruption.

    Change-Id: I45a2221c0518213dc8132831c0bde9db4734da2b
    Closes-Bug: 1813609

 os_brick/exception.py | 5 +++++
 os_brick/initiator/connectors/iscsi.py | 3 +++
 os_brick/initiator/linuxscsi.py | 10 ++++++++++
 os_brick/tests/initiator/test_linuxscsi.py | 12 ++++++++++++
 releasenotes/notes/verify-iscsi-wwns-f6cb536a7fbd4b42.yaml | 9 +++++++++