-
Notifications
You must be signed in to change notification settings - Fork 453
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
exportKubeConfig.server not working after 0.21.4 #2390
Comments
To confirm, you don't have a secret on your exportKubeConfig setting ? If you did it would end up changing the other secret. |
The name of the secret is I've made a minimal example where I've created an vcluster named
The way I've tested the minimal example is with minikube with flux installed with In the secret In the secret If you want, I can also try running helm directly, not using flux, but I'm not personally convinced it would have a different outcome, but I'd be willing to try. |
I just tested with running the helm CLI directly on a clean cluster, and the results are the same, server gets set on values.yaml:
Commands:
|
@jawsper This looks like its by design , added in the PR - #2273 The default secret that gets created in vcluster namespace is not updated, you need to add a different secret name in the eg:
|
Oh wow, that is exactly what I needed! This way it also prevents Sorry for all the noise here! Would it be possible to add explicitly to the documentation that it's required to set a secret name if you want to adjust the server? |
Yeah we can add it to the documentation. |
Agree that the above is confusing. We have written up a wider solution that will allow for more flexibility and less confusion in the future. When that gets published, the docs will naturally get upgraded to reflect that system. In the short term if you'd like to make a PR to the docs to reflect the current situation, that would be great, you can assign me as a reviewer. |
What happened?
I created a vcluster using the helm chart version 0.22.4, where I set this in the values:
(
${env}
gets replaced by the postBuild of the flux kustomization)But the generated config still has:
What did you expect to happen?
I expected the config to contain the configured server:
How can we reproduce it (as minimally and precisely as possible)?
Use helm chart 0.22.4 or later, and set the config above.
Anything else we need to know?
I downgraded to 0.21.4, and it worked in that version. Additionally, it does work to change other elements of the exported kubeconfig, specifically
context
worked for me.Host cluster Kubernetes version
vcluster version
Though I never used the CLI since I did everything with the helm chart.
VCluster Config
Where
${env}
gets replaced with the specific environment I created.The text was updated successfully, but these errors were encountered: