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

delay kubectl calls when required subs are departing #28

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

delay kubectl calls when required subs are departing #28

wants to merge 1 commit into from

Conversation

kwmonroe
Copy link
Member

@kwmonroe kwmonroe commented Apr 8, 2022 •
edited
Loading

When we know required subordinate charms are going away, don't run kubectl commands -- they take forever to time out.

Fixes https://bugs.launchpad.net/charm-kubernetes-master/+bug/1968237

Depends on charmed-kubernetes/charm-kubernetes-control-plane#214

@kwmonroe kwmonroe marked this pull request as draft April 8, 2022 14:06
@kwmonroe
Copy link
Member Author

kwmonroe commented Apr 8, 2022

Need to noodle on this a bit more. I think there are cases where kubectl can succeed without subordinates (fetching secrets). Also need to think about people that are switching cni/cri but aren't tearing the whole cluster down.

@kwmonroe kwmonroe mentioned this pull request Apr 8, 2022
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@kwmonroe