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

Replace blutgang with spamoor replicas #2370

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

fridrik01
Copy link
Contributor

This PR replaces the use of a load balancer (blutgang by default but ngingx also suppported) when running a single instance of spamoor with a running multiple replicas of spamoor where each target a single execution client.

This replicates how we conduct testing in our staging env we have had issues with blutgang where it was running unstably and restarting frequently. Also when scanning the blutgang logs it frequently removes nodes from the active set due to them falling behind (and then later adding them back) resulting in spamoor frequently hitting the same rpc node.

NOTE: The e2e CI test is failing, this is fixed in followup PR where we remove blutgang/nginx altogether from all integration tests

@fridrik01 fridrik01 self-assigned this Jan 18, 2025
Copy link

codecov bot commented Jan 18, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 31.15%. Comparing base (39c4ebe) to head (a6c6111).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #2370   +/-   ##
=======================================
  Coverage   31.15%   31.15%           
=======================================
  Files         346      346           
  Lines       15438    15438           
  Branches       20       20           
=======================================
  Hits         4810     4810           
  Misses      10289    10289           
  Partials      339      339           

@fridrik01 fridrik01 marked this pull request as ready for review January 18, 2025 18:39
@fridrik01 fridrik01 requested a review from a team as a code owner January 18, 2025 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant