-
Notifications
You must be signed in to change notification settings - Fork 392
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
pkg/daemon: run login monitor for Degraded nodes #375
pkg/daemon: run login monitor for Degraded nodes #375
Conversation
/approve |
/hold until pre-req is merged |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/approve
Yes, this should be the correct behaviour, thanks for the fix!
If there would ever be a way to reconcile a Degraded node, I believe it would be worth to still learn if someone jumped on a node and messed up with it so we can decide what to do. Signed-off-by: Antonio Murdaca <runcom@linux.com>
c79a817
to
eea9777
Compare
rebased and push forced |
/hold cancel |
pre-req merged, code looks fine to me. Deferring to a second review for merge. |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ashcrow, cgwalters, runcom, yuqi-zhang The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
If there would ever be a way to reconcile a Degraded node, I believe it
would be worth to still learn if someone jumped on a node and messed up
with it so we can decide what to do.
Feel free to close this telling I'm dumb as well, I'm probably missing some context which led to this PR while playing around reading the code but it makes sense to me to still have something like this. This is also not fixing anything related to super early startup where someone can jump on a node before the daemon is running and tracking accesses.
This PR builds on #373, I'll rebase it once that merges