Repository navigation
chore: prepare Operator v3.17.0 (EN-2723) - #549
Conversation
|
🟡 [minor] Correct and complete the schema migration inventory The upgrade gate says monitoring.traces is removed, but the head CRD retains traces and removes only traces.sampling (config CRD lines 2120–2144). The diff also removes persistence.coldCache and replaces monitoring.pyroscope.authToken/basicAuthPassword with Secret references, neither of which is listed. An operator using this inventory to prepare migration can overlook pruned profiling credentials/cache configuration while unnecessarily removing valid tracing configuration. Suggestion: List traces.sampling rather than traces, include coldCache and both Pyroscope credential replacements, and document moving the profiling credentials into referenced Secrets before applying the schema. |
|
Addressed the schema migration finding in head The notes now list all 35 removed property paths, including Verified against the generated candidate schema and |
|
Current-head CI checkpoint for Tests and Dirty passed. Kubernetes E2E 1.31 and 1.32 passed after one individual retry each for Docker Hub HTTP 500 errors before tests; 1.35 passed initially. Kubernetes 1.34 also passed after its single individual network retry. The run is complete and remains failed solely on the 1.33 cleanup below. No further job retry is planned.
The independent review confirms the cleanup observation and that this test fixture is unchanged from v3.16.0. Root-cause evidence is still needed before changing the frozen candidate ( |
NumaryBot
left a comment
There was a problem hiding this comment.
The required automated review completed with no remaining findings.
NumaryBot
left a comment
There was a problem hiding this comment.
The required automated review completed with no remaining findings.
NumaryBot
left a comment
There was a problem hiding this comment.
The required automated review completed with no remaining findings.
NumaryBot
left a comment
There was a problem hiding this comment.
The required automated review completed with no remaining findings.
Summary
Prepare the Operator v3.17.0 minor release from
main, including the changes merged since v3.16.0:operatorandoperator-crdschart versions and appVersions.The Credentials fixes (#548 and #544) are already merged into
mainand are included in this release candidate. They producesuperuser=falsewith the required Ledger scopes, matching the Ledger Operator beta.10 Credentials schema. The Credentials CRD is provided by the Ledger Operator chart.Validation
Both charts pass Helm lint, the Operator chart renders successfully, and
git diff --checkpasses. Full local Go tests and Nix pre-commit were not rerun. CI and reviews must cover the latest commit.This PR prepares the version; artifact publication and runtime qualification remain pending.
EN-2723