On a Wednesday in 2020, Andrea Bolognani wrote:
On Tue, 2020-04-21 at 18:25 +0100, Daniel P. Berrangé wrote:
> On Tue, Apr 21, 2020 at 07:15:00PM +0200, Andrea Bolognani wrote:
> > When the job monitoring logic was refactored, these two commands
> > were not converted properly and the result is that a successful
> > dump or migration (char '0') would be reported as a failed one
> > (int 48) instead.
> >
> > Fixes: dc0771cfa2e78ffecd7c8234538ee548748d7bef
> > Reported-by: Brian Rak <brak(a)gameservers.com>
> > Signed-off-by: Andrea Bolognani <abologna(a)redhat.com>
> > ---
> > tools/virsh-domain.c | 10 +++-------
> > 1 file changed, 3 insertions(+), 7 deletions(-)
>
> Reviewed-by: Daniel P. Berrangé <berrange(a)redhat.com>
Thanks, pushed now. And thanks again to Brian for reporting the issue
and tracking it down!
Do you think we should backport this to the 6.1 and 6.2 mainteinance
branches? It fixes a fairly bad regression... But I'm not very
familiar with how we handle those branches.
We don't because nobody was using them anymore.
Jano
--
Andrea Bolognani / Red Hat / Virtualization