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

Add flag to keep node cordoned for reap failures #60

Merged
merged 10 commits into from
Nov 15, 2021

Conversation

shailshah9
Copy link
Member

If the node fails to drain or if the node-reaper fails because of Bad PDBs, the new flag ignore-failure would keep the node cordoned so that no new pods are scheduled on the bad node.

@shailshah9 shailshah9 requested a review from a team as a code owner November 8, 2021 05:26
@shailshah9 shailshah9 force-pushed the feature/ignoreDrainFailure branch 3 times, most recently from 3f3e0ab to c6b65f7 Compare November 8, 2021 16:01
sshah31 and others added 2 commits November 8, 2021 08:06
Signed-off-by: shailshah9 <shailshah9@gmail.com>
…koproj#59)

Signed-off-by: Jonah Back <jonah@jonahback.com>
Signed-off-by: shailshah9 <shailshah9@gmail.com>
Signed-off-by: shailshah9 <shailshah9@gmail.com>
@codecov
Copy link

codecov bot commented Nov 9, 2021

Codecov Report

Merging #60 (c71d445) into master (edf7b7b) will increase coverage by 2.52%.
The diff coverage is 60.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #60      +/-   ##
==========================================
+ Coverage   60.00%   62.52%   +2.52%     
==========================================
  Files           9        9              
  Lines        1425     1425              
==========================================
+ Hits          855      891      +36     
+ Misses        476      441      -35     
+ Partials       94       93       -1     
Impacted Files Coverage Δ
pkg/reaper/nodereaper/types.go 89.47% <ø> (ø)
pkg/reaper/nodereaper/helpers.go 76.52% <60.00%> (+14.78%) ⬆️
pkg/reaper/nodereaper/nodereaper.go 46.79% <0.00%> (+0.41%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update edf7b7b...c71d445. Read the comment docs.

Signed-off-by: shailshah9 <shailshah9@gmail.com>
Copy link
Member

@tekenstam tekenstam left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@kevdowney kevdowney left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: shailshah9 <shailshah9@gmail.com>
Signed-off-by: shailshah9 <shailshah9@gmail.com>
Signed-off-by: shailshah9 <shailshah9@gmail.com>
Signed-off-by: shailshah9 <shailshah9@gmail.com>
@shailshah9 shailshah9 merged commit 2063eba into keikoproj:master Nov 15, 2021
shaoxt pushed a commit to shaoxt/governor that referenced this pull request Dec 21, 2021
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.

5 participants