-
Notifications
You must be signed in to change notification settings - Fork 13
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
Feat: add data source for getting a list of Cloud Credentials #375
Conversation
"credential_type": { | ||
Type: schema.TypeString, | ||
Description: "the type of cloud credential to filter by " + allowedCredentialTypesStr, | ||
Optional: true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
setting to optional incases where someone (for some reason) wants to get all the credentials regardless of type.
required_providers { | ||
env0 = { | ||
source = "terraform-registry.env0.com/env0/env0" | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this a new thing or am I wrong? I mean, shouldn't the integration tests use the local version and not the version from the public registry?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes. This is my bad.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@yaronya - actually looking at it again. This is not new. All tests are like this.
I guess it's somehow working as expected...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not the real registry (terraform-registry.env0.com
doesn't exist).
This is set up in buildFakeTerraformRegistry
in harness.go
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Issue & Steps to Reproduce / Feature Request
resolves #358
Solution
Added the new data source.
Added acceptance tests.
Added integration tests.
Added example.