On Thu, Apr 30, 2015 at 08:46:50AM -0400, John Ferlan wrote:
On 04/30/2015 07:06 AM, Ján Tomko wrote:
> Just as we allow stopping filesystem pools when they were unmounted
> externally, do not fail to stop an iscsi pool when someone else
> closed the session externally.
>
> Reported at:
>
https://bugzilla.redhat.com/show_bug.cgi?id=1171984
> ---
> v2: Do not report errors when returning success (use probe=true)
> refrain from making controversial claims in the commit message
>
> src/storage/storage_backend_iscsi.c | 5 +++++
> 1 file changed, 5 insertions(+)
>
So if someone does :
iscsiadm -m session
to find the session(s), $iqn and $ipaddr value and they perform a :
iscsiadm -m node -T $iqn -p $ipaddr --logout
In order to logout out of the session, this patch will ensure that the
pool-destroy path doesn't fail. For that I agree. I still disagree with
the bz being listed, but that's your choice.
Well it is where I saw the bug reported (although it was libvirt doing
the logout there, not the user).
ACK
Thanks, pushed now.
Jan