1596586 - ubuntu1804 - refactor pulseaudio initialization

Status: RESOLVED WORKSFORME

(Firefox Build System :: Task Configuration, enhancement)

Description

[Edwin Takahashi (:egao | infrequent contributor)]

Currently, the start-pulseaudio step is in the base harness that is run by all test instances, not just Linux:

https://searchfox.org/mozilla-central/source/testing/mozharness/scripts/desktop_unittest.py#613-635

This should be considered a kludge as platform-specific items should not be located in a harness script run by all platforms.

Once debian10 works stabilizes, investigate if possible to move this step to a Linux specific piece of code.

Comment 2

[Edwin Takahashi (:egao | infrequent contributor)]

Attachment: Bug 1596586 - refactor pulseaudio initialization for docker-worker

Comment 3

[Edwin Takahashi (:egao | infrequent contributor)]

Attachment: Bug 1596586 - rename start_pulseaudio to load_pulseaudio_module

Changes:

• move mozharness call to load the module-null-sink to the base class.
• rename method to better reflect the intended action.

Comment 4

[BugBot [:suhaib / :marco/ :calixte]]

The bug assignee didn't login in Bugzilla in the last 7 months.
:ahal, could you have a look please?
For more information, please visit auto_nag documentation.

Comment 5

[Andrew Halberstadt [:ahal]]

This is marked as blocking the move to 18.04, but that happened awhile back. Plus we're likely going to be migrating to a newer Ubuntu some point in the next year or so. So going to mark this as WFM.