Message ID | 20191112151051.2205-1-alex.bennee@linaro.org |
---|---|
State | New |
Headers | show |
Series | [RFC] tests/vm: make --interactive (and therefor DEBUG=1) unconditional | expand |
On 11/12/19 9:10 AM, Alex Bennée wrote: In the subject: s/therefor/therefore/ > While the concepts of only dropping to ssh if a test fails is nice it s/concepts/concept/ > is more useful for this to be unconditional. You usually just want to > get the build up and running and then noodle around debugging or > attempting to replicate. > > Cc: Peter Maydell <peter.maydell@linaro.org> > Signed-off-by: Alex Bennée <alex.bennee@linaro.org> > --- > tests/vm/basevm.py | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tests/vm/basevm.py b/tests/vm/basevm.py > index 91a9226026d..0b8c1b26576 100755 > --- a/tests/vm/basevm.py > +++ b/tests/vm/basevm.py > @@ -403,7 +403,7 @@ def main(vmcls): > exitcode = 0 > if vm.ssh(*cmd) != 0: > exitcode = 3 > - if exitcode != 0 and args.interactive: > + if args.interactive: > vm.ssh() > > if not args.snapshot: >
diff --git a/tests/vm/basevm.py b/tests/vm/basevm.py index 91a9226026d..0b8c1b26576 100755 --- a/tests/vm/basevm.py +++ b/tests/vm/basevm.py @@ -403,7 +403,7 @@ def main(vmcls): exitcode = 0 if vm.ssh(*cmd) != 0: exitcode = 3 - if exitcode != 0 and args.interactive: + if args.interactive: vm.ssh() if not args.snapshot:
While the concepts of only dropping to ssh if a test fails is nice it is more useful for this to be unconditional. You usually just want to get the build up and running and then noodle around debugging or attempting to replicate. Cc: Peter Maydell <peter.maydell@linaro.org> Signed-off-by: Alex Bennée <alex.bennee@linaro.org> --- tests/vm/basevm.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)