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

feat(EKS cluster configuration): add support for specifying Karpenter version #115

Merged
merged 1 commit into from
Jun 14, 2024

Conversation

jaxxstorm
Copy link
Member

@jaxxstorm jaxxstorm commented Jun 14, 2024

🚀 This description was created by Ellipsis for commit dce762c

Summary:

Added support for specifying the Karpenter version in the EKS cluster configuration across multiple SDKs.

Key points:

  • Added karpenterVersion parameter to ClusterArgs in provider/pkg/provider/cluster.go.
  • Updated CRD URLs in NewCluster function to use karpenterVersion in provider/pkg/provider/cluster.go.
  • Modified schema.yaml to include karpenterVersion with a default value of 0.36.2.
  • Updated .NET SDK in sdk/dotnet/Eks/Cluster.cs to include KarpenterVersion property.
  • Updated Go SDK in sdk/go/eks/cluster.go to include KarpenterVersion property.
  • Updated Node.js SDK in sdk/nodejs/cluster.ts to include karpenterVersion property.
  • Updated Python SDK in sdk/python/lbrlabs_pulumi_eks/cluster.py to include karpenter_version property.

Generated with ❤️ by ellipsis.dev

@ellipsis-dev ellipsis-dev bot changed the title ... feat(EKS cluster configuration): add support for specifying Karpenter version Jun 14, 2024
Copy link

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

❌ Changes requested. Reviewed everything up to dce762c in 3 minutes and 20 seconds

More details
  • Looked at 232 lines of code in 6 files
  • Skipped 0 files when reviewing.
  • Skipped posting 0 drafted comments based on config settings.

Workflow ID: wflow_9h4CQ2rShpbRwIdB


Want Ellipsis to fix these issues? Tag @ellipsis-dev in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

sdk/go/eks/cluster.go Show resolved Hide resolved
@jaxxstorm jaxxstorm merged commit 265166a into main Jun 14, 2024
15 checks passed
@jaxxstorm jaxxstorm deleted the karpenter_version branch June 14, 2024 02:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant