Skip to content

Fix AudioRecord testcase ID collision for LAVA CI#383

Merged
smuppand merged 1 commit intoqualcomm-linux:mainfrom
tmoida:main
Apr 2, 2026
Merged

Fix AudioRecord testcase ID collision for LAVA CI#383
smuppand merged 1 commit intoqualcomm-linux:mainfrom
tmoida:main

Conversation

@tmoida
Copy link
Copy Markdown
Contributor

@tmoida tmoida commented Apr 2, 2026

Description

This PR fixes the testcase ID collision issue in LAVA by implementing flexible testcase naming for the AudioRecord test suite, enabling parallel execution of multiple AudioRecord configurations in CI.

Problem

LAVA was experiencing testcase ID collisions because:

  • The AudioRecord script was hardcoding the testcase name as AudioRecord in the .res file
  • When running multiple AudioRecord configurations in parallel (e.g., Config01, Config03, Config05), all tests reported the same testcase ID
  • LAVA expected unique testcase IDs like AudioRecord_Config01, AudioRecord_Config03, etc. based on the job definition
  • This collision caused LAVA to overwrite test results and prevented proper CI enablement
  • Blocked issue: AudioRecord: Issue in reporting PASS/FAIL result, resulting in failure #378

Solution

Implemented a flexible testcase naming mechanism that allows LAVA to specify unique testcase IDs:

  • Added RESULT_TESTNAME variable - Defaults to AudioRecord but can be overridden via CLI parameter
  • Added --lava-testcase-id parameter - Allows LAVA to specify unique testcase names for each configuration
  • Added LAVA_TESTCASE_ID YAML parameter - Passes the testcase ID from LAVA job definition to the script
  • Updated all result handling - Changed all 22 .res file writes to use $RESULT_TESTNAME instead of $TESTNAME
  • Pre-parse support - Early parsing of --lava-testcase-id ensures unique result files even if setup fails

- Add RESULT_TESTNAME variable to support unique testcase IDs
- Add --lava-testcase-id CLI parameter
- Add LAVA_TESTCASE_ID parameter to AudioRecord.yaml
- Update Read_me.md with LAVA_TESTCASE_ID documentation

Resolves issue qualcomm-linux#378 by ensuring unique testcase IDs
in LAVA when running multiple AudioRecord configurations in parallel.

Signed-off-by: Teja Swaroop Moida <tmoida@qti.qualcomm.com>
@tmoida
Copy link
Copy Markdown
Contributor Author

tmoida commented Apr 2, 2026

Here are the details of the validation job:
https://lava.infra.foundries.io/scheduler/job/169530

Copy link
Copy Markdown
Contributor

@bhargav0610 bhargav0610 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
is this validated on all Targets ?

Copy link
Copy Markdown
Contributor

@smuppand smuppand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@smuppand smuppand merged commit 8d08206 into qualcomm-linux:main Apr 2, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants