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

The kubectl plugin should log a warning if a Pod is missing #1640

Closed
johscheuer opened this issue May 22, 2023 · 0 comments · Fixed by #1669
Closed

The kubectl plugin should log a warning if a Pod is missing #1640

johscheuer opened this issue May 22, 2023 · 0 comments · Fixed by #1669
Labels
enhancement New feature or request plugin Kubectl plugin

Comments

@johscheuer
Copy link
Member

What would you like to be added/changed?

In some commands like the restart command missing Pods are currently ignored. We should log a warning if a Pod for a process group is missing. The code that has to be adjusted is: https://github.com/FoundationDB/fdb-kubernetes-operator/blob/main/kubectl-fdb/cmd/k8s_client.go#L163.
@johscheuer johscheuer added enhancement New feature or request plugin Kubectl plugin labels May 22, 2023
manfontan added a commit to cognitedata/fdb-kubernetes-operator that referenced this issue Jun 13, 2023
@manfontan
Fixes issue FoundationDB#1640
d50a7d8 
Signed-off-by: Manuel Fontan <[email protected]>
@manfontan manfontan mentioned this issue Jun 13, 2023
Merged
johscheuer added a commit that referenced this issue Jun 16, 2023
@manfontan @johscheuer
Fixes issue #1640 (#1669)
41bfc01 
* Fixes issue #1640

Signed-off-by: Manuel Fontan <[email protected]>
Co-authored-by: Johannes Scheuermann <[email protected]>
ammolitor pushed a commit that referenced this issue Jun 19, 2023
@manfontan @johscheuer @ammolitor
Fixes issue #1640 (#1669)
ccadbb0 
* Fixes issue #1640

Signed-off-by: Manuel Fontan <[email protected]>
Co-authored-by: Johannes Scheuermann <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request plugin Kubectl plugin
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixes issue #1640 cognitedata/fdb-kubernetes-operator
1 participant
@johscheuer