[release-1.9] chore(ci): remove unused Xvfb from E2E test runner

[zdrapela]

Summary

• Cherry-pick of chore(ci): remove unused Xvfb from E2E test runner #5188 / RHIDP-15894 onto release-1.9
• Remove unused Xvfb / DISPLAY setup from .ci/pipelines/utils.sh (equivalent of testing.sh on main; testing.sh does not exist on this branch)

Playwright runs headless in CI, so the virtual display only added _XSERVTransmkdir / xkbcomp log noise.

Test plan

• Confirm PR diff is only .ci/pipelines/utils.sh
• e2e-ocp-helm (or equivalent release-1.9 e2e) reaches Playwright without Missing X server / DISPLAY errors
• Job log has no _XSERVTransmkdir / xkbcomp XF86* warnings

[rhdh-qodo-merge]

PR Summary by Qodo

CI: remove unused Xvfb/DISPLAY setup from E2E test runner (release-1.9)

⚙️ Configuration changes 🕐 Less than 10 minutes

AI Description

• Remove Xvfb startup and DISPLAY export from the E2E test runner.
• Drop Xvfb cleanup step to reduce CI log noise for headless Playwright.

Diagram

graph TD
  A["CI E2E job"] --> B["utils.sh: run_tests"] --> C("Playwright (headless)")
  B -. "removed" .-> D["Xvfb + DISPLAY"]

Loading

High-Level Assessment

Removing Xvfb is the simplest and correct approach given Playwright runs headless in CI; keeping Xvfb (even behind a flag) would add ongoing maintenance and doesn’t provide value unless headed-browser runs are a supported CI mode.

Files changed (1) +0 / -5

Other (1) +0 / -5

utils.shRemove Xvfb/DISPLAY setup and teardown from run_tests() +0/-5

Remove Xvfb/DISPLAY setup and teardown from run_tests()

• Deletes the background Xvfb launch and DISPLAY export before running Playwright-based E2E tests. Also removes the corresponding 'pkill Xvfb' cleanup step after tests complete.

.ci/pipelines/utils.sh

[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

[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.}

[zdrapela]

/retest

[github-actions]

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

• quay.io/rhdh-community/rhdh:pr-5190
• quay.io/rhdh-community/rhdh:pr-5190-203ce94d

[zdrapela]

/test e2e-ocp-helm

[openshift-ci]

@zdrapela: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name	Commit	Details	Required	Rerun command
ci/prow/e2e-ocp-helm	203ce94	link	true	/test e2e-ocp-helm

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.