Page MenuHomePhabricator
Create Task
Maniphest T412259

Update PatchDemo to Node 20
Closed, ResolvedPublic2 Estimated Story Points
Actions

Description

Currently we're still on Node 18, which is breaking PatchDemo builds that include a Codex patch (for example, try to build a PatchDemo for https://gerrit.wikimedia.org/r/c/design/codex/+/1217251/ ), because Codex's build process requires Node 20.

Details

Related Changes in GitLab:
TitleReferenceAuthorSource BranchDest Branch
Bump mw-chart-baserepos/test-platform/catalyst/ci-charts!112jhuneidiT412259main
catalyst-mw-chart-base: bump node versionrepos/releng/dev-images!86jhuneidiT412259main
Customize query in GitLab

Event Timeline

Catrope created this task.Dec 10 2025, 7:54 PM
Restricted Application added a subscriber: • Aklapper. · View Herald TranscriptDec 10 2025, 7:54 PM
Catrope added a comment.Dec 10 2025, 8:19 PM

Unfortunately this probably requires updating the nodejs20-slim image as well, because that includes Node v20.5.1 but Codex requires >=20.19.1.

• thcipriani subscribed.Jan 8 2026, 6:11 PM

For PatchDemo it looks like we're using upstream node images and stealing bits of them for our image: https://gitlab.wikimedia.org/repos/releng/dev-images/-/blob/main/dockerfiles/catalyst-mw-chart-base/Dockerfile.template?ref_type=heads#L1

So maybe not as big of a lift here.

• thcipriani edited projects, added: Catalyst (Luka Ijo Pimeja Jan); removed: Catalyst (PatchDemo).Jan 8 2026, 6:12 PM
• thcipriani assigned this task to jeena.Jan 8 2026, 6:18 PM
• thcipriani set the point value for this task to 2.
• thcipriani moved this task from Backlog to Ready on the Catalyst (Luka Ijo Pimeja Jan) board.
jeena moved this task from Ready to In progress on the Catalyst (Luka Ijo Pimeja Jan) board.Jan 13 2026, 8:46 PM
CodeReviewBot added a project: Patch-For-Review.Jan 13 2026, 8:48 PM

jhuneidi opened https://gitlab.wikimedia.org/repos/releng/dev-images/-/merge_requests/86

catalyst-mw-chart-base: bump node version

CodeReviewBot added a comment.Jan 14 2026, 9:51 PM

jhuneidi merged https://gitlab.wikimedia.org/repos/releng/dev-images/-/merge_requests/86

catalyst-mw-chart-base: bump node version

Stashbot added a comment.Jan 14 2026, 10:12 PM

Mentioned in SAL (#wikimedia-operations) [2026-01-14T22:12:10Z] <jeena> Updating development images on contint primary for T412259

Maintenance_bot removed a project: Patch-For-Review.Jan 14 2026, 10:31 PM
CodeReviewBot added a project: Patch-For-Review.Jan 16 2026, 8:21 PM

jhuneidi opened https://gitlab.wikimedia.org/repos/test-platform/catalyst/ci-charts/-/merge_requests/112

Bump mw-chart-base

CodeReviewBot added a comment.Jan 20 2026, 5:52 PM

jhuneidi merged https://gitlab.wikimedia.org/repos/test-platform/catalyst/ci-charts/-/merge_requests/112

Bump mw-chart-base

jeena added a comment.Jan 20 2026, 6:16 PM

@Catrope, I updated our install image to use node 22 and was able to create a demo with a codex patch. Can you confirm the issue is resolved?

Maintenance_bot removed a project: Patch-For-Review.Jan 20 2026, 6:31 PM
• thcipriani moved this task from In progress to Waiting for review on the Catalyst (Luka Ijo Pimeja Jan) board.Jan 20 2026, 7:07 PM
• thcipriani closed this task as Resolved.Jan 22 2026, 5:46 PM

Optimistically closing! We can create a demo with codex and it succeeds! @Catrope please reopen if you're still seeing issues.

Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits