lava-test install does not fail if the test fails to install

Bug #833632 reported by Spring Zhang on 2011-08-25
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
LAVA Android Test
Invalid
Medium
Yongqin Liu
Obsolete LAVA Test
Fix Released
Medium
Le Chi Thu

Bug Description

See log:
root@linaro:~# lava-test run stream
/bin/sh: ./stream: not found
ABREK TEST RUN COMPLETE: Result id is 'stream921.0'
root@linaro:~# echo $?
0

Paul Larson (pwlars) wrote :

This could be fixed in the upcoming merge proposal for lava-test to integrate it with lava-tool

Changed in lava-test:
milestone: none → 2011.09
summary: - if stream is not installed, lava-test running stream also get a zero
- return value
+ lava-test install does not fail if the test fails to install
Changed in lava-test:
importance: Undecided → Medium
status: New → Confirmed
Paul Larson (pwlars) wrote :

So the new lava-test branch seems to report when a command exits with a non-zero status, but it doesn't return non-zero for the whole thing. This is not yet fixed, but should be easy to fix for 11.10

Changed in lava-test:
milestone: 2011.09 → 2011.10
assignee: nobody → Paul Larson (pwlars)
Le Chi Thu (le-chi-thu) on 2011-09-27
Changed in lava-test:
assignee: Paul Larson (pwlars) → Le Chi Thu (le-chi-thu)
status: Confirmed → Fix Committed
Paul Larson (pwlars) wrote :

We should also make sure lava-android-test behaves this way.

Changed in lava-android-test:
assignee: nobody → Yongqin Liu (liuyq0307)
importance: Undecided → Medium
milestone: none → 2011.10
Fathi Boudra (fboudra) on 2011-10-06
Changed in lava-android-test:
status: New → Confirmed
Yongqin Liu (liuyq0307) wrote :

lava-android-test will fail when failed to install.
There is no need to fix

Changed in lava-android-test:
status: Confirmed → Won't Fix
status: Won't Fix → Invalid
Fathi Boudra (fboudra) on 2011-10-20
Changed in lava-android-test:
milestone: 2011.10 → none
Fathi Boudra (fboudra) on 2011-10-24
Changed in lava-test:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers