ceph / ceph-iscsi-config

NOTICE: moved to https://github.com/ceph/ceph-iscsi
GNU General Public License v3.0
22 stars 28 forks source link

Fix session check during lun deactivate #96

Closed mikechristie closed 5 years ago

mikechristie commented 5 years ago

In the ceph-iscsi-cli commit:

commit 9cec0ddbccaeffbaa6212360dd64019172a839d6 Author: Jason Dillaman dillaman@redhat.com Date: Fri Oct 26 09:08:30 2018 -0400

Fixed flake8 warnings

there was a mismerge due to a conflict and it added back the old attached lun check bypassing the mapped lun check which was needed to make sure we only test if a lun is mapped to a client before trying to deactivate. The ceph-isccsi-config lun deactivate call was then based on that ceph-iscsi-cli code.

This patch reverts the attached lun only check.