mirror of
https://gitlab.freedesktop.org/mesa/mesa.git
synced 2026-05-08 17:48:10 +02:00
ci/lava: Propagate errors in SSH tests
The `lava_ssh_test_case` wrapper was missing the `set -e` shell option,
which made LAVA system interpret the job was succeeding, because the
`container` namespace was exiting normally, even though the `dut`
namespace was failing.
Signed-off-by: Guilherme Gallo <guilherme.gallo@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/33839>
(cherry picked from commit 1169f704d3)
This commit is contained in:
parent
9347962591
commit
989f2d8b34
2 changed files with 2 additions and 2 deletions
|
|
@ -68,7 +68,7 @@ EOF
|
|||
ping -c 5 -w 60 $(lava-target-ip)
|
||||
|
||||
lava_ssh_test_case() {
|
||||
set -x
|
||||
set -ex
|
||||
local test_case="${1}"
|
||||
shift
|
||||
lava-test-case \"${test_case}\" --shell \\
|
||||
|
|
|
|||
|
|
@ -644,7 +644,7 @@
|
|||
"description": "ci/lava: Propagate errors in SSH tests",
|
||||
"nominated": false,
|
||||
"nomination_type": 0,
|
||||
"resolution": 4,
|
||||
"resolution": 1,
|
||||
"main_sha": null,
|
||||
"because_sha": null,
|
||||
"notes": null
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue