MaterializeInc / k8s-eip-operator

Apache License 2.0
11 stars 2 forks source link

handle unresponsive nodes #474

Closed evanharmon closed 6 days ago

evanharmon commented 3 weeks ago

Motivation

Updates the eip-operator to handle unresponsive egress nodes. A new label is set on egress nodes indicating their status. New logic is added to disassociate EIPs from unresponsive nodes. A separate function handles cleaning up unresponsive egress nodes to update their status label.

Important: The README.md has been updated to reflect this repo being archived in the near future.

Notes for reviewer

Clippy warnings did not like the nested conditional blocks on match statements. I did some reasonable refactoring to meet the warnings. Not sure if it actually helps readability. What was more important to me was minimizing unnecessary API calls where possible.

Testing

I've tested egress nodes going unresponsive and egress nodes being rolled as well. The code changes match the existing behavior except now egress moves to the new node when the old one goes unresponsive.