[libvirt] [PATCHv2] iscsi: do not fail to stop a stopped pool

[Ján Tomko]

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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20150430/dbc6d9a0/attachment-0001.sig>