Skip to content

chore(release): prepare v3.14.0 Helm charts - #534

Merged
flemzord merged 1 commit into
mainfrom
feat/prepare-v3.14.0-charts
Sep 2, 2026
Merged

chore(release): prepare v3.14.0 Helm charts#534
flemzord merged 1 commit into
mainfrom
feat/prepare-v3.14.0-charts

Conversation

@flemzord

@flemzord flemzord commented Sep 2, 2026

Copy link
Copy Markdown
Member

Summary

  • bump the operator and operator-crds Helm charts to 3.14.0
  • set both application versions to v3.14.0
  • regenerate the operator chart dependency lock

Impact

This prepares the next minor Operator release. It does not create the Git tag or publish release artifacts; those remain post-merge steps.

Validation

  • nix develop --impure --command just pre-commit
  • nix develop --impure --command just tests
  • helm lint --strict and helm template for both charts

@flemzord
flemzord requested a review from a team as a code owner September 2, 2026 09:31
@NumaryBot

Copy link
Copy Markdown
Contributor

✅ Approve — automated review

The chart and application versions are consistently updated to 3.14.0, and the regenerated dependency lock resolves the matching CRD chart version.

No findings.

@flemzord
flemzord merged commit e771b8d into main Sep 2, 2026
10 checks passed
@flemzord
flemzord deleted the feat/prepare-v3.14.0-charts branch September 2, 2026 09:33
@shipfox-ai

shipfox-ai Bot commented Sep 2, 2026

Copy link
Copy Markdown

Final Review — PR #534: chore(release): prepare v3.14.0 Helm charts

Summary. This is a pure release-preparation version bump of the Helm charts from 3.13.4 to 3.14.0. I independently verified every substantive claim from both reviews against the working tree: helm/crds/Chart.yaml and helm/operator/Chart.yaml both carry version: "3.14.0" / appVersion: "v3.14.0"; helm/operator/Chart.lock locks operator-crds at 3.14.0 (with matching digest/timestamp); the operator chart's dependency constraint version: "3.X" is satisfied by 3.14.0; a repo-wide grep for 3.13.4 across yaml/yml/lock/md/tpl returns no matches; and helm/operator/templates/deployment.yaml:35 and :65 correctly default the image tag and --utils-version to .Chart.AppVersion, with values.yaml leaving tag: "" so the bump drives the deployed versions. The two reviews agree and report no defects; that assessment holds up. Overall recommendation: approve.

Confirmed findings

No material finding was confirmed. The change is internally consistent, contains no leftover old-version references, and introduces no correctness, security, compatibility, or test-coverage risk. Reviewer A's remaining points (images must be published post-merge; the machine-generated Chart.lock digest could not be recomputed without a helm binary) are informational and not defects — the publish ordering is the standard release flow, and the locked dependency version/repository are consistent with the Chart.yaml changes.


Reviewed by Claude (claude-opus-4.8) and Codex (gpt-5.6-sol) via Shipfox; verified and synthesized by Claude.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants