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

Proposal to change source of kubectl #8

Closed
nmofonseca opened this issue Jul 1, 2024 · 2 comments · Fixed by #9
Closed

Proposal to change source of kubectl #8

nmofonseca opened this issue Jul 1, 2024 · 2 comments · Fixed by #9
Assignees

Comments

@nmofonseca
Copy link

Hello @rpadovani ,

Would it be ok to change the kubectl provider to alekc/kubectl.

gavinbunney fork seems to have been abandoned and is also not working properly with K8s 1.27+ as can be seen here

Current versions.tf

terraform {
  required_providers {
    kubectl = {
      source = "gavinbunney/kubectl"
    }
  }
}

Proposed versions.tf

terraform {
  required_providers {
    kubectl = {
      source = "alekc/kubectl"
    }
  }
}

Thank you so much in advance.

@rpadovani
Copy link
Owner

Hi @nmofonseca, thanks for opening the issue, and for the information about the provider.

I think this is a good idea, however it will recreate all the CRDs, and that is a huge change. I think I will have to bump the major version to mark this change, and hopefully we are not going to break anything. I will work on it this week

@nmofonseca
Copy link
Author

Hello @rpadovani ,

Thanks for replying and accepting the change. I can also do some testing to confirm it will actually recreate everything.

Will let you know my findings

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 a pull request may close this issue.

2 participants