Comment 20 for bug 1732199

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to os-brick (stable/rocky)

Reviewed: https://review.openstack.org/643039
Committed: https://git.openstack.org/cgit/openstack/os-brick/commit/?id=80ee8f846c85a48f12828fab0de5c4c866349283
Submitter: Zuul
Branch: stable/rocky

commit 80ee8f846c85a48f12828fab0de5c4c866349283
Author: imacdonn <email address hidden>
Date: Tue Sep 25 03:40:31 2018 +0000

    'iscsiadm -m session' failure handling

    Sometimes 'iscsiadm -m session' outputs warnings on stderr, but
    still serves its purpose (usable stdout). We should not give up on
    it when stderr is non-empty - instead, rely on the exit status, log
    the stderr as a warning, and move on.

    This change also removes 1 (ISCSI_ERR) from the list of acceptable
    exit codes for the iscsiadm command, to ensure that failures get
    caught.

    Change-Id: Id8183cf3d8baf2f8ba6a00a47fa2ad7cc2a96aa5
    Closes-Bug: #1732199
    (cherry picked from commit d398fa82334bbabbc4c7d5b8f1fe5a7a4465200e)