make Audio_Record_Playback more deterministic for offline CI#411
Open
smuppand wants to merge 1 commit intoqualcomm-linux:mainfrom
Open
make Audio_Record_Playback more deterministic for offline CI#411smuppand wants to merge 1 commit intoqualcomm-linux:mainfrom
smuppand wants to merge 1 commit intoqualcomm-linux:mainfrom
Conversation
Contributor
Author
|
@nitinn22 to review |
nitinn22
reviewed
Apr 19, 2026
nitinn22
reviewed
Apr 19, 2026
nitinn22
reviewed
Apr 19, 2026
nitinn22
reviewed
Apr 19, 2026
nitinn22
reviewed
Apr 19, 2026
nitinn22
reviewed
Apr 19, 2026
nitinn22
reviewed
Apr 19, 2026
nitinn22
reviewed
Apr 19, 2026
nitinn22
reviewed
Apr 19, 2026
Contributor
|
Main suggestions are around avoiding silent copy failures and improving portability/robustness of the generation pipelines (audioresample, encoder property compatibility). Also, if a LAVA job can be posted with these encode followed by decode and full suite working fine would be perfect. |
… overrides Update audio clip provisioning to prefer locally generated GStreamer sample clips for OGG and MP3 when no external clip source is explicitly requested, making CI runs less dependent on network availability. This keeps backward compatibility by preserving existing --clip-path, --clip-url, AUDIO_CLIP_PATH, and AUDIO_CLIP_URL behavior. The run.sh changes now track whether an external source was explicitly requested and pass that state into provision_test_files(), while the provisioning logic can still fall back to user-provided clips or downloads when requested. Signed-off-by: Srikanth Muppandam <smuppand@qti.qualcomm.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Improve Audio_Record_Playback stability and result reporting for CI runs. Enhancements to address as part of #408
This update keeps backward compatibility for user-provided clip sources, but makes the default path more reliable by preferring locally generated OGG/MP3 sample clips when no clip path or clip URL is explicitly requested.
It also tightens result handling for dependent pulsesrc playback cases. When running in all mode, playback_pulsesrc_wav and playback_pulsesrc_flac now fail if the corresponding pulsesrc recording
in the same run did not produce a valid file, instead of being marked as skip. This makes the final summary reflect the actual end-to-end failure more accurately.
Changes included:
prerequisite recording failed in the same run
Why:
Lava jobs for reference with these code changes.
Base/ Config 1 - https://lava.infra.foundries.io/scheduler/job/183417