Skip to content

check-alsabat: add option to test with pipewire#1310

Merged
KamilxPaszkiet merged 1 commit intothesofproject:mainfrom
ekurdybx:1312-pipewire-with-loopback
Dec 15, 2025
Merged

check-alsabat: add option to test with pipewire#1310
KamilxPaszkiet merged 1 commit intothesofproject:mainfrom
ekurdybx:1312-pipewire-with-loopback

Conversation

@ekurdybx
Copy link
Contributor

@ekurdybx ekurdybx commented Oct 16, 2025 •
edited
Loading

  1. Added option to run alsabat test with pipewire - checks what sinks/sources are corresponding to given alsa devices, sets them as defaults and run alsabat tests
  2. Added sleep to enable_pipewire func - without it test fails sporadically, it was tested
  3. Removed running aplay with "timeout" when using pipewire - pipewire does support time parameter, so it's not needed

@sofci
Copy link
Collaborator

sofci commented Oct 16, 2025

Can one of the admins verify this patch?

reply test this please to run this test once

@ekurdybx ekurdybx force-pushed the 1312-pipewire-with-loopback branch 5 times, most recently from cf54cca to 7cfa5bb Compare October 22, 2025 13:52
@ekurdybx ekurdybx force-pushed the 1312-pipewire-with-loopback branch 2 times, most recently from 1d4526e to 92ddd47 Compare November 24, 2025 13:42
@ekurdybx ekurdybx force-pushed the 1312-pipewire-with-loopback branch from 92ddd47 to 15019f3 Compare December 5, 2025 08:38
@ekurdybx ekurdybx force-pushed the 1312-pipewire-with-loopback branch 5 times, most recently from 2e5d782 to 979eb11 Compare December 12, 2025 15:39
@ekurdybx ekurdybx marked this pull request as ready for review December 15, 2025 09:30
@ekurdybx ekurdybx requested review from a team, golowanow, lgirdwood and marc-hb as code owners December 15, 2025 09:30
@ekurdybx ekurdybx marked this pull request as draft December 15, 2025 11:45
@ekurdybx ekurdybx force-pushed the 1312-pipewire-with-loopback branch from 979eb11 to 97b2b93 Compare December 15, 2025 13:03
@ekurdybx ekurdybx force-pushed the 1312-pipewire-with-loopback branch from 97b2b93 to 88cc644 Compare December 15, 2025 13:39
@ekurdybx ekurdybx marked this pull request as ready for review December 15, 2025 14:25
@KamilxPaszkiet KamilxPaszkiet merged commit 0f42a37 into thesofproject:main Dec 15, 2025
5 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@PLangowski PLangowski PLangowski approved these changes

@KamilxPaszkiet KamilxPaszkiet KamilxPaszkiet approved these changes

@lgirdwood lgirdwood Awaiting requested review from lgirdwood lgirdwood is a code owner

@marc-hb marc-hb Awaiting requested review from marc-hb marc-hb is a code owner

@golowanow golowanow Awaiting requested review from golowanow golowanow is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ekurdybx @sofci @PLangowski @KamilxPaszkiet