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

dkg: add customizable timeout #3130

Merged
merged 1 commit into from
Jun 11, 2024
Merged

dkg: add customizable timeout #3130

merged 1 commit into from
Jun 11, 2024

Conversation

gsora
Copy link
Collaborator

@gsora gsora commented Jun 11, 2024

Users might DKG large clusters, containing high amounts of validators (like 2.5K).

With large validator amounts, the DKG process needs either more time or more power (CPU, network bandwidth) to complete.

Allow users to specify a custom timeout for the entire process, allowing large DKGs on normal-powered machines.

Tested 2.5K validator DKGs locally with 0.relay.obol.tech, 10 minutes timeout: completed with no issues.

category: feature
ticket: none

Users might DKG large clusters, containing high amounts of validators (like 2.5K).

With large validator amounts, the DKG process needs either more time or more power (CPU, network bandwidth) to complete.

Allow users to specify a custom timeout for the entire process, allowing large DKGs on normal-powered machines.

Tested 2.5K validator DKGs locally with 0.relay.obol.tech, 10 minutes timeout: completed with no issues.
Copy link

sonarcloud bot commented Jun 11, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link

codecov bot commented Jun 11, 2024

Codecov Report

Attention: Patch coverage is 84.61538% with 2 lines in your changes missing coverage. Please review.

Project coverage is 56.73%. Comparing base (f286523) to head (c58c4bb).

Files Patch % Lines
cmd/dkg.go 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3130      +/-   ##
==========================================
- Coverage   56.73%   56.73%   -0.01%     
==========================================
  Files         208      208              
  Lines       29308    29317       +9     
==========================================
+ Hits        16628    16632       +4     
- Misses      10838    10845       +7     
+ Partials     1842     1840       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@gsora gsora added the merge when ready Indicates bulldozer bot may merge when all checks pass label Jun 11, 2024
@obol-bulldozer obol-bulldozer bot merged commit 360aece into main Jun 11, 2024
12 checks passed
@obol-bulldozer obol-bulldozer bot deleted the gsora/dkg-timeout branch June 11, 2024 11:34
gsora added a commit that referenced this pull request Jun 11, 2024
Users might DKG large clusters, containing high amounts of validators (like 2.5K).

With large validator amounts, the DKG process needs either more time or more power (CPU, network bandwidth) to complete.

Allow users to specify a custom timeout for the entire process, allowing large DKGs on normal-powered machines.

Tested 2.5K validator DKGs locally with 0.relay.obol.tech, 10 minutes timeout: completed with no issues.

category: feature
ticket: none
obol-bulldozer bot pushed a commit that referenced this pull request Jun 11, 2024
- #3130
- #3131
- #3132

category: misc
ticket: none
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge when ready Indicates bulldozer bot may merge when all checks pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants