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

Make AWM Primary Network Optimization configurable #1230

Open
martineckardt opened this issue Jul 9, 2024 · 2 comments
Open

Make AWM Primary Network Optimization configurable #1230

martineckardt opened this issue Jul 9, 2024 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@martineckardt
Copy link
Contributor

Context and scope
When running a Subnet with all validators in partial sync mode, it will not be able to verify incoming messages from the primary network. The optimization will try to gather signatures over the C-Chain's message from the Subnet's validator set. Since these do not sync the C-Chain they won't be able to sign the message.

We should make this optimization configurable.

Discussion and alternatives
TBD

Open questions
TBD

@martineckardt martineckardt added the enhancement New feature or request label Jul 9, 2024
@michaelkaplan13
Copy link

FWIW, this optimization will need to be removed all together in the E upgrade in order to allow for subnets with only subnet-only validators to verify messages from the C-Chain. This will also need to be accounted for in the awm-relayer so that it is able to determine which validators to query for BLS signatures.

@darioush darioush self-assigned this Jul 9, 2024
@aaronbuchwald
Copy link
Collaborator

It could still be configurable in the E Upgrade if subnets opt in to validating the C-Chain. I'm not opposed to removing it completely in the E Upgrade if that's the easiest path, but it's worth evaluating the performance of aggregating a signature from >= 67% of the primary network before making that decision.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Backlog 🗄️
Development

No branches or pull requests

4 participants