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 kubeconfig script #3881

Closed
wants to merge 2 commits into from
Closed

Fix kubeconfig script #3881

wants to merge 2 commits into from

Conversation

bigkevmcd
Copy link
Contributor

@bigkevmcd bigkevmcd commented Jul 25, 2023

Closes

What changed?
Fixes the indentation in the static-kubeconfig.sh to generate valid YAML.

Why was this change made?
The script was generating invalid YAML

How was this change implemented?

How did you validate the change?
Running the recommend command from the docs with the old version creates an invalid YAML that fails with kubectl --kubeconfig with invalid YAML on line 5, running it with a file generated using this command it works.

Release notes
No

Documentation Changes

@bigkevmcd bigkevmcd added the documentation Improvements or additions to documentation label Jul 25, 2023
This updates the static-kubeconfig.sh script to generate valid YAML.
@bigkevmcd bigkevmcd force-pushed the fix-kubeconfig-script branch from ba291fd to 0622dfb Compare July 25, 2023 19:28
@lasomethingsomething
Copy link
Contributor

Weren't any reviewers chosen so I added a few to get this moving forward.

@enekofb
Copy link
Contributor

enekofb commented Sep 6, 2023

reviewed e2e journey for that documentation with a few suggestions in form of new PR #3988

@lasomethingsomething
Copy link
Contributor

What's the status of this?

@bigkevmcd bigkevmcd closed this Nov 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants