-
Notifications
You must be signed in to change notification settings - Fork 189
INTMDB-436: Update privatelink endpoint service resources timeout config #1329
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
Conversation
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.
Let's wait to release 1.10.2 before merging this PR
...dbatlas/resource_mongodbatlas_privatelink_endpoint_service_data_federation_online_archive.go
Show resolved
Hide resolved
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.
LGTM
@@ -182,7 +183,7 @@ func resourceMongoDBAtlasPrivateLinkEndpointServerlessDelete(ctx context.Context | |||
Refresh: resourcePrivateLinkEndpointServerlessRefreshFunc(ctx, conn, projectID, instanceName, endpointID), | |||
Timeout: d.Timeout(schema.TimeoutDelete), | |||
MinTimeout: 5 * time.Second, | |||
Delay: 3 * time.Second, | |||
Delay: 5 * time.Second, |
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.
just curious, why 5 seconds delay?
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.
@Zuhairahmed just making it consistent across privatelink_endpoint resources.
Description
Enable configurable timeout, and make consistent 2 hour defaulting timeout across dedicated, serverless, and privatelink_endpoint_service_data_federation_online_archive
Link to any related issue(s): https://jira.mongodb.org/browse/INTMDB-436
Type of change:
Required Checklist:
Further comments