Skip to content

multiple pipelines: skip XTOS + NOCODEC configurations - #1229

Merged
fredoh9 merged 2 commits into
thesofproject:mainfrom
marc-hb:skip-multi-nocodec
Aug 15, 2024
Merged

fredoh9 merged 2 commits into
thesofproject:mainfrom
marc-hb:skip-multi-nocodec

Conversation

@marc-hb

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

Copy link
Copy Markdown
Collaborator

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 <marc.herbert@intel.com>
pipeline_comp_reset() bug sof#9135 is not going to get fixed, so stop
polluting the test results.

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
@marc-hb

marc-hb commented Aug 15, 2024

Copy link
Copy Markdown
Collaborator Author

Test results all as expected.

@fredoh9 fredoh9 left a comment

Copy link
Copy Markdown
Contributor

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
@marc-hb
marc-hb deleted the skip-multi-nocodec branch August 15, 2024 23:17
@marc-hb

marc-hb commented Aug 15, 2024

Copy link
Copy Markdown
Collaborator Author

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