chore(ci): drop leftover X11 MIT-SHM env from e2e-runner

[zdrapela]

Summary

• Remove unused QT_X11_NO_MITSHM, _X11_NO_MITSHM, and _MITSHM from .ci/images/Dockerfile
• Follow-up to chore(ci): remove unused Xvfb from E2E test runner #5188 / RHIDP-15894 after Xvfb was removed from the E2E test runner

These env vars only matter when an X server is in use. CI Playwright runs headless and no longer starts Xvfb.

Test plan

• Confirm PR diff is only .ci/images/Dockerfile
• e2e-runner image builds successfully on the PR
• An e2e job still reaches Playwright without Missing X server / DISPLAY errors

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[zdrapela]

/hold

[rhdh-qodo-merge]

PR Summary by Qodo

ci: Remove unused X11 MIT-SHM env vars from e2e-runner image

⚙️ Configuration changes 🕐 Less than 5 minutes

AI Description

• Remove obsolete X11 MIT-SHM environment variables from the e2e-runner Dockerfile.
• Align CI image config with headless Playwright runs (no Xvfb / no X server).
• Reduce CI image surface area and avoid misleading X11-related settings.

High-Level Assessment

The straightforward cleanup is appropriate: since CI Playwright runs headless and Xvfb is no longer started, keeping X11 MIT-SHM env vars only adds confusion. No alternative approach provides meaningful benefit beyond leaving dead config in place.

Files changed (1) +0 / -3

Other (1) +0 / -3

DockerfileDrop unused X11 MIT-SHM env vars from CI image +0/-3

Drop unused X11 MIT-SHM env vars from CI image

• Removes QT_X11_NO_MITSHM, _X11_NO_MITSHM, and _MITSHM from the e2e-runner container environment. These variables are only relevant when an X server is used, which is no longer the case for headless Playwright in CI.

.ci/images/Dockerfile

[rhdh-qodo-merge]

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

_{To customize comments, go to the Qodo configuration screen, or learn more in the docs.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.96%. Comparing base (ddfe8d0) to head (8a594a9).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #5191      +/-   ##
==========================================
- Coverage   63.69%   59.96%   -3.74%     
==========================================
  Files         123      111      -12     
  Lines        2424     2198     -226     
  Branches      548      546       -2     
==========================================
- Hits         1544     1318     -226     
  Misses        879      879              
  Partials        1        1              

Flag	Coverage Δ
rhdh	59.96% <ø> (-3.74%)	⬇️

Components	Coverage Δ
Backend plugins	∅ <ø> (∅)
Backend app	66.66% <ø> (ø)
Frontend app	58.89% <ø> (ø)
Plugin utils	∅ <ø> (∅)
Dynamic plugins utils	∅ <ø> (∅)

---

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ddfe8d0...8a594a9. Read the comment docs.

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[github-actions]

Image was built and published successfully. It is available at:

• quay.io/rhdh-community/rhdh:pr-5191
• quay.io/rhdh-community/rhdh:pr-5191-8a594a9b