Skip to content

Update eks module, fix cluster id output#2

Merged
darkowlzz merged 1 commit intomainfrom
eks-update
Dec 13, 2022
Merged

Update eks module, fix cluster id output#2
darkowlzz merged 1 commit intomainfrom
eks-update

Conversation

@darkowlzz
Copy link
Copy Markdown
Contributor

  • Update terraform EKS module to use v19 and above.
  • EKS module v19 introduced breaking change for cluster_id output. Replace cluster_id with cluster_name in the test EKS module to handle this fix transparently for the consumers of the module.
  • Update cluster version to 1.23, current EKS default.

- Update terraform EKS module to use v19 and above.
- EKS module v19 introduced breaking change for `cluster_id` output.
  Replace `cluster_id` with `cluster_name` in the test EKS module to
  handle this fix transparently for the consumers of the module.
- Update cluster version to 1.23, current EKS default.

Signed-off-by: Sunny <darkowlzz@protonmail.com>
Copy link
Copy Markdown
Member

@pjbgf pjbgf 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 @darkowlzz! 🙇

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