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

Adding support for existing cluster #21

Merged
merged 8 commits into from
Nov 22, 2024
Merged

Conversation

dunefro
Copy link
Contributor

@dunefro dunefro commented Nov 21, 2024

  1. Add support for existing cluster using variable use_existing_cluster.
  2. Following variables are introduced
use_existing_cluster
existing_cluster_node_role_arn
existing_cluster_node_security_group_id
existing_cluster_oidc_issuer_arn
existing_cluster_oidc_issuer_url
  1. moved the eks and eks-addons to a count block. This will cause a drift which should be checked in under the migration-guide.md
  2. Introduced a scripts folder to ensure better practices for upgrades or any additional use.

@dunefro dunefro self-assigned this Nov 21, 2024
output.tf Outdated Show resolved Hide resolved
DeeAjayi
DeeAjayi previously approved these changes Nov 21, 2024
locals.tf Outdated Show resolved Hide resolved
locals.tf Outdated Show resolved Hide resolved
shubhamrai1993
shubhamrai1993 previously approved these changes Nov 22, 2024
@dunefro dunefro merged commit 4d46c64 into main Nov 22, 2024
6 checks passed
@dunefro dunefro deleted the existing-cluster-support branch November 22, 2024 06:29
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.

3 participants