Skip to content
This repository has been archived by the owner on Jun 25, 2024. It is now read-only.

fix: remove prerelease versioning #1

Closed
wants to merge 1 commit into from
Closed

Conversation

adamconnelly
Copy link
Collaborator

More recent versions of packer compare the version number output by the plugin via the describe command with the filename, and if they don't match it fails. This has broken our AMI builds because the plugin reports a prerelease version (2.0.0-dev), but the filename version is just 2.0.0.

More recent versions of packer compare the version number output by the plugin via the `describe` command with the filename, and if they don't match it fails. This has broken our AMI builds because the plugin reports a prerelease version (`2.0.0-dev`), but the filename version is just `2.0.0`.
Copy link
Collaborator

@peterdeme peterdeme left a comment

Choose a reason for hiding this comment

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

i assume you'll bump the tag to 2.0.1 as well :)

@adamconnelly
Copy link
Collaborator Author

I'm closing this PR because it looks like the upstream repo now has support for filtering based on tags, so we can switch back instead of maintaining the fork.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants