Skip to content

Conversation

bryantbiggs
Copy link
Contributor

Description

  • Update ML patterns to use latest Terraform modules and AWS/Helm provider versions
    • Latest Helm provider will accept patterns and map to authorization tokens which is a UX improvement
    • Latest AWS provider supports region argument which means the aliased 2nd AWS provider for gathering the Public ECR token is no longer required
    • The defaults for the latest EKS module are "better", allowing users to specify less for the intended behavior; there were a few name changes to reduce redundancy (i.e. cluster_* was dropped to shorten names)

Motivation and Context

  • Ensure folks are seeing the latest to avoid asking questions/raising issues on prior versions that are no longer supported

How was this change tested?

  • Yes, I have tested the PR using my local account setup (Provide any test evidence report under Additional Notes)

    ** I have not - GPUs are expensive 😬. However, changes should work (in theory) as advertised

  • Yes, I have updated the docs for this feature

  • Yes, I ran pre-commit run -a with this PR

Additional Notes

@bryantbiggs bryantbiggs requested a review from a team as a code owner September 12, 2025 18:30
Copy link
Contributor

@askulkarni2 askulkarni2 left a comment

Choose a reason for hiding this comment

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

Thank you @bryantbiggs !! Much appreciated!

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