Skip to content

[meta-version-updater] Version updates - #225

Open
platformsh-devrel wants to merge 3 commits into
masterfrom
update/meta-version-updater
Open

platformsh-devrel wants to merge 3 commits into
masterfrom
update/meta-version-updater

Conversation

@platformsh-devrel

Copy link
Copy Markdown
Contributor

This PR is auto-generated by meta-version-updater.

Summary of updates

Resource File
runtime versions resources/image/registry.json

@upsun-dispatch

upsun-dispatch Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

📋 PR Summary

This is an automated meta-version-updater PR that bumps runtime package versions in the image registry. The latest push changes two entries: the 26.8 image goes from 26.8.31.2 to 26.8.32.14 and the 1.27 image goes from 1.27.2-0psh1 to 1.27.3-0psh1. Both entries get matching updates to their normalized patch versions.

Changes
Layer / File(s) Summary
Runtime version registry
resources/image/registry.json Updates the raw and normalized package versions for the 26.8 and 1.27 image entries to their latest patch releases.

@upsun-dispatch upsun-dispatch Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Warning

Changes suggested — 🟡 1 warning

🔍 Full review · 1 file reviewed

Verification
  • Both ClickHouse package_version bumps (26.8.31.2, 26.3.56.2) keep normalized equal to the first three components of raw and leave majorMinor unchanged.
  • The edited registry.json still parses as valid JSON.
  • The nodejs 24 status change is in the upstream block only; upsun.status stays supported, as it does for the other upstream-deprecated entries such as nodejs 22.

The diff adds or changes no tests. I found no code in the repository that reads these fields (a grep for isEoas in Go/JS/TS/Python/PHP sources turned up nothing), so nothing automated checks that status and isEoas agree. I checked the JSON parses and looked for field mismatches with a one-off Python script.

Review details
  • Commit: b4d1dab
  • Model: claude-opus-5-5

Review 1 of 10 for this pull request · View the full run

},
"upstream": {
"status": "supported",
"status": "deprecated",

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Warning — Docs and tooling that read isEoas will keep calling Node 24 actively supported after it was deprecated upstream.

This change moves nodejs 24 to upstream.status: "deprecated" but leaves isEoas: false in the same object, a few lines down. Its eoasFrom is 2026-10-20, which is today, so support has ended. Elsewhere in the registry, 14 deprecated entries pair the status with isEoas: true, including nodejs 22 just above. The new entry contradicts itself: a consumer that reads isEoas reports Node 24 as still supported, and one that reads status reports it as deprecated. The only other deprecated entries with isEoas: false are redis 8.0 and redis 7.2, and their inconsistency was already in the file before this PR.

@upsun-dispatch upsun-dispatch Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note

Reviewed — No new issues found · 1 still open

🔁 Incremental · 1 file reviewed

Outstanding from earlier reviews:

  • 🟡 #4320109209 — resources/image/registry.json:6356: Docs and tooling that read isEoas will keep calling Node 24 actively supported after it was deprecated upstream. — nodejs 24 upstream still reads status "deprecated" with isEoas false, even though eoasFrom is 2026-10-20 (today).
Verification
  • The ClickHouse 26.3 bump changes raw to 26.3.57.12 and normalized to 26.3.57; both still match majorMinor 26.3.
  • The 1.26 package bump changes raw to 1.26.10-0psh1 and normalized to 1.26.10; both stay consistent with majorMinor 1.26.
  • The merged registry.json still parses as valid JSON.

No test in this diff covers these changes. This pass only bumped two package_version entries in generated data, and I checked by hand that they agree with their majorMinor fields.

Review details

Review 2 of 10 for this pull request · View the full run

@upsun-dispatch upsun-dispatch Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note

Reviewed — No new issues found · 1 still open

🔁 Incremental · 1 file reviewed

Outstanding from earlier reviews:

  • 🟡 #4320109209 — resources/image/registry.json:6356: Docs and tooling that read isEoas will keep calling Node 24 actively supported after it was deprecated upstream. — nodejs 24 upstream still has status "deprecated" next to isEoas: false, even though eoasFrom is 2026-10-20 and that date has passed.
Verification
  • The ClickHouse 26.8 bump keeps raw 26.8.32.14, normalized 26.8.32 and majorMinor 26.8 consistent with each other.
  • The 1.27 bump keeps raw 1.27.3-0psh1 and normalized 1.27.3 consistent with majorMinor 1.27.
  • The merged registry.json still parses as valid JSON.

No tests cover this data-only registry update. I checked by hand that the changed version fields agree with each other and that the file still parses as JSON.

Review details

Review 3 of 10 for this pull request · View the full run

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.

1 participant