Skip to content

add validator script #519

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

Merged
merged 1 commit into from
Jun 13, 2025
Merged

add validator script #519

merged 1 commit into from
Jun 13, 2025

Conversation

oliviassss
Copy link
Contributor

Issue #, if available:

Description of changes:
Add a validator script to check

  1. aws-node vpc cni version is >= 1.19.4
  2. aws-node container env vars
  3. VPC subnets have /12 CIDR with prefix reservation

Tested in dev cluster

% ./eks_cluster_validator.sh 
Checking aws-node VPC CNI version...
[PASS] aws-node VPC CNI version 1.19.5 is >= 1.19.4
Checking aws-node container environment variables...
[FAIL] DISABLE_LEAKED_ENI_CLEANUP is not set to 'true'. Current value: 
[PASS] ENABLE_PREFIX_DELEGATION is set to 'true'
[PASS] MINIMUM_IP_TARGET is set to '30'
[PASS] WARM_IP_TARGET is set to '5'
Checking VPC subnets for /12 CIDR blocks and prefix delegation...
[WARN] Subnet subnet-0f358dc6e17a750e5 has /18 CIDR block (not /12)
[WARN] Subnet subnet-0de5770ba205ed3cf has /18 CIDR block (not /12)
[WARN] Subnet subnet-0cd7407b3ff938260 has /18 CIDR block (not /12)
[FAIL] No subnets with /12 CIDR blocks and prefix delegation found
Validation FAILED. Please address the issues highlighted above.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@hakuna-matatah hakuna-matatah merged commit 9bd4fec into awslabs:main Jun 13, 2025
4 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.

2 participants