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.

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

Node Drain Request/Status API #4366

Merged
merged 1 commit into from
Nov 6, 2024

Conversation

andrewd-zededa
Copy link
Contributor

The base of an upcoming PR to enable deferring node reboot and shutdown operations. In an Edge-Node-Cluster each node can hold a replica copy of a cluster volume. If the other nodes 1 and 2 in a cluster have recently recovered from a failure they could be rebuilding their replicas (1.x, 1.y, 2.x, 2.y) off of the only remaining readable copies (3.x, 3.y) on node 3.
This API is to enable zedagent,nodeagent,baseosmgr to request a drain of replicas off of an edge-node (rebuild completion) before allowing a node outage.

@rene
Copy link
Contributor

rene commented Oct 17, 2024

@andrewd-zededa , you have some Yetus errors to fix.

@andrewd-zededa andrewd-zededa force-pushed the kubeapi-drain-api branch 3 times, most recently from 0517587 to f8ffca4 Compare October 17, 2024 14:10
@andrewd-zededa andrewd-zededa force-pushed the kubeapi-drain-api branch 3 times, most recently from 1bb11b3 to 8b25b80 Compare October 17, 2024 17:11
@andrewd-zededa
Copy link
Contributor Author

Forgot to actually commit the kube-status change in nodedrain.go, its there now.

The base of an upcoming PR to enable deferring node
reboot and shutdown operations.  In an Edge-Node-Cluster
each node can hold a replica copy of a cluster volume.
If the other nodes 1 and 2 in a cluster have recently recovered
from a failure they could be rebuilding their replicas
(1.x, 1.y, 2.x, 2.y) off of the only remaining readable copies
(3.x, 3.y) on node 3.
This API is to enable zedagent,nodeagent,baseosmgr to
request a drain of replicas off of an edge-node (rebuild completion) before allowing
a node outage.

Signed-off-by: Andrew Durbin <[email protected]>
@andrewd-zededa
Copy link
Contributor Author

@eriknordmark changes pushed up to address review questions. Please let me know if any further changes are needed.

Copy link
Contributor

@eriknordmark eriknordmark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@eriknordmark eriknordmark merged commit f4d5f73 into lf-edge:master Nov 6, 2024
29 of 40 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants