Skip to content

Align third-party versions with calico-private release-calient-v3.22 - #5178

Merged
rene-dekker merged 1 commit into
release-v1.40from
dimitri-align-thirdparty-versions-v1.40
Aug 14, 2026
Merged

Align third-party versions with calico-private release-calient-v3.22#5178
rene-dekker merged 1 commit into
release-v1.40from
dimitri-align-thirdparty-versions-v1.40

Conversation

@dimitri-nicolo

Copy link
Copy Markdown
Contributor

Description

Moves this branch's third-party pins to match calico-private release-calient-v3.22, which is bumping them in tigera/calico-private#13238 (components) and #13255 (Go/k8s).

This branch tracks release-calient-v3.22 (config/enterprise_versions.ymllibcalico-go), and the operator deploys those images, so its pins have to move with calico's or it will deploy versions that no longer get built.

pin before after from
eck-kibana 8.19.19 8.19.20 #13238
eck-elasticsearch 8.19.19 8.19.20 #13238
coreos-prometheus v3.12.0 v3.13.2 #13238
coreos-alertmanager v0.32.1 v0.33.1 #13238
ISTIO_VERSION (Makefile) 1.29.2 1.29.6 #13238
GO_VERSION (Makefile) 1.25.12 1.25.13 #13255
api/go.mod go directive 1.25.12 1.25.13 #13255

GO_BUILD_VER is composed from GO_VERSION/LLVM_VERSION/K8S_VERSION, so it becomes 1.25.13-llvm18.1.8-k8s1.35.7 — byte-for-byte the build image calico-private v3.22 moves to in #13255. That tag is published; I built against it.

pkg/components/enterprise.go is regenerated from the yaml by gen-versions — not hand-edited — so validate-gen-versions stays clean.

Already aligned, so untouched

  • K8S_VERSION is already v1.35.7, which is exactly what #13255 moves v3.22 to.
  • eck-elasticsearch-operator is already 3.4.1, matching v3.22, and this branch's pkg/crds/enterprise/01-crd-eck-bundle.yaml is already the 3.4.1 bundle. (The v1.42 companion needs that bump; this branch doesn't.)

One judgement call

The api/go.mod go directive is normally owned by its own Renovate rule — 365268f8b bumped it to 1.25.12 separately from the GO_VERSION bump, so the two are tracked independently even though they currently agree. I've included it so they don't disagree on the branch after this PR, but it's the one line here that Renovate would otherwise carry on its own. Happy to drop it if you'd rather avoid the overlap.

Note this only changes the api submodule's language version; the main module stays at go 1.26.5 and the binary still compiles with Go 1.26.5 via the branch's GOTOOLCHAIN=go1.26.4+auto, unchanged by this PR.

Verified

  • go build ./... passes on calico/go-build:1.25.13-llvm18.1.8-k8s1.35.7.
  • Istio charts re-fetched at 1.29.6 — pkg/render/istio passes (no golden fixtures are pinned to the chart version).
  • pkg/render/logstorage/... (elasticsearch, kibana, eck, esgateway, esmetrics, linseed, dashboards) and pkg/components all pass.

Companion PR

Release Note

None

This branch tracks calico-private release-calient-v3.22 (see
config/enterprise_versions.yml libcalico-go), which is bumping its
third-party pins in tigera/calico-private#13238 and its Go/k8s pins in
tigera/calico-private#13255. The operator deploys those images, so its own
pins have to move with them.

| pin | before | after | source |
|---|---|---|---|
| eck-kibana | 8.19.19 | 8.19.20 | #13238 |
| eck-elasticsearch | 8.19.19 | 8.19.20 | #13238 |
| coreos-prometheus | v3.12.0 | v3.13.2 | #13238 |
| coreos-alertmanager | v0.32.1 | v0.33.1 | #13238 |
| ISTIO_VERSION | 1.29.2 | 1.29.6 | #13238 |
| GO_VERSION | 1.25.12 | 1.25.13 | #13255 |
| api/go.mod go directive | 1.25.12 | 1.25.13 | #13255 |

GO_BUILD_VER is composed from GO_VERSION/LLVM_VERSION/K8S_VERSION, so it
becomes 1.25.13-llvm18.1.8-k8s1.35.7 — the same build image calico-private
v3.22 moves to in #13255. That image is published.

pkg/components/enterprise.go is regenerated from the yaml by gen-versions,
not hand-edited.

Already aligned, so not touched: K8S_VERSION is v1.35.7 and
eck-elasticsearch-operator is 3.4.1, which is what v3.22 already targets.

The api/go.mod directive is normally owned by its own Renovate rule
(365268f bumped it to 1.25.12 separately from the GO_VERSION bump). It is
included here so the two do not disagree on the same branch; drop it if you
would rather let Renovate carry it.

Verified:
- go build ./... passes on calico/go-build:1.25.13-llvm18.1.8-k8s1.35.7.
- Istio charts re-fetched at 1.29.6; pkg/render/istio passes.
- pkg/render/logstorage/... (elasticsearch, kibana, eck) and pkg/components
  all pass.
Copilot AI lite review requested due to automatic review settings August 13, 2026 23:51
@dimitri-nicolo
dimitri-nicolo requested a review from a team as a code owner August 13, 2026 23:51
@marvin-tigera marvin-tigera added this to the v1.40.14 milestone Aug 13, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Aligns the operator’s pinned third-party component versions and build/tooling versions with the upstream calico-private release-calient-v3.22 branch, preventing the operator from deploying image versions that are no longer built.

Changes:

  • Bump ECK Kibana/Elasticsearch pins to 8.19.20 and CoreOS Prometheus/Alertmanager pins to v3.13.2 / v0.33.1.
  • Update build-time pins in the Makefile: GO_VERSION to 1.25.13 and ISTIO_VERSION to 1.29.6.
  • Keep the API submodule’s go directive aligned with the Makefile Go version (1.25.13).

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
pkg/components/enterprise.go Updates generated component version constants to match the enterprise versions config.
Makefile Bumps Go build image Go version pin and Istio chart version pin.
config/enterprise_versions.yml Updates enterprise third-party version pins that drive generated component versions.
api/go.mod Updates the API submodule’s go directive to align with the updated Go pin.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@rene-dekker
rene-dekker merged commit 28723c7 into release-v1.40 Aug 14, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file docs-not-required release-note-not-required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants