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

fix: Update Gardener kubectl How-To guide #298

Merged

Conversation

colder-is-better
Copy link
Contributor

From Kubernetes 1.27 on, we support certificate-based kubeconfig files only. For that, we update the prose and all relevant screenshots. We also add a couple of new screenshots and modify the prose to fix some inaccuracies and better showcase the installation of the first sample application using kubectl.

Copy link
Contributor

@fghaas fghaas left a comment

Choose a reason for hiding this comment

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

Looks good in general; I have a few notes.

From Kubernetes 1.27 on, we support certificate-based
kubeconfig files only. For that, we update the prose
and all relevant screenshots. We also add a couple
of new screenshots and modify the prose to fix some
inaccuracies and better showcase the installation
of the first sample application using kubectl.
@colder-is-better colder-is-better force-pushed the managing_shoot_howto_refresh branch from de3a71c to 7715f3e Compare October 2, 2024 12:11
Copy link
Contributor

@fghaas fghaas left a comment

Choose a reason for hiding this comment

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

Looks good to me now. Any second thoughts, or is this OK to be merged?

@colder-is-better
Copy link
Contributor Author

As it is, I cannot think of a reason not to merge. Thanks!

@fghaas fghaas merged commit 7715f3e into citynetwork:main Oct 2, 2024
3 checks passed
@colder-is-better colder-is-better deleted the managing_shoot_howto_refresh branch December 19, 2024 18:56
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.

2 participants