Support non-headless mode for autowebcompat-repro - #6689
Open
jgraham wants to merge 3 commits into
Open
Conversation
jgraham
force-pushed
the
autowebcompat_no_headless
branch
from
August 21, 2026 17:41
6b334ff to
e472205
Compare
jgraham
force-pushed
the
autowebcompat_repro_script_mozregression
branch
from
August 21, 2026 17:42
f36b21f to
b9a8db0
Compare
jgraham
force-pushed
the
autowebcompat_no_headless
branch
2 times, most recently
from
August 21, 2026 17:49
8c5b6c0 to
2837a4a
Compare
Base automatically changed from
autowebcompat_repro_script_mozregression
to
master
August 21, 2026 17:57
ksy36
force-pushed
the
autowebcompat_no_headless
branch
from
August 21, 2026 17:57
2837a4a to
f9dd5d1
Compare
jgraham
force-pushed
the
autowebcompat_no_headless
branch
from
August 25, 2026 14:51
f9dd5d1 to
cdaf5c6
Compare
ksy36
reviewed
Aug 26, 2026
| && chown agent:agent /workspace /app/repro | ||
| && chown agent:agent /workspace /app/repro \ | ||
| && usermod -a -G sudo agent \ | ||
| && echo 'ALL ALL = (ALL) NOPASSWD: ALL' >> /etc/sudoers |
Contributor
There was a problem hiding this comment.
Wonder if there's a reason to give the agent passwordless root? fwiw I tried it without that, dropped the sudo package, the usermod/sudoers lines, and the two sudo prefixes in start_xvfb() and the display works fine (took a screenshot from inside the container to confirm).
Member
Author
There was a problem hiding this comment.
Apparently not! I thought this was needed, but now removed since it doesn't seem to be.
ksy36
reviewed
Aug 26, 2026
| ), | ||
| ] | ||
| ) | ||
| self.start(["sudo", "fluxbox", "-display", os.environ["DISPLAY"]]) |
Contributor
There was a problem hiding this comment.
This won't be a problem on production, only for development, but I've noticed if the container is reused (docker compose up on a stopped-but-not-removed container rather than a fresh one), the second run fails to get a display:
autowebcompat-repro-agent-1 | (EE)
autowebcompat-repro-agent-1 | Fatal server error:
autowebcompat-repro-agent-1 | (EE) Server is already active for display 99
autowebcompat-repro-agent-1 | If this server is no longer running, remove /tmp/.X99-lock
autowebcompat-repro-agent-1 | and start again.
autowebcompat-repro-agent-1 | (EE)
autowebcompat-repro-agent-1 | Warning: Failed to open file(fluxbox.cat)
autowebcompat-repro-agent-1 | for translation, using default messages.
autowebcompat-repro-agent-1 | Error: Couldn't connect to XServer:99.0
Member
Author
There was a problem hiding this comment.
I've tried to make the shutdown cleaner, I think this will help.
jgraham
force-pushed
the
autowebcompat_no_headless
branch
from
August 26, 2026 08:39
cdaf5c6 to
5297eac
Compare
This uses fluxbox running in Xvfb as the WM/display, similar to the configuration we have for web-platform-tests
jgraham
force-pushed
the
autowebcompat_no_headless
branch
from
August 26, 2026 08:41
5297eac to
dd9a6ec
Compare
ksy36
approved these changes
Aug 26, 2026
jgraham
enabled auto-merge (squash)
August 27, 2026 13:35
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This uses fluxbox running in Xvfb as the WM/display, similar to the configuration we have for web-platform-tests