Skip to content
This repository was archived by the owner on Jan 22, 2025. It is now read-only.

Add watchtower feature to watch through multiple URLs#35435

Closed
Frixoe wants to merge 4 commits intosolana-labs:masterfrom
Frixoe:watchtower-multiple-urls
Closed

Add watchtower feature to watch through multiple URLs#35435
Frixoe wants to merge 4 commits intosolana-labs:masterfrom
Frixoe:watchtower-multiple-urls

Conversation

@Frixoe
Copy link
Copy Markdown

@Frixoe Frixoe commented Mar 9, 2024

Problem

Watchtower could only use one URL so during outages it would seem as if only that RPC might have been affected.

Summary of Changes

Added feature to pass multiple URLs to the CLI tool

  • Latency and health of every URL is evaluated and recorded at the start of the program and after every 10 iterations of the main loop.
  • Only the lowest latency URL is picked for operations
  • Latency for current URL is measured on every iteration
  • At the end of every iteration, the current URL's latency is compared to all existing URLs latencies since last evaluation. If any URL is found to have a lower latency, the first one in the evaluation loop is picked to be the current URL.

Fixes #

@Frixoe Frixoe changed the title Add feature to watch through multiple URLs Add watchtower feature to watch through multiple URLs Mar 9, 2024
@mergify mergify Bot added community Community contribution need:merge-assist labels Mar 9, 2024
@mergify mergify Bot requested a review from a team March 9, 2024 04:25
@steviez
Copy link
Copy Markdown
Contributor

steviez commented Mar 9, 2024

@Frixoe - Thanks for the contribution. We have recently forked this repo, and further work will be done in:
https://github.com/anza-xyz/agave

Feel free to re-open a new PR there; the PR against this repo will not be reviewed / merged.

@steviez steviez added the blocked Unable to proceed label Mar 9, 2024
@Frixoe Frixoe closed this by deleting the head repository Mar 9, 2024
@Frixoe
Copy link
Copy Markdown
Author

Frixoe commented Mar 9, 2024

@Frixoe - Thanks for the contribution. We have recently forked this repo, and further work will be done in: https://github.com/anza-xyz/agave

Feel free to re-open a new PR there; the PR against this repo will not be reviewed / merged.

Got it. I've opened a pull request on the new repo too. Please take a look.

anza-xyz#161

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

blocked Unable to proceed community Community contribution need:merge-assist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants