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

Support user defined kubeconfig, fix merging context #266

Merged
merged 2 commits into from
Dec 6, 2023

Conversation

dereknola
Copy link
Member

@dereknola dereknola commented Dec 1, 2023

Signed-off-by: Derek Nola [email protected]

Changes

This is a follow on to discussion in #263

  • Adds new var kubeconfig which if specified copies the ansible cluster kubeconfig to that path location
  • Cleans up merging of kuebconfig context under default conditions. Now covers the cases of:
    • Kubectl is not installed on the control node
    • An existing cluster is already using the default context. We now merge everything under the k3s-ansible context/cluster/user.
    • Swapping to a different inventory.yml setup and deploying again. We correctly overwrite the k3s-ansible context.

@dereknola dereknola merged commit 9998f50 into master Dec 6, 2023
2 checks passed
@dereknola dereknola deleted the alt_kubeconfig branch December 20, 2023 18:20
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