Update flux-operator to 0.22.0, Flux to 2.6.x - #1035
Conversation
Signed-off-by: Kingdon B <[email protected]>
WalkthroughThis update increases the version numbers for both the Flux Operator and flux-instance Helm charts from 0.20.0 to 0.22.0. It also introduces stricter validation rules and schema enhancements for several CustomResourceDefinitions (CRDs) in the FluxCD operator, improving resource validation and flexibility in referencing input providers. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant Helm
participant Kubernetes API
participant CRD Validator
User->>Helm: Install/Upgrade flux-operator chart (v0.22.0)
Helm->>Kubernetes API: Apply CRDs with enhanced schemas
Kubernetes API->>CRD Validator: Validate CRD objects on creation/update
CRD Validator-->>Kubernetes API: Enforce new rules (name, fields, selectors)
Kubernetes API-->>User: Accept or reject resources based on validation
Possibly related PRs
Suggested labels
Suggested reviewers
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
packages/system/fluxcd-operator/charts/flux-operator/templates/crds.yaml (1)
918-928: Introduce static provider and optional URL.
AddingStaticto thetypeenum and relaxing theurlpattern to allow empty strings enables static input providers.Consider simplifying the regex to
^(https?://.*)?$for readability.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (5)
packages/system/fluxcd-operator/charts/flux-operator/Chart.yaml(2 hunks)packages/system/fluxcd-operator/charts/flux-operator/README.md(1 hunks)packages/system/fluxcd-operator/charts/flux-operator/templates/crds.yaml(6 hunks)packages/system/fluxcd/charts/flux-instance/Chart.yaml(2 hunks)packages/system/fluxcd/charts/flux-instance/README.md(1 hunks)
🔇 Additional comments (11)
packages/system/fluxcd/charts/flux-instance/README.md (1)
3-3: Version bump is consistent with Chart.yaml.
The Version and AppVersion badges have been updated to 0.22.0, aligning with the chart metadata.packages/system/fluxcd-operator/charts/flux-operator/README.md (1)
3-3: Version bump is consistent with chart metadata.
The Version and AppVersion badges have been updated to 0.22.0, aligning with Chart.yaml.packages/system/fluxcd/charts/flux-instance/Chart.yaml (2)
11-11: Update appVersion to v0.22.0.
Matches the latest Flux 2.6.1 release and aligns with flux-operator version bump.
28-28: Bump chart version to 0.22.0.
This matches the appVersion update and ensures coherent releases.packages/system/fluxcd-operator/charts/flux-operator/Chart.yaml (2)
11-11: Update appVersion to v0.22.0.
Aligns with the upstream Flux 2.6.1 release.
28-28: Bump chart version to 0.22.0.
Ensures the Helm chart version aligns with appVersion.packages/system/fluxcd-operator/charts/flux-operator/templates/crds.yaml (5)
487-490: Enforce single FluxInstance naming.
Thex-kubernetes-validationsrule ensures that FluxInstance resources must be named "flux", preventing duplicate instances.
559-574: Add cluster metadata to FluxReport spec.
Including node count, platform, and serverVersion enriches the report with cluster context. Ensure clients consuming the CRD handle the optionalclusterblock.
770-773: Enforce single FluxReport naming.
Thex-kubernetes-validationsrule restricts FluxReport resources to the name "flux", aligning with FluxInstance naming conventions.
933-938: Validate URL presence based on provider type.
The dualx-kubernetes-validationsensureurlis empty forStaticproviders and required otherwise.
1166-1182: Enhance inputsFrom schema for flexible referencing.
Allowing eithernameorselectorwith atomic map type and enforcing mutual exclusivity improves input provider references.Also applies to: 1221-1228
Signed-off-by: Kingdon B <[email protected]>
Andrei Kvapil (kvaps)
left a comment
There was a problem hiding this comment.
LGTM, thank you!
Flux 2.6.1 is the latest Flux release now
Summary by CodeRabbit
New Features
Bug Fixes
Documentation