Skip to content

Conversation

jmt-lab
Copy link
Contributor

@jmt-lab jmt-lab commented Sep 15, 2025

Issue number: #574

Closes #574

Description of changes:
Enable use of tags in amispec.toml in pubsys

Testing done:

  • Created a bottlerocket build and then created a amispec.toml with:
[tags]
tag0 = "foo"
tag1 = "bar"
  • Ran pubsys publish ami through cargo make
  • us-west-2 ami has tag0 = "foo", tag1 = "bar"
  • us-east-1 ami has global tag0 = "foo", tag1 = "bar"

Terms of contribution:

By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.

@jmt-lab
Copy link
Contributor Author

jmt-lab commented Sep 15, 2025

Continued from #577 which got accidentally canceled cause of incorrect git command

@jmt-lab
Copy link
Contributor Author

jmt-lab commented Sep 15, 2025

Sorry this isn't ready yet i need to add the validation logic to validate-ami

@jmt-lab
Copy link
Contributor Author

jmt-lab commented Sep 15, 2025

Actually it looks like we do not do any validation of anything amispec.toml related in validate-ami aso unsure if we should change it

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.

Twoliter pubsys should be able to tag AMIs after publish/copy

1 participant