Skip to content

Run the Linux starter canary under Xvfb - #5863

Merged
shai-almog merged 1 commit into
masterfrom
fix/starter-canary-xvfb
Sep 18, 2026
Merged

shai-almog merged 1 commit into
masterfrom
fix/starter-canary-xvfb

Conversation

@shai-almog

Copy link
Copy Markdown
Collaborator

The first live canary run exposed a runner problem rather than a starter regression: Windows completed build 23635151-1559-4cfc-83c6-54bdfe6b2283, while Ubuntu failed in CompileCSSMojo before submission because the headless runner had no X display. CN1 CSS compilation initializes Java AWT, and the repository already runs equivalent Linux builds under Xvfb.

This change:

  • installs Xvfb only on the Linux matrix leg through the existing bounded scripts/ci/apt-get-install.sh helper
  • runs the complete Linux canary under xvfb-run -a, so Maven and the forked CSS compiler inherit DISPLAY
  • leaves the Windows command unchanged
  • adds workflow-contract tests so the Linux display requirement cannot disappear silently

Verification:

  • python3 scripts/ci/starter-canary/test_starter_canary.py — 45 passed
  • scripts/check-workflows.sh .github/workflows/starter-canary.yml — passed
  • git diff --check — passed

Follow-up after merge: dispatch the live canary again. Both OS legs passing will close #5862 automatically.

@shai-almog
shai-almog merged commit 2bc65b5 into master Sep 18, 2026
20 checks passed
@shai-almog
shai-almog deleted the fix/starter-canary-xvfb branch September 18, 2026 18:49
@github-actions

Copy link
Copy Markdown
Contributor

✅ Continuous Quality Report

Test & Coverage

Static Analysis

  • SpotBugs [Report archive]
    • ✅ ByteCodeTranslator: 0 findings (no issues)
    • ✅ android: 0 findings (no issues)
    • ✅ build-hint-catalog: 0 findings (no issues)
    • ✅ build-hint-tools: 0 findings (no issues)
    • ✅ codenameone-maven-plugin: 0 findings (no issues)
    • ✅ core-unittests: 0 findings (no issues)
    • ✅ ios: 0 findings (no issues)
  • ✅ PMD: 0 findings (no issues) [Report archive]
  • ✅ Checkstyle: 0 findings (no issues) [Report archive]

Generated automatically by the PR CI workflow.

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.

Cloud starter is broken for new users

1 participant