-
Notifications
You must be signed in to change notification settings - Fork 16
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
Manifest example in provider's documentation is wrong. #94
Comments
Thanks for the submission! We'll take a look as soon as we're able. |
Hi @JfcAtCyberArk! Thank you for the submission! I'm having trouble reproducing what you're seeing. I'm also seeing our Jenkins tests passing, and they're configured with a Also, when I try the change you suggested, I see the following error:
So... I'm wondering if I'm testing with a different version of Terraform, or just not configuring things the same way that you are. One problem that I do see is that all of our example manifests in https://github.com/cyberark/terraform-provider-conjur repository don't include a
Some questions:
Thanks, |
Thanks for the explanation, that helps!!! It looks like several things need to be updated:
I would think that the main README.md can just have examples using the Terraform registry, and we can move the examples that use local binaries to CONTRIBUTING.md. Quick sanity check... Is this the best syntax for the manifests?:
|
This is still an issue that needs to be resolved. |
@infamousjoeg Will create a Jira issue for this and work it into sprint planning. |
Summary
Manifest example in provider's documentation is wrong.
Expected Results
The provider name should be "cyberark/conjur".
Actual Results (including error logs, if applicable)
The provider name is "conjur" and
terraform init
fails with the below:Reproducible
The text was updated successfully, but these errors were encountered: