Skip to content
This repository was archived by the owner on Aug 2, 2022. It is now read-only.

set medium priority for process signed block - 2.0.x - #9370

Merged
dimas1185 merged 1 commit into
release/2.0.xfrom
net_pluging_sign_block_med_priority_20x
Aug 14, 2020
Merged

dimas1185 merged 1 commit into
release/2.0.xfrom
net_pluging_sign_block_med_priority_20x

Conversation

@allenhan2

@allenhan2 allenhan2 commented Aug 7, 2020 •

Copy link
Copy Markdown
Contributor

Change Description

This is change related to EOSIO/eosio-security#340,

Still see the unlinkable block for each restart but it is expected when switching from in-sync to head-catchup and that is the current block that the node just produced

Co-Authored-By: Kevin Heifner heifnerk@objectcomputing.com

Change Type

Select ONE

  • Documentation
  • Stability bug fix
  • Other
  • Other - special case

Consensus Changes

  • Consensus Changes

API Changes

  • API Changes

Documentation Additions

  • Documentation Additions

0

@allenhan2 allenhan2 changed the title set medium priority for process signed block set medium priority for process signed block - 2.0.x Aug 7, 2020
@allenhan2

Copy link
Copy Markdown
Contributor Author

Will do more testing on 2.0.x with config bp<--API<--API.

@allenhan2

Copy link
Copy Markdown
Contributor Author

It works fine after test env config fix, see less " Unlinkable block", and syncing can always be re-established.

@allenhan2
allenhan2 removed the request for review from brianjohnson5972 August 14, 2020 16:55
@heifner

heifner commented Aug 14, 2020

Copy link
Copy Markdown
Contributor

For more information see #9369

@dimas1185
dimas1185 self-requested a review August 14, 2020 17:09
@dimas1185
dimas1185 merged commit 91e57cb into release/2.0.x Aug 14, 2020
@jeffreyssmith2nd
jeffreyssmith2nd deleted the net_pluging_sign_block_med_priority_20x branch August 14, 2020 20:16
@dimas1185

Copy link
Copy Markdown
Contributor

merged on behalf of Yue

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants