Skip to content

Sequencer coordinator: reconnect to chosen if initial connection fails - #797

Merged
PlasmaPower merged 4 commits into
masterfrom
seq-coordinator-reconnect
Jul 15, 2022
Merged

Sequencer coordinator: reconnect to chosen if initial connection fails#797
PlasmaPower merged 4 commits into
masterfrom
seq-coordinator-reconnect

Conversation

@PlasmaPower

Copy link
Copy Markdown
Contributor

No description provided.

@PlasmaPower PlasmaPower added the node Changes the node software label Jul 15, 2022
@PlasmaPower
PlasmaPower requested a review from tsahee July 15, 2022 01:42
@cla-bot cla-bot Bot added the cla-signed label Jul 15, 2022
Comment thread arbnode/seq_coordinator.go Outdated
tsahee
tsahee previously approved these changes Jul 15, 2022

@tsahee tsahee left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think this should work.. another option in comments

Comment thread arbnode/seq_coordinator.go Outdated
@PlasmaPower
PlasmaPower enabled auto-merge July 15, 2022 03:06
@codecov

codecov Bot commented Jul 15, 2022

Copy link
Copy Markdown

Codecov Report

Merging #797 (0fc0bc3) into master (c22c64f) will decrease coverage by 4.02%.
The diff coverage is 55.00%.

❗ Current head 0fc0bc3 differs from pull request most recent head 6676e00. Consider uploading reports for the commit 6676e00 to get more accurate results

@@            Coverage Diff             @@
##           master     #797      +/-   ##
==========================================
- Coverage   52.72%   48.69%   -4.03%     
==========================================
  Files         223      211      -12     
  Lines       27052    22933    -4119     
  Branches      494      494              
==========================================
- Hits        14262    11167    -3095     
+ Misses      11251    10236    -1015     
+ Partials     1539     1530       -9     

@PlasmaPower
PlasmaPower merged commit 7f407ae into master Jul 15, 2022
@hkalodner
hkalodner deleted the seq-coordinator-reconnect branch July 15, 2022 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed node Changes the node software

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants