Skip to content

ZOOKEEPER-5116: Pin website and CI build runner to ubuntu-24.04 to fix website and CI build - #2487

Merged
kezhuw merged 4 commits into
apache:masterfrom
PDavid:ZOOKEEPER-5116
Nov 4, 2026
Merged

kezhuw merged 4 commits into
apache:masterfrom
PDavid:ZOOKEEPER-5116

Conversation

@PDavid

@PDavid PDavid commented Nov 2, 2026 •

Copy link
Copy Markdown
Contributor

Pin website CI runner to ubuntu-24.04

Playwright's bundled browsers are compiled against ICU 74 which is
not available on Ubuntu 26.04 (the current ubuntu-latest). Pin the
runner to ubuntu-24.04 where the required system libraries
(libicudata.so.74, libicui18n.so.74, libicuuc.so.74, libxml2.so.2)
are available.

Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com

Pin CI build runner to ubuntu-24.04

The newer GCC on Ubuntu 26.04 is stricter about strchr() returning const char* when given a const char* input (aligning with C23 semantics), which triggers -Werror=discarded-qualifiers.

About runner pinning

Ubuntu 24.04 LTS reaches end of life in April 2029, but GitHub typically drops runner images ~6 months after EOL, so realistically mid-to-late 2029.

For the website the longer-term fix is upgrading Playwright to a version that supports the newer Ubuntu (and its ICU version). Worth revisiting when Playwright releases support for 26.04, at which point you can switch back to ubuntu-latest.

Use newer actions/cache version

To be consistent with other usages of cache action.

@PDavid PDavid self-assigned this Nov 2, 2026
Playwright's bundled browsers are compiled against ICU 74 which is
not available on Ubuntu 26.04 (the current ubuntu-latest). Pin the
runner to ubuntu-24.04 where the required system libraries
(libicudata.so.74, libicui18n.so.74, libicuuc.so.74, libxml2.so.2)
are available.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@PDavid PDavid changed the title ZOOKEEPER-5116: Split Playwright install to fix missing system dependencies in CI ZOOKEEPER-5116: Pin website CI runner to ubuntu-24.04 Nov 3, 2026
@PDavid PDavid changed the title ZOOKEEPER-5116: Pin website CI runner to ubuntu-24.04 ZOOKEEPER-5116: Pin website CI runner to ubuntu-24.04 to fix website CI build Nov 3, 2026
@PDavid
PDavid requested a review from anmolnar November 3, 2026 09:36
@yuriipalam

Copy link
Copy Markdown
Contributor

Thanks David, LGTM

@PDavid PDavid changed the title ZOOKEEPER-5116: Pin website CI runner to ubuntu-24.04 to fix website CI build ZOOKEEPER-5116: Pin website and CI build runner to ubuntu-24.04 to fix website and CI build Nov 3, 2026
@kezhuw
kezhuw merged commit a9d2ce6 into apache:master Nov 4, 2026
20 of 22 checks passed
@kezhuw

kezhuw commented Nov 4, 2026

Copy link
Copy Markdown
Member

Ci failure is unrelated. Merged. Thank you! @PDavid

@PDavid
PDavid deleted the ZOOKEEPER-5116 branch November 4, 2026 07:59
@PDavid

PDavid commented Nov 4, 2026

Copy link
Copy Markdown
Contributor Author

Ci failure is unrelated. Merged. Thank you! @PDavid

SaslAuthFailFipsModeTest.testDigestMd5RejectedInFipsMode test failed in the CI build - which is unrelated. Now the CI build is not stuck at the C client module, it goes through and runs the tests.
Many thanks @kezhuw for your review and for merging! 👍

anmolnar pushed a commit that referenced this pull request Nov 4, 2026
…2491)

Reviewers: kezhuw
Author: PDavid
Closes #2487 from PDavid/ZOOKEEPER-5116

(cherry picked from commit a9d2ce6)
anmolnar pushed a commit that referenced this pull request Nov 4, 2026
…2492)

Reviewers: kezhuw
Author: PDavid
Closes #2487 from PDavid/ZOOKEEPER-5116

(cherry picked from commit a9d2ce6)
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.

3 participants