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

Define Internal Partners set in the root block finalization for validation #6483

Closed
zhangchiqing opened this issue Sep 19, 2024 · 1 comment
Labels
Stale Label used when marking an issue stale.

Comments

@zhangchiqing
Copy link
Member

Context

Addressing review comments

The root block creation bootstrap command uses information from both internal nodes and partner nodes to create the root block. Internal node information is retrieved locally, while partner node information is obtained from the identity table state.

During a spork, data from some partners (Dapper nodes) are treated as internal nodes for root block creation. This results in an overlap between internal and partner nodes, causing incorrect calculations for the percentage of partner collection nodes and leading to validation failures.

We addressed this by identifying the overlap between internal and partner nodes, treating the overlapping nodes as internal nodes.

However, this approach automatically accepts the overlap without review or validation, introducing potential risks.

A better solution would be to specify the list of internal partners in a file, using it to validate the overlap between internal and partner nodes. Overlaps should only be accepted if they match the specified list, with exceptions thrown for mismatches.

This approach not only provides a clearer definition of internal partners in the code but also helps prevent errors.

Copy link
Contributor

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the Stale Label used when marking an issue stale. label Dec 19, 2024
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Dec 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Stale Label used when marking an issue stale.
Projects
None yet
Development

No branches or pull requests

1 participant