search

nbering / terraform-provider-ansible

"Logical" provider for integrating with an Ansible Dynamic Inventory script.
https://nbering.github.io/terraform-provider-ansible/
Mozilla Public License 2.0
329 stars 64 forks source link

Release script creates binary by invalid name. #2

Closed nbering closed 6 years ago

nbering commented 6 years ago

The plugin filename convention was inaccurately documented by Terraform, so the "helpful" file name I used for the release binary filenames is incorrect. The dash(-) between the plugin name and the version number should be an underscore(_).

https://github.com/hashicorp/terraform/issues/15705

imchairmanm commented 6 years ago

Thanks for opening this! I had a bit of trouble trying to figure out why Terraform was unable to find the plugin. I didn't think to look at the binary name.

Also, just wanted to say that this approach has been working really well for me. Thank you for providing the tools and for writing up the blog post on them!

nbering commented 6 years ago

Thanks @imchairmanm! There's a fair bit of traffic coming to the blog post and Repo. I have a little vacation time coming up, so I may do some further clean-up and documentation while I have the time.

Clearing up this issue would be included in that effort.

nbering commented 6 years ago

Re-opening until I get a new release builds attached to the GitHub release.

nbering commented 6 years ago

Closing again... should be fixed in archives attached to v0.0.2.

v0.0.2 is functionally equivalent to v0.0.1, except that the release build process has been improved.