On Wed, May 18, 2016 at 07:15:54 -0400, John Ferlan wrote:
> On 05/18/2016 01:55 AM, Peter Krempa wrote:

[...]

> > "cannot find session iscsiadm session" followed by the message and/or
> > containing the code will be probably the best case here. The function
> > you are modifying is meant to find the session so any error is basically
> > that it couldn't find it. The message from iscsiadm then helps
> > debugging.
> > 
> 
> Fair enough... Primary thought was to be consistent with previous error,
> but generating/using a unique error is fine... So rather than the if
> then else based on exit status, how about:
> 
>        virReportError(VIR_ERR_INTERNAL_ERROR,
>                        _("cannot find iscsiadm session: %s"),
>                        NULLSTR(error));

That is reasonable. ACK to that including 1/4 if I didn't ACK it before.

Attachment: signature.asc
Description: Digital signature

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to