example/lava.job: boot: cmsis-dap: Add "failure_retry: 3".

Linaro / lite-lava-docker-compose

LITE Team LAVA docker dispatcher

MIT License

5 stars 10 forks source link

example/lava.job: boot: cmsis-dap: Add "failure_retry: 3". #138

Closed pfalcon closed 3 years ago

pfalcon commented 3 years ago

This change depends on the cmsis-dap boot action to detect failure (flashing failure, to be specific). Currently LAVA doesn't do, but the patch was submitted to do that: https://git.lavasoftware.org/lava/lava/-/merge_requests/1416

So, proactively add this setting, so it wasn't forgotten later.

Signed-off-by: Paul Sokolovsky paul.sokolovsky@linaro.org

galak commented 3 years ago

Is there any implication and submitting this job to a LAVA that doesn't support the feature?

pfalcon commented 3 years ago

No, it's a standard LAVA feature, and completely safe to add. The only point is that currently failures in "cmsis-dap" boot action aren't detected, so this "failure_retry" LAVA functionality won't be triggered (until https://git.lavasoftware.org/lava/lava/-/merge_requests/1416 lands).