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
Add root volume tags for CF and TF targets #8693
Add root volume tags for CF and TF targets #8693
Conversation
faf8474
to
725dd1a
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.
I don't know Terraform/Cloudformation, so won't be able to lgtm.
upup/pkg/fi/cloudup/awstasks/launchtemplate_target_cloudformation.go
Outdated
Show resolved
Hide resolved
upup/pkg/fi/cloudup/awstasks/launchtemplate_target_cloudformation.go
Outdated
Show resolved
Hide resolved
upup/pkg/fi/cloudup/awstasks/launchtemplate_target_terraform.go
Outdated
Show resolved
Hide resolved
upup/pkg/fi/cloudup/awstasks/launchtemplate_target_terraform.go
Outdated
Show resolved
Hide resolved
upup/pkg/fi/cloudup/awstasks/launchtemplate_target_terraform.go
Outdated
Show resolved
Hide resolved
No problem. Thanks for checking the texts. |
1294d97
to
650df01
Compare
650df01
to
576e546
Compare
Co-Authored-By: John Gardiner Myers <jgmyers@proofpoint.com>
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: hakman, rifelpet The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest |
Adding root volume tags has been requested for a long time #3358 and was implemented in #8462 and #8466, but only for the
direct
target.This PR adds root volume tags for
terraform
andcloudformation
targets when using launch templates or mixed instances policies.Notes:
EnableLaunchTemplates
feature flag must be specified