Skip to content

Conversation

@fredoh9
Copy link
Contributor

@fredoh9 fredoh9 commented Jun 28, 2020

Current SOF CI is not use pulseaudio.

Signed-off-by: Fred Oh fred.oh@linux.intel.com

Found consistent error while restoring pulseaudio in Ubuntu 20.04 devices.

2020-06-25 23:55:26 UTC [REMOTE_INFO] checking for general errors after kmod insert with sof-kernel-log-check tool
2020-06-25 23:55:26 UTC [REMOTE_INFO] checking if firmware is loaded successfully
2020-06-25 23:55:26 UTC [REMOTE_INFO] ==== firmware boot complete: 2 of 2 ====
2020-06-25 23:55:27 UTC [REMOTE_INFO] Restoring pulseaudio
usage: sudo -h | -K | -k | -V
usage: sudo -v [-AknS] [-g group] [-h host] [-p prompt] [-u user]
usage: sudo -l [-AknS] [-g group] [-h host] [-p prompt] [-U user] [-u user]
            [command]
usage: sudo [-AbEHknPS] [-r role] [-t type] [-C num] [-g group] [-h host] [-p
            prompt] [-T timeout] [-u user] [VAR=value] [-i|-s] []
usage: sudo -e [-AknS] [-r role] [-t type] [-C num] [-g group] [-h host] [-p
            prompt] [-T timeout] [-u user] file ...
2020-06-25 23:55:37 UTC [REMOTE_INFO] Time out. Pulseaudio not restored in 10 seconds
2020-06-25 23:55:37 UTC [REMOTE_INFO] Test Result: FAIL!

Current SOF CI is not use pulseaudio.

Signed-off-by: Fred Oh <fred.oh@linux.intel.com>
Copy link
Collaborator

@marc-hb marc-hb left a comment

Choose a reason for hiding this comment

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

pulseaudio can have significant side effects so we cannot run tests in an "unknown pulseaudio state".

This looks like a simple sudo syntax error, let's just investigate and fix it.

Are these errors observed in CI too? if not why not?

Did you also observe errors while disabling it? If not why ignore errors we haven't seen yet?

@fredoh9
Copy link
Contributor Author

fredoh9 commented Jun 29, 2020

The proper fix for sudo is posted,
#266

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