Skip to content

multiple-pipeline.sh: temporarily disable capture on BDW - #863

Merged
marc-hb merged 2 commits into
thesofproject:mainfrom
marc-hb:dis-cap-bdw
Feb 28, 2022
Merged

marc-hb merged 2 commits into
thesofproject:mainfrom
marc-hb:dis-cap-bdw

Conversation

@marc-hb

@marc-hb marc-hb commented Feb 19, 2022

Copy link
Copy Markdown
Collaborator

Known issue thesofproject/sof#3170 has been
polluting the test results for years.

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

multiple-pipeline.sh: add usual TESTDIR variable for re-use

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
Known issue thesofproject/sof#3170 has been
polluting the test results for years.

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
@marc-hb
marc-hb marked this pull request as ready for review February 19, 2022 04:46
@marc-hb
marc-hb requested a review from a team as a code owner February 19, 2022 04:46
@marc-hb

marc-hb commented Feb 19, 2022

Copy link
Copy Markdown
Collaborator Author

https://sof-ci.01.org/softestpr/PR863/build986/devicetest/?model=BDW_WSB_RT286&testcase=multiple-pipeline-capture is skipped as expected. The other failures are obviously unrelated.

@mengdonglin mengdonglin 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.

@marc-hb Can we also disable BYT and CHT? All they're legacy platforms of low priority.

@keqiaozhang

Copy link
Copy Markdown
Contributor

Issue 3170 mainly happens on BDW_WSB_RT286 when testing "multiple-pipeline-capture", but we also observed this issue on BYT_MB_NOCODEC when testing "simultaneous-playback-capture", but the reproduction rate is relativity lower than BDW.
Test ID:9950/9924 ,etc.

@marc-hb

marc-hb commented Feb 20, 2022

Copy link
Copy Markdown
Collaborator Author

I would like to start with BDW first. If we see it regularly on BYT we can very easily add it later.

@XiaoyunWu6666

Copy link
Copy Markdown
Contributor

@marc-hb

marc-hb commented Feb 23, 2022

Copy link
Copy Markdown
Collaborator Author

This is being skipped from the (unfortunately closed-source) test runner now, example: https://sof-ci.01.org/sofpr/PR5393/build12115/devicetest/?model=BDW_WSB_RT286&testcase=multiple-pipeline-capture

@marc-hb marc-hb closed this Feb 23, 2022
@marc-hb marc-hb reopened this Feb 23, 2022
@marc-hb

marc-hb commented Feb 23, 2022

Copy link
Copy Markdown
Collaborator Author

see alternative PR sof-framework/pull/185

This is too static, it requires to list the test cases one by one. In daily run 10459 only one test was skipped and 3 others failed. So let's skip in sof-test where we have the exact and up to date information at run-time no matter what exact test suite is run and no matter what test runner is used. Morever this skip runs after test setup.

Skipping in sof-framework can be useful sometimes but here it's not.

@marc-hb
marc-hb marked this pull request as ready for review February 23, 2022 23:55
@marc-hb
marc-hb merged commit ae42182 into thesofproject:main Feb 28, 2022
@marc-hb
marc-hb deleted the dis-cap-bdw branch February 28, 2022 17:25
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.

5 participants