-
Notifications
You must be signed in to change notification settings - Fork 166
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: new mongodbatlas_stream_instances
data source
#1701
Conversation
@@ -59,7 +59,7 @@ jobs: | |||
- 'internal/service/cluster/*.go' | |||
search_deployment: | |||
- 'internal/service/searchdeployment/*.go' | |||
stream_instance: | |||
stream: |
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.
transitioned to a stream
CI group to include the coming stream_connection resource in here as well. Both define acceptance tests that run fast.
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.
good
@@ -6,6 +6,8 @@ import ( | |||
"github.com/hashicorp/terraform-plugin-framework/diag" | |||
"github.com/hashicorp/terraform-plugin-framework/types" | |||
"github.com/hashicorp/terraform-plugin-framework/types/basetypes" | |||
"github.com/hashicorp/terraform-plugin-sdk/v2/helper/id" |
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.
does it need v2 being a new resource?
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 a helper function provided by hashicorp is for generating a unique id, did not find a framework alternative but nevertheless we are not coupling to sdk v2 logic, this is simply for a obtaining a random id.
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.
Couple of nits, LGTM.
84b837e
to
c059a6f
Compare
64389dd
to
bf0fb3f
Compare
c059a6f
to
a5f3669
Compare
bf0fb3f
to
c6b0819
Compare
|
Description
Includes:
Type of change:
Required Checklist:
Further comments