On 07/17/2018 08:46 PM, John Ferlan wrote: > > > On 07/04/2018 05:23 AM, Michal Privoznik wrote: >> Introduce one basic test that tests the simplest case: >> logging into portal without any IQN. >> >> Signed-off-by: Michal Privoznik <mpriv...@redhat.com> >> --- >> tests/viriscsitest.c | 46 ++++++++++++++++++++++++++++++++++++++++++++++ >> 1 file changed, 46 insertions(+) >> > > Reviewed-by: John Ferlan <jfer...@redhat.com> > > John > > FWIW: This is successful for me w/o patch 6 >
Of course it is. The testIscsiadmCb() is not failing because of hunk 1. But try removing both patch 6 AND the first hunk (so that testIscsiadmCb() sets *status = -1;). I guess now you can see my reasoning for patch 6. Michal -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list