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

README.md: better usage instructions #67

Merged
merged 2 commits into from
Dec 22, 2020
Merged

Conversation

Nowaker
Copy link
Contributor

@Nowaker Nowaker commented Dec 16, 2020

No description provided.

README.md Outdated Show resolved Hide resolved
@mbag
Copy link
Collaborator

mbag commented Dec 22, 2020

hi @Nowaker I commented about adding note that the syntax in README is for terraform v0.13 or higher. Once that is added I'll merge the PR. I don't expect someone to be using terraform <=0.12 but just so that it's clear with which version the syntax used works.

@Nowaker
Copy link
Contributor Author

Nowaker commented Dec 22, 2020

Done!

@c33s
Copy link
Contributor

c33s commented Dec 22, 2020

@mbag why the drop of older terraform versions? the provider runs quite fine with 0.11. and everything over 0.11 has a security impact for us. see hashicorp/terraform#26141

edit: oh i think i missed that you only wrote about syntax. i feared you merge something which actually break something on older terraform versions. and yes there are still old version out there :)

@mbag
Copy link
Collaborator

mbag commented Dec 22, 2020

@c33s I'm actually not sure if older versions will be able to fetch newer releases directly from the Registry. I think that older versions are fetching providers from different URLs than registry.terraform.io.

@mbag mbag merged commit 8fd0774 into pan-net:master Dec 22, 2020
@c33s
Copy link
Contributor

c33s commented Dec 22, 2020

@mbag it works, i manually downloaded the powerdns provider yesterday (updated from 1.3 to 1.4 with terraform 0.11) works like charm. having only the problems where i opened issues for. so 1.4 is 0.11 compatible

@mbag
Copy link
Collaborator

mbag commented Dec 22, 2020

@c33s manual download and placing provider in the search path of the terraform should work of course, that's why I wrote directly from Registry. Because Hashicorp has discontinued old way of releasing and only support publishing providers on the Registry.

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