Skip to content

made the peer dialing less aggressive#2799

Merged
pompon0 merged 7 commits intorelease/v6.3from
gprusak-fix
Feb 6, 2026
Merged

made the peer dialing less aggressive#2799
pompon0 merged 7 commits intorelease/v6.3from
gprusak-fix

Conversation

@pompon0
Copy link
Contributor

@pompon0 pompon0 commented Feb 4, 2026

  • DialRate is now exposed in config.toml. It defaults to 10s (instead of previous 1.5s).
  • Reduced MaxAcceptRate from 0.5s to 1s
  • Fixed NewConnections metric (outgoing dials were reported twice). Added success/failure label.

@github-actions
Copy link

github-actions bot commented Feb 4, 2026

The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedFeb 6, 2026, 9:53 AM

DialTimeout time.Duration `mapstructure:"dial-timeout"`

// How often node should dial a new peer.
DialRate time.Duration `mapstructure:"dial-rate"`
Copy link
Contributor

Choose a reason for hiding this comment

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

Shall we also add to config.toml?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

you mean the template?

Copy link
Contributor

Choose a reason for hiding this comment

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

yeah

Copy link
Contributor Author

Choose a reason for hiding this comment

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

will add it in another pr

@codecov
Copy link

codecov bot commented Feb 4, 2026

Codecov Report

❌ Patch coverage is 60.00000% with 14 lines in your changes missing coverage. Please review.
✅ Project coverage is 43.36%. Comparing base (97d1db2) to head (fff7f01).
⚠️ Report is 1 commits behind head on release/v6.3.

Files with missing lines Patch % Lines
sei-tendermint/internal/p2p/metrics.gen.go 12.50% 7 Missing ⚠️
sei-tendermint/internal/p2p/peermanager.go 0.00% 6 Missing ⚠️
sei-tendermint/internal/p2p/router.go 93.75% 1 Missing ⚠️

❌ Your patch status has failed because the patch coverage (60.00%) is below the target coverage (70.00%). You can increase the patch coverage or adjust the target coverage.
❌ Your project status has failed because the head coverage (38.27%) is below the target coverage (40.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files

Impacted file tree graph

@@               Coverage Diff               @@
##           release/v6.3    #2799     +/-   ##
===============================================
  Coverage         43.36%   43.36%             
===============================================
  Files              1866     1811     -55     
  Lines            155305   149120   -6185     
===============================================
- Hits              67351    64673   -2678     
+ Misses            81941    78763   -3178     
+ Partials           6013     5684    -329     
Flag Coverage Δ
sei-chain 42.52% <ø> (-0.01%) ⬇️
sei-cosmos 38.03% <ø> (+<0.01%) ⬆️
sei-db 69.13% <ø> (+24.20%) ⬆️
sei-ibc-go 55.96% <ø> (ø)
sei-tendermint 47.51% <60.00%> (-0.03%) ⬇️
sei-wasmd 41.56% <ø> (-0.02%) ⬇️
sei-wasmvm 39.88% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
sei-tendermint/internal/p2p/channel.go 77.77% <100.00%> (+1.30%) ⬆️
sei-tendermint/internal/p2p/metrics.go 100.00% <ø> (ø)
sei-tendermint/internal/p2p/routeroptions.go 86.04% <100.00%> (ø)
sei-tendermint/node/setup.go 66.90% <100.00%> (+0.12%) ⬆️
sei-tendermint/internal/p2p/router.go 73.03% <93.75%> (-0.60%) ⬇️
sei-tendermint/internal/p2p/peermanager.go 84.07% <0.00%> (-1.92%) ⬇️
sei-tendermint/internal/p2p/metrics.gen.go 15.58% <12.50%> (-0.13%) ⬇️

... and 73 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@pompon0 pompon0 requested a review from philipsu522 February 4, 2026 17:14
@pompon0 pompon0 enabled auto-merge (squash) February 6, 2026 09:52
@pompon0 pompon0 merged commit 4436bed into release/v6.3 Feb 6, 2026
42 of 45 checks passed
@pompon0 pompon0 deleted the gprusak-fix branch February 6, 2026 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants