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

Bug #833632 reported by Spring Zhang
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

Revision history for this message
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
Revision history for this message
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)
Changed in lava-test:
assignee: Paul Larson (pwlars) → Le Chi Thu (le-chi-thu)
status: Confirmed → Fix Committed
Revision history for this message
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)
Changed in lava-android-test:
status: New → Confirmed
Revision history for this message
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)
Changed in lava-android-test:
milestone: 2011.10 → none
Fathi Boudra (fboudra)
Changed in lava-test:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.