Skip to content
This repository has been archived by the owner on Apr 18, 2024. It is now read-only.

keep nodes even if bad. stronger up-weighting #49

Closed
wants to merge 1 commit into from

Conversation

willscott
Copy link
Contributor

Without digging deep into error logs, this updates two of the most likely places that could lead to #48

  • for now, don't populate the removedTimeCache so that nodes given by the orchestrator stay considered even if we'd previously decided they were failing
  • if a node is partially failing, don't restrict the 'upvotes' to the same schedule as the 'downvotes', so a node that mostly is succeeding will stay in consideration.

lidel added a commit to ipfs-inactive/bifrost-gateway that referenced this pull request Feb 24, 2023
Copy link
Contributor

@lidel lidel left a comment

Choose a reason for hiding this comment

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

Deployed bifrost-gateway 2023-02-24-95c903b with this hotfix to bifrost-stage1-ny
Data flow is restored 👍

@aarshkshah1992
Copy link
Contributor

@willscott Raised #53 to counter this.

@aarshkshah1992
Copy link
Contributor

Should be taken care of by #59.

We should close this PR once #59 lands.

@aarshkshah1992
Copy link
Contributor

Superseded by #59

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants