[libvirt] [PATCH v2 07/10] viriscsitest: Test virISCSIConnectionLogin

Michal Privoznik mprivozn at redhat.com
Tue Jul 24 14:41:05 UTC 2018


On 07/24/2018 04:08 PM, John Ferlan wrote:
> 
>  <snip/>> Reading something like:
> 
>  /* mocking real environment output is not feasible for [creating |
> updating | logging into], example of real environment is:
> 
> xxx
> 
>  */
> 
> I believe is better - it's not difficult to add, but I'm at the point of
> not caring right now because this truly has gone on too long.
> 
> For the code/logic:
> 
> Reviewed-by: John Ferlan <jferlan at redhat.com>
> 
> Do whatever you want for the comments. I disagree that "nada" or "no
> output" is good enough, but it's not worth holding this up.

Thanks. I'll add the real output into comments as you're suggesting.

Michal




More information about the libvir-list mailing list