-
Notifications
You must be signed in to change notification settings - Fork 4
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
cluster and clusters data source #22
Conversation
docs/data-sources/clusters.md
Outdated
## Example Usage | ||
|
||
```terraform | ||
data "astronomer_deployments" "example_deployments" {} |
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.
should we change these to clusters?
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.
yep, accidentally had them switched in the examples folder
fixed this
type ClustersDataSource struct { | ||
Clusters types.List `tfsdk:"clusters"` | ||
CloudProvider types.String `tfsdk:"cloud_provider"` // query parameter | ||
Names types.List `tfsdk:"names"` // query parameter |
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.
Do we also need ClusterIds here?
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.
its not in v1beta1 yet but ill add it when/if it is implemented
Description
This PR creates the cluster data source and clusters data source and adds some acceptance integration tests.
🎟 Issue(s)
#14 #13
🧪 Functional Testing
Tested with a terraform project
Can see the examples in the
example
file📸 Screenshots
📋 Checklist
make test
before taking out of draftmake fmt
before taking out of draft