-
Notifications
You must be signed in to change notification settings - Fork 80
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
fix(nodeSecurityGroupTags): only expose option through Cluster class #126
Conversation
908859e
to
2b5409b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
nodejs/eks/nodegroup.ts
Outdated
* | ||
* Note: If you want to tag this `nodeSecurityGroup`, you must manually | ||
* tag it during its external creation, before it is used in this node | ||
* group. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps phrase this as nodeSecurityGroup
and nodeSecurityGroupTags
are mutually exclusive? We could also test that at most one is set and error.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@lukehoban both the option comment, and the mutual exclusion check have been addressed. PTAL
2b5409b
to
aad56f0
Compare
aad56f0
to
c1cd2c3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Fixes #125.
Related #122.
Update for the new tag opts (based on orig in #122):
Cluster
class:.tags
- A map of tags that is applied to all AWS resources under management by apulumi/eks
cluster. Currently applied to:eksClusterSecurityGroup
,nodeSecurityGroup
, the Worker AutoScalingGroup (ASG), and cloudFormation stack..clusterSecurityGroupTags
- A map of tags applied to the EKS Cluster security group.nodeSecurityGroupTags
- A map of tags applied to the Worker Nodes security groupNodeGroup
class:.cloudFormationTags
- A map of tags applied to the CloudFormation Stack of Worker Nodes.autoScalingGroupTags
- A map of tags applied to the Worker NodeGroup ASG & InstancesnodeSecurityGroup
, you can tag it by manually setting the tags during its external creation, and before it is used in the NodeGroup.