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.

Sign up for GitHub

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

Jump to bottom

Multiple k9s panels open in parallel for the same cluster breaks config.yaml #2527

Closed
ventsislav-georgiev opened this issue Feb 6, 2024 · 1 comment
Closed

Multiple k9s panels open in parallel for the same cluster breaks config.yaml #2527

ventsislav-georgiev opened this issue Feb 6, 2024 · 1 comment
Labels
bug Something isn't working InProgress Marks an issue has being worked on

Comments

@ventsislav-georgiev
Copy link




Describe the bug
There seems to be a race condition with the config.yaml generation in the k9s/cluster/../../config.yaml

To Reproduce
Steps to reproduce the behavior:
Using launch configuration from Warp (terminal) for example opening 4 or more k9s processes in separate panels for different resources/namespaces like:

  • k9s -n x1 -c dp
  • k9s -n x1 -c po
  • k9s -n x2 -c po
  • etc

Doing it multiple times breaks the config.yaml and k9s no longer works for that cluster until the config.yaml is manually deleted. There is an error message for config validation.

Screenshots
image

Versions (please complete the following information):

  • OS: OSX
  • K9s: 0.31.7
  • K8s: 1.28.3
@derailed
Copy link
Owner

derailed commented Feb 6, 2024

@ventsislav-georgiev Thank you for the heads up Ventsislav!
I've ran into this here as well. Will push a fix in the next drop.

@derailed derailed added bug Something isn't working InProgress Marks an issue has being worked on labels Feb 6, 2024
derailed added a commit that referenced this issue Feb 7, 2024
@derailed
[Bug] Fix #2527
11ec9c5
@derailed derailed mentioned this issue Feb 7, 2024
Merged
derailed added a commit that referenced this issue Feb 7, 2024
@derailed
[Bug] Fix #2527
368a59b
placintaalexandru pushed a commit to placintaalexandru/k9s that referenced this issue Apr 3, 2024
@derailed
K9s/release v0.31.8 (derailed#2528)
01a2f7c 
* [Maint] Fix race condition issue

* [Bug] Fix derailed#2501

* [Maint] Allow reference to resource aliases for plugins

* [Feat] Intro cp namespace command + misc cleanup

* [Maint] Rev k8s v0.29.1

* [Bug] Fix derailed#1033, derailed#1558

* [Bug] Fix derailed#2527

* [Bug] Fix derailed#2520

* rel v0.31.8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working InProgress Marks an issue has being worked on
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@derailed @ventsislav-georgiev