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

fix: fix validator update logic #187

Merged
merged 3 commits into from
Apr 20, 2023
Merged

Conversation

keefel
Copy link
Contributor

@keefel keefel commented Apr 20, 2023

Description

When the relayer address or bls key changed, the tendermint can't sync this change, so this PR is aimed to fix this issue, update the cross-chain related fields when updating the validator set.

Rationale

Fix validator update logic of cross-chain part.

Example

NA

Changes

Notable changes:

  • Update validator set when cross-chain related fields changed.

@forcodedancing
Copy link
Collaborator

For Edit validator relayer address/blskey, do we need to include it in this pr?

@unclezoro
Copy link
Collaborator

For Edit validator relayer address/blskey, do we need to include it in this pr?

It is ok to add it in next PR.

@unclezoro unclezoro merged commit 059767b into bnb-chain:develop Apr 20, 2023
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