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

chore(distro): update eksctl package #3474

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

chore(distro): update eksctl package #3474

wants to merge 3 commits into from

Conversation

AustinAbro321
Copy link
Contributor

@AustinAbro321 AustinAbro321 commented Feb 6, 2025

Description

reopening #3473 outside of a fork to enable manual workflow triggers for EKS tests

Checklist before merging

a1994sc and others added 2 commits February 5, 2025 10:58
Copy link

netlify bot commented Feb 6, 2025

Deploy Preview for zarf-docs canceled.

Name Link
🔨 Latest commit d9af037
🔍 Latest deploy log https://app.netlify.com/sites/zarf-docs/deploys/67a4d5203123440008fe93c7

@AustinAbro321
Copy link
Contributor Author

@a1994sc
Copy link
Contributor

a1994sc commented Feb 6, 2025

Sorry, I forgot to update this line

./build/zarf package deploy build/zarf-package-distro-eks-amd64-0.0.3.tar.zst \
--components=deploy-eks-cluster \
--set=EKS_CLUSTER_NAME=${{ inputs.cluster_name || 'zarf-nightly-eks-e2e-test' }} \
--set=EKS_INSTANCE_TYPE=${{ inputs.instance_type || 't3.medium' }} \
--confirm

@AustinAbro321
Copy link
Contributor Author

@a1994sc If you don't mind, I can take over completing the update in this PR. It's hard to test using a fork since we only run EKS tests nightly and manually for time and money reasons

@a1994sc
Copy link
Contributor

a1994sc commented Feb 6, 2025

Please feel free

@AustinAbro321 AustinAbro321 changed the title chore(distro): update eksctl package (testing) chore(distro): update eksctl package Feb 6, 2025
Signed-off-by: Austin Abro <AustinAbro321@gmail.com>
Copy link

codecov bot commented Feb 6, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

see 1 file with indirect coverage changes

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