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

amazon: Enhance AMI name and tags #405

Merged
merged 3 commits into from
Jul 14, 2021
Merged

Conversation

Itxaka
Copy link
Contributor

@Itxaka Itxaka commented Jul 13, 2021

This patch adds the github.sha to the build name of the AMI so we can
easily identify where the image came from if pushed from CI and to be
able to upload more than one image per day on CI as well.

Also adds it to the tags, and add a new Flavor tag, so we can use data
lifecycle policies to auto remove images and search for an X flavor by
using those tags.

The default value for no git_sha is none.

Signed-off-by: Itxaka [email protected]

@Itxaka
Copy link
Contributor Author

Itxaka commented Jul 13, 2021

This probably breaks #399 so dont merge it before that one is merge, this one is simpler to rebase.

Copy link
Contributor

@davidcassany davidcassany left a comment

Choose a reason for hiding this comment

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

LGTM

This patch adds the github.sha to the build name of the AMI so we can
easily identify where the image came from if pushed from CI and to be
able to upload more than one image per day on CI as well.

Also adds it to the tags, and add a new Flavor tag, so we can use data
lifecycle policies to auto remove images and search for an X flavor by
using those tags.

Also adds that tag to the aws_upload.sh script to tag the recovery
images as "recovery"

The default value for no git_sha is none.

Signed-off-by: Itxaka <[email protected]>
@Itxaka
Copy link
Contributor Author

Itxaka commented Jul 13, 2021

rebased and reworked a bit on adding the same tags to the aws_upload.sh script

@mudler mudler merged commit 822937c into rancher:master Jul 14, 2021
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.

3 participants