-
-
Notifications
You must be signed in to change notification settings - Fork 93
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
Problem with add and merge #602
Comments
In order to avoid cluster name conflicts, cluster names are renamed when adding and merging. I would also like to try to change this situation, please ask if there is a better suggestion for improvement to solve this problem. |
Just check for duplicate names first or do a dry-run of the merge or merge it to a temp place first and check if the names are ok? With the current renaming it is pretty hard to remove a config based on the clustername. |
This issue will be fixed in the next version. |
I love the concept of kubecm, but it wrecked my existing kube config in the first minute of trying it out when I used merge to add a kubeconfig to my existing config. |
@soostdijck Very sorry for the bad experience. You can actually use the |
Describe the bug
When merging 2 configs or adding a config, the name is not preserved
To Reproduce
I have a cluster
+------------+---------+------------+ .....
|..CURRENT...|....NAME...|...CLUSTER..| .....
+=========+=======+=========+ .....
| ....................|..test-vm..|...test-vm.....| .....
When adding a second config via kubecm add -f /tmp/config or kubecm merge "~/.kube/config" /tmp/config -y the name (context) of the first cluster is changed to config
+------------+---------+------------+ .....
|..CURRENT...|....NAME...|...CLUSTER..| .....
+=========+=======+=========+ .....
| ....................|...config...|...test-vm.....| .....
+------------+---------+------------+ .....
| ....................|..cluster2.|....cluster2....| .....
Expected behavior
It should not rename the cluster config name
Desktop (please complete the following information):
Thanks!
The text was updated successfully, but these errors were encountered: