Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Twister does not report qemu errors in all cases #72995

Closed
1 task done
kuikka opened this issue May 19, 2024 · 2 comments
Closed
1 task done

Twister does not report qemu errors in all cases #72995

kuikka opened this issue May 19, 2024 · 2 comments
Assignees
Labels
area: Twister Twister bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug

Comments

@kuikka
Copy link

kuikka commented May 19, 2024

Describe the bug
When running tests using twister and qemu fails to start or crashes, it is reported as "Unknown" reason instead of return value from qemu.

Example:
In this case my system does not have qemu installed for cortex_a9 so this test will fail to run, but the produced error is not helpful for debugging the reason.

Commandline: ./scripts/twister -T tests/subsys/shell/shell_history -p qemu_cortex_a9

Pertinent output:

qemu_cortex_a9            tests/subsys/shell/shell_history/shell.history      FAILED : Unknown
see: /Users/kuikka/zephyrproject/zephyr/twister-out/qemu_cortex_a9/tests/subsys/shell/shell_history/shell.history/build.log
INFO    - Total complete:    1/   1  100%  skipped:    0, failed:    1, error:    0

To Reproduce
Steps to reproduce the behavior:
Version: 34c84ec

  1. Assume qemu for arm cortex a9 is not available
  2. ./scripts/twister -T tests/subsys/shell/shell_history -p qemu_cortex_a9

Expected behavior

qemu_cortex_a9            tests/subsys/shell/shell_history/shell.history      FAILED : Exited with 1

Impact
Annoying and makes debugging tests in CI difficult due to not all information being captured in logs.

Tasks

  1. Trivial area: Twister
    nashif
@kuikka kuikka added the bug The issue is a bug, or the PR is fixing a bug label May 19, 2024
@kuikka
Copy link
Author

kuikka commented May 19, 2024

#72996 Fixes this issue for me but further thoughts may be needed as I'm not sure this works in all cases.

@nashif nashif added the priority: low Low impact/importance bug label May 21, 2024
@kuikka
Copy link
Author

kuikka commented May 22, 2024

My commit was accepted so closing this.

@kuikka kuikka closed this as completed May 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Twister Twister bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug
Projects
None yet
Development

No branches or pull requests

3 participants