Skip to content

Make kube-proxy Service stoppable#574

Closed
mangelajo wants to merge 1 commit intoopenshift:mainfrom
mangelajo:kube-proxy-stoppable
Closed

Make kube-proxy Service stoppable#574
mangelajo wants to merge 1 commit intoopenshift:mainfrom
mangelajo:kube-proxy-stoppable

Conversation

@mangelajo
Copy link
Copy Markdown
Contributor

The kube-proxy k8s code does not support an stop channel, but
at least we can accept cancel on the context and move on, so
MicroShift can be stopped properly when necessary.

Related-Issue: #556

Signed-off-by: Miguel Angel Ajo majopela@redhat.com

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Jan 27, 2022

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please ask for approval from mangelajo after the PR has been reviewed.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci Bot requested review from cooktheryan and copejon January 27, 2022 11:07
@mangelajo mangelajo requested review from fzdarsky and oglok January 27, 2022 11:18
@mangelajo mangelajo force-pushed the kube-proxy-stoppable branch from dd0b062 to 298fdce Compare January 27, 2022 15:40
Comment thread pkg/node/kube-proxy.go Outdated
@mangelajo
Copy link
Copy Markdown
Contributor Author

@fzdarsky @copejon @oglok ^ %)

The kube-proxy k8s code does not support an stop channel, but
at least we can accept cancel on the context and move on, so
MicroShift can be stopped properly when necessary.

Related-Issue: openshift#556

Signed-off-by: Miguel Angel Ajo <majopela@redhat.com>
@mangelajo mangelajo force-pushed the kube-proxy-stoppable branch from 660eff9 to ff937d6 Compare March 9, 2022 10:19
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Mar 9, 2022

@mangelajo: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-openshift-conformance-sig-storage ff937d6 link false /test e2e-openshift-conformance-sig-storage
ci/prow/e2e-openshift-conformance-sig-node ff937d6 link false /test e2e-openshift-conformance-sig-node
ci/prow/e2e-reboot ff937d6 link false /test e2e-reboot

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@mangelajo mangelajo added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Mar 9, 2022
@mangelajo
Copy link
Copy Markdown
Contributor Author

Being handled on another PR

@mangelajo mangelajo closed this Mar 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants