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

Bump kubernetes-client to 7.0.0 #247

Open
adietish opened this issue Oct 8, 2024 · 3 comments · May be fixed by #248
Open

Bump kubernetes-client to 7.0.0 #247

adietish opened this issue Oct 8, 2024 · 3 comments · May be fixed by #248
Assignees
Labels
enhancement New feature or request

Comments

@adietish
Copy link
Contributor

adietish commented Oct 8, 2024

related to

Support for multiple kubernetes config files will be available in kubernetes-client 7.0 (see fabric8io/kubernetes-client#6240, redhat-developer/intellij-kubernetes#779). We'll have to bump the kubernetes-client 7.0 in all our plugins/libraries

@adietish adietish self-assigned this Oct 8, 2024
@adietish adietish moved this to 📝 In Progress in IDE Cloudaptors Oct 8, 2024
@adietish adietish changed the title bump kubernetes-client to 7.0.0 Bump kubernetes-client to 7.0.0 Oct 8, 2024
@adietish adietish added the enhancement New feature or request label Oct 8, 2024
adietish added a commit to adietish/intellij-common that referenced this issue Oct 8, 2024
@adietish adietish moved this from 📝 In Progress to 📋 Backlog in IDE Cloudaptors Oct 25, 2024
@adietish
Copy link
Contributor Author

adietish commented Nov 11, 2024

This is a larger change. Kubernetes client now assumes that there are several config files, there's no way to limit yourself to a single one. The process should now work as follows:

  • watch all files (Config.getKubeconfigFilenames)
  • when a file changes -> create a new config (new ConfigBuilder().build())
  • then compare
    • current context
    • all contexts
    • auth token

@adietish
Copy link
Contributor Author

as of today kubernetes-client 7.0 wasn't released yet.

@adietish
Copy link
Contributor Author

adietish commented Dec 4, 2024

kubernetes-client 7.0.0 was released yesterday. They dropped config.getKubeconfigFilename(), replaced it with config.getKubeconfigFilenames() requiring us to add support multiple config files #256. For now we're using a shortcut where we use the first entry or the one that points to a file called config

@adietish adietish moved this from 📋 Backlog to 📝 In Progress in IDE Cloudaptors Dec 6, 2024
adietish added a commit to adietish/intellij-common that referenced this issue Dec 6, 2024
adietish added a commit to adietish/intellij-common that referenced this issue Dec 9, 2024
adietish added a commit to adietish/intellij-common that referenced this issue Dec 10, 2024
adietish added a commit to adietish/intellij-common that referenced this issue Dec 11, 2024
adietish added a commit to adietish/intellij-common that referenced this issue Dec 12, 2024
adietish added a commit to adietish/intellij-common that referenced this issue Dec 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: 📝 In Progress
Development

Successfully merging a pull request may close this issue.

1 participant