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

feat(nodegroup): use the latest recommended AMIs from the SSM store #366

Merged
merged 1 commit into from Apr 6, 2020

Conversation

metral
Copy link
Contributor

@metral metral commented Apr 2, 2020

Proposed changes

feat(nodegroup): use the latest recommended AMIs from the SSM store

  • Default to the latest recommended AMI IDs from the AWS SSM Parameter Store.
  • Add new NodeGroupOption: gpu to use the latest recommended EKS
    Optimized Linux AMI with GPU support.

See for more details:

Related issues (optional)

Closes #258
Closes #354

@metral metral force-pushed the metral/ssm-ami-ids branch 3 times, most recently from baa5711 to 0c9f9bd Compare April 2, 2020 19:35
nodejs/eks/cluster.ts Outdated Show resolved Hide resolved
nodejs/eks/cluster.ts Outdated Show resolved Hide resolved
nodejs/eks/nodegroup.ts Outdated Show resolved Hide resolved
nodejs/eks/nodegroup.ts Outdated Show resolved Hide resolved
@lukehoban
Copy link
Member

Yes, this is being left in for backward compat for existing clusters that did not specify an amiId and relied on the filtered search.

But we apply ignoreChanges in that case, so they wouldn't see any updates anyway, right?

@metral
Copy link
Contributor Author

metral commented Apr 3, 2020

Feedback has been addressed. PTAL @lblackstone @lukehoban

@metral metral force-pushed the metral/ssm-ami-ids branch 2 times, most recently from e64cfab to 66fbb20 Compare April 3, 2020 22:16
@metral metral force-pushed the metral/ssm-ami-ids branch 4 times, most recently from fa954e1 to 5d2da74 Compare April 4, 2020 00:07
@metral metral force-pushed the metral/ssm-ami-ids branch 2 times, most recently from 1855a6f to 5d98053 Compare April 4, 2020 05:47
@metral metral changed the title feat(nodegroup): add opts to use the recommended AMIs from the SSM store feat(nodegroup): use the latest recommended AMIs from the SSM store Apr 4, 2020
@metral metral force-pushed the metral/ssm-ami-ids branch 2 times, most recently from 506b3cb to 3131886 Compare April 4, 2020 05:51
nodejs/eks/nodegroup.ts Outdated Show resolved Hide resolved
nodejs/eks/nodegroup.ts Outdated Show resolved Hide resolved
nodejs/eks/nodegroup.ts Show resolved Hide resolved
nodejs/eks/nodegroup.ts Outdated Show resolved Hide resolved
- Default to the latest recommended AMI IDs from the AWS SSM Parameter Store.
- Add new `NodeGroupOption`: `gpu` to use the latest recommended EKS
  Optimized Linux AMI with GPU support.

See for more details:
  - https://docs.aws.amazon.com/eks/latest/userguide/retrieve-ami-id.html
  - https://docs.aws.amazon.com/eks/latest/userguide/eks-optimized-ami.html
  - https://docs.aws.amazon.com/systems-manager/latest/userguide/systems-manager-parameter-store.html
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants