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

Blacklist improvements #207

Open
nritsche opened this issue Jun 3, 2020 · 0 comments
Open

Blacklist improvements #207

nritsche opened this issue Jun 3, 2020 · 0 comments
Labels
enhancement New feature or request

Comments

@nritsche
Copy link
Contributor

nritsche commented Jun 3, 2020

Problem:
Nodes get blacklisted for various reasons and we need some way to attach the reason to the changes of the blacklist so that it's easier to follow up on it.

Solutions:

  • Add command: blacklist for a limited time
  • Add save/load blacklist functionality
    • Very easy to implement
    • The issue with that is that you don't know how far back to undo, e.g. imagine that this happens:
      • Black list a rack because it's bad
      • Black list a node because it's bad
      • Black list another rack because it's bad
      • Niko fixes both bad racks
      • Now which state do you revert back to?
  • Multiple levels of blacklists (hierarchy)
  • Multiple named blacklists
@nritsche nritsche added the enhancement New feature or request label Jun 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant