Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(crossplane): upgrade controller v1.19.0 and providers #718

Merged
merged 4 commits into from
Feb 15, 2025

Conversation

Smana
Copy link
Owner

@Smana Smana commented Feb 15, 2025

PR Type

Enhancement, Dependencies


Description

  • Upgraded Crossplane controller to version 1.19.0.

  • Updated multiple Crossplane providers to v1.20.1.

  • Bumped Flux Operator version to 0.14.0.

  • Updated Zitadel Helm chart to version 8.11.3.


Changes walkthrough 📝

Relevant files
Enhancement
9 files
variables.tf
Updated Flux Operator version to 0.14.0                                   
+1/-1     
helmrelease.yaml
Upgraded Crossplane Helm chart to version 1.19.0                 
+1/-1     
provider-ec2.yaml
Updated AWS EC2 provider to version 1.20.1                             
+1/-1     
provider-eks.yaml
Updated AWS EKS provider to version 1.20.1                             
+1/-1     
provider-iam.yaml
Updated AWS IAM provider to version 1.20.1                             
+1/-1     
provider-kms.yaml
Updated AWS KMS provider to version 1.20.1                             
+1/-1     
provider-kubernetes.yaml
Updated Kubernetes provider to version 0.16.3                       
+1/-1     
provider-s3.yaml
Updated AWS S3 provider to version 1.20.1                               
+1/-1     
helmrelease.yaml
Updated Zitadel Helm chart to version 8.11.3                         
+1/-1     
Dependencies
2 files
go.mod
Updated Go dependencies to newer versions                               
+8/-8     
go.sum
Updated Go module checksums for dependency updates             
+12/-12 

Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.
  • Copy link
    Contributor

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Recommended focus areas for review

    Default Value Update

    The default value for flux_operator_version has been updated to "0.14.0". Ensure this change aligns with the intended deployment and compatibility requirements.

    default     = "0.14.0"
    Crossplane Version Update

    The Crossplane Helm chart version has been updated to "1.19.0". Verify that this version is compatible with the rest of the infrastructure and does not introduce breaking changes.

    version: "1.19.0"
    Zitadel Chart Version Update

    The Zitadel Helm chart version has been updated to "8.11.3". Confirm that this version is compatible with the existing configuration and does not introduce any regressions.

    version: "8.11.3"

    Copy link
    Contributor

    PR Code Suggestions ✨

    Explore these optional code suggestions:

    CategorySuggestion                                                                                                                                    Impact
    General
    Verify compatibility of updated chart version

    Ensure compatibility of the updated Crossplane chart version 1.19.0 with the
    existing configurations and dependencies to avoid runtime issues.

    infrastructure/base/crossplane/controller/helmrelease.yaml [16]

    -version: "1.19.0"
    +version: "1.19.0" # Ensure compatibility with configurations
    Suggestion importance[1-10]: 5

    __

    Why: The suggestion to ensure compatibility of the updated Crossplane chart version is valid and important to avoid runtime issues. However, it is not actionable as it only advises verification without providing specific improvements or changes.

    Low
    Validate compatibility of updated provider version

    Validate that the updated provider package version v1.20.1 is compatible with the
    runtime configuration and does not introduce breaking changes.

    infrastructure/base/crossplane/providers/provider-ec2.yaml [6]

    -package: xpkg.upbound.io/upbound/provider-aws-ec2:v1.20.1
    +package: xpkg.upbound.io/upbound/provider-aws-ec2:v1.20.1 # Ensure compatibility with runtime configuration
    Suggestion importance[1-10]: 5

    __

    Why: The suggestion to validate compatibility of the updated provider package version is relevant and helps prevent potential issues. However, it is not actionable and does not provide specific guidance or changes, reducing its impact.

    Low
    Test updated Zitadel chart version

    Test the updated Zitadel chart version 8.11.3 to ensure it does not introduce
    unexpected behavior or require additional configuration changes.

    security/base/zitadel/helmrelease.yaml [13]

    -version: "8.11.3"
    +version: "8.11.3" # Test for compatibility and required changes
    Suggestion importance[1-10]: 5

    __

    Why: Testing the updated Zitadel chart version is a reasonable suggestion to ensure stability and compatibility. However, it is not actionable and only advises testing without offering concrete improvements or changes.

    Low

    @Smana Smana force-pushed the chore_crossplane_v1.19.0 branch from 383bdfe to 6389b28 Compare February 15, 2025 08:31
    @Smana Smana force-pushed the chore_crossplane_v1.19.0 branch from 6389b28 to 5e27f53 Compare February 15, 2025 08:33
    @Smana Smana merged commit ec09756 into main Feb 15, 2025
    3 checks passed
    @Smana Smana deleted the chore_crossplane_v1.19.0 branch February 15, 2025 09:01
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    1 participant