You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
[FEATURE] sof-test needs to support Fedora 34 #633
I think the largest task is probably to integrate the Jenkins based audio test infrastructure we have to additionally support RPM pagkages. Today, Debian based tooling is used to update kernels, modules and tooling on each DUT and we need to make sure we can support RPM here too on F34 and beyond.
Agreed: 95% of sof-test should just work on Fedora (please report bugs and/or submit fixes) whereas the Debian packaging of the kernel and ALSA packages is obviously not going to work.
Custom ALSA packages are less often required on Fedora though :-) thesofproject/sof#2543
Checking group memberships: Warning
Solution:
Missing permission to run command as sudo
Please use the following command to add current user to the group sudo:
sudo usermod --append --groups sudo
why is this needed?
Missing /var/log/kern.log file, which is where we'll catch the kernel log
Please create the link of your distribution kernel log file at /var/log/kern.log
I thought we are now using journalctl? What is this dependency, is it still needed?
There are many commands in sof-test that need sudo, starting with inserting and removing modules.
I thought we are now using journalctl? What is this dependency, is it still needed?
It's unfortunately not complete, see internal issue 477 and others.
cut: /var/log/kern.log: No such file or directory [INFO] Test Result: PASS!
#312 should help, it has been implemented on some tests but not others. Does every test "PASS" like this?
The good news is: these are not new issues and not specific to Fedora. The bad news is: all this is down to prioritization. When everything is urgent, nothing is.
PR #639 was separated to two other PRs #643, #649 that were merged. So sof-test is ready. The remaining work is remove usage of sudo group but to use audio group for all OSes. Close this issue.
Activity
lgirdwood commentedon Mar 23, 2021
@perexg @jwrdegoede fyi
perexg commentedon Mar 24, 2021
Adding Wim to the chain. @wtay
Could you link us to scripts which should be checked / modified?
lgirdwood commentedon Mar 24, 2021
We have a lot of test scripts here https://github.com/thesofproject/sof-test/tree/master/test-case and IIIUC they all use the ALSA APIs (i.e. none use any PA APIs) so my expectation is that these should work (or require minimal configuration changes).
I think the largest task is probably to integrate the Jenkins based audio test infrastructure we have to additionally support RPM pagkages. Today, Debian based tooling is used to update kernels, modules and tooling on each DUT and we need to make sure we can support RPM here too on F34 and beyond.
@kv2019i @aiChaoSONG @marc-hb any further comments here ?
marc-hb commentedon Mar 24, 2021
Agreed: 95% of sof-test should just work on Fedora (please report bugs and/or submit fixes) whereas the Debian packaging of the kernel and ALSA packages is obviously not going to work.
Custom ALSA packages are less often required on Fedora though :-) thesofproject/sof#2543
Related: [RFC] enable sof-test on ChromeOS #576
plbossart commentedon Mar 29, 2021
Installing dependencies is straightforward:
the rest is less clear:
why is this needed?
I thought we are now using journalctl? What is this dependency, is it still needed?
plbossart commentedon Mar 29, 2021
fantastic.
marc-hb commentedon Mar 29, 2021
There are many commands in
sof-testthat needsudo, starting with inserting and removing modules.It's unfortunately not complete, see internal issue 477 and others.
#312 should help, it has been implemented on some tests but not others. Does every test "PASS" like this?
The good news is: these are not new issues and not specific to Fedora. The bad news is: all this is down to prioritization. When everything is urgent, nothing is.
aiChaoSONG commentedon Mar 30, 2021
@plbossart check this ongoing PR, it totally removes /var/log/kern.log, and use journalctl.
Edit: PR #639, @marc-hb thanks.
Unfortunately, only Ubuntu use this file
marc-hb commentedon Mar 30, 2021
I guess @aiChaoSONG is referring to #639
mengdonglin commentedon Apr 21, 2021
PR #639 was separated to two other PRs #643, #649 that were merged. So sof-test is ready. The remaining work is remove usage of sudo group but to use audio group for all OSes. Close this issue.