-
Notifications
You must be signed in to change notification settings - Fork 2
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
Min Pinners #17
Min Pinners #17
Conversation
… pinners we want.
SKY-279 Cluster-wide setting for underpinned threshold
we should make the Originally posted by @MSevey in https://github.com/SkynetLabs/pinner/pull/2#discussion_r856472538 ProposalAs we want this to be a cluster-wide setting, we cannot use an environment variable for it because each server will have its own value and they will need to do a complicated negotiation in order to decide on a common, cluster-wide number. That's why we should add a GitHub: #8 |
# Conflicts: # workers/scanner.go
# Conflicts: # workers/scanner.go
# Conflicts: # workers/scanner.go
PULL REQUEST
Overview
Add a cluster-wide configuration field defining the minimum number of pinners we want.
In the process of doing that, this PR also defines a DB collection for cluster-wide configuration values and a generic way for getting those.
Example for Visual Changes
Checklist
Review and complete the checklist to ensure that the PR is complete before assigned to an approver.
Issues Closed
Closes #8
Closes SKY-279