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

update default k8s version to v1.30 #381

Merged
merged 1 commit into from
Sep 3, 2024

Conversation

tapas4java
Copy link
Contributor

Updating default k8s version

Copy link
Contributor

@nabuskey nabuskey left a comment

Choose a reason for hiding this comment

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

LGTM

@nabuskey nabuskey merged commit 1284628 into cnoe-io:main Sep 3, 2024
5 checks passed
@OneCricketeer
Copy link

Should ENVTEST_K8S_VERSION be updated as well?

@@ -47,7 +47,7 @@ func init() {
// cluster related flags
CreateCmd.PersistentFlags().BoolVar(&recreateCluster, "recreate", false, "Delete cluster first if it already exists.")
CreateCmd.PersistentFlags().StringVar(&buildName, "build-name", "localdev", "Name for build (Prefix for kind cluster name, pod names, etc).")
CreateCmd.PersistentFlags().StringVar(&kubeVersion, "kube-version", "v1.29.2", "Version of the kind kubernetes cluster to create.")
CreateCmd.PersistentFlags().StringVar(&kubeVersion, "kube-version", "v1.30.0", "Version of the kind kubernetes cluster to create.")

Choose a reason for hiding this comment

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

Why not latest patch release? 1.30.4?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah good idea. I looked at the kind version we are using and its default k8s version (v1.30.0). Bumping it up to the next patch release is a good idea. Your PR would be awesome here 😃

@cmoulliard
Copy link
Contributor

A table about the kubernetes version(s) supported should be included part of the documentation and release note !

@nabuskey
Copy link
Contributor

nabuskey commented Sep 6, 2024

Agreed. I really would love to automate this bit for sure.

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.

4 participants