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

multiple pipelines: skip XTOS + NOCODEC configurations #1229

Merged
merged 2 commits into from
Aug 15, 2024

Conversation

marc-hb
Copy link
Collaborator

@marc-hb marc-hb commented Aug 15, 2024

2 commits

Stop polluting test results daily.

pipeline_comp_reset() bug sof#9135 is not going to get fixed, so stop
polluting the test results.

Signed-off-by: Marc Herbert <[email protected]>
pipeline_comp_reset() bug sof#9135 is not going to get fixed, so stop
polluting the test results.

Signed-off-by: Marc Herbert <[email protected]>
@marc-hb
Copy link
Collaborator Author

marc-hb commented Aug 15, 2024

Test results all as expected.

Copy link
Collaborator

@fredoh9 fredoh9 left a comment

Choose a reason for hiding this comment

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

I don't like the idea of skipping test case(s) in the test case itself. And I'm afraid we might forget the known issue(s) and give impression of no known issue.

But we don't fix this issue for stable-v2.2. Without this known issue, it gives a lot easier tracking the test results.
So approved.

@fredoh9 fredoh9 merged commit 79a9634 into thesofproject:main Aug 15, 2024
5 of 7 checks passed
@marc-hb marc-hb deleted the skip-multi-nocodec branch August 15, 2024 23:17
@marc-hb
Copy link
Collaborator Author

marc-hb commented Aug 15, 2024

Thanks! I don't like it either but this one will never get fixed.

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.

2 participants