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

config: check tracked subnets does not exceed the limit #405

Merged
merged 1 commit into from
Aug 2, 2024

Conversation

najeal
Copy link
Contributor

@najeal najeal commented Aug 1, 2024

Why this should be merged

Closes #399

How this works

This checks that the provided subnets does not exceed the limit (16) and returns error during config validation when exceeding.

How this was tested

unit test has been added

How is this documented

Copy link
Collaborator

@cam-schultz cam-schultz left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for putting this together 🙏

Copy link
Contributor

@iansuvak iansuvak left a comment

Choose a reason for hiding this comment

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

👍

@cam-schultz cam-schultz merged commit b40a3c8 into ava-labs:main Aug 2, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enforce a limit of 16 tracked subnets
3 participants