-
Notifications
You must be signed in to change notification settings - Fork 190
feat(search deployments): add no_wait_for_state_transition parameter to skip reindexing wait #3109
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
This PR has gone 7 days without any activity and meets the project’s definition of "stale". This will be auto-closed if there is no new activity over the next 7 days. If the issue is still relevant and active, you can simply comment with a "bump" to keep it open, or add the label "not_stale". Thanks for keeping our repository healthy! |
Thank you @ridmagroupby for the PR. We are discussing the it with the team and will get back to you soon. |
This PR has gone 7 days without any activity and meets the project’s definition of "stale". This will be auto-closed if there is no new activity over the next 7 days. If the issue is still relevant and active, you can simply comment with a "bump" to keep it open, or add the label "not_stale". Thanks for keeping our repository healthy! |
bump |
This PR has gone 7 days without any activity and meets the project’s definition of "stale". This will be auto-closed if there is no new activity over the next 7 days. If the issue is still relevant and active, you can simply comment with a "bump" to keep it open, or add the label "not_stale". Thanks for keeping our repository healthy! |
Re-opening this issue while we still discuss internally. Apologies for further waiting on a decision here. |
This PR has gone 7 days without any activity and meets the project’s definition of "stale". This will be auto-closed if there is no new activity over the next 7 days. If the issue is still relevant and active, you can simply comment with a "bump" to keep it open, or add the label "not_stale". Thanks for keeping our repository healthy! |
bump |
This PR has gone 7 days without any activity and meets the project’s definition of "stale". This will be auto-closed if there is no new activity over the next 7 days. If the issue is still relevant and active, you can simply comment with a "bump" to keep it open, or add the label "not_stale". Thanks for keeping our repository healthy! |
Hi @ridmagroupby, once again thank you for your patience. Our team has reached a decision and we are going to move forward with this proposal. We'll make few changes to your proposal, such as changing the name of the field and add some clarity on the documentation. We want to make sure the usage of the field will be intuitive for everyone and be clear on "what it means" when customers use it. Let me know if you have any questions, and thank you for your contributions. |
@marcosuma @EspenAlbert thanks a lot for considering it. Please keep me in the loop if possible and lemme know if I can be any help further. I guess thats it for this PR then . Cheers thanks a lot !!! |
This PR has gone 7 days without any activity and meets the project’s definition of "stale". This will be auto-closed if there is no new activity over the next 7 days. If the issue is still relevant and active, you can simply comment with a "bump" to keep it open, or add the label "not_stale". Thanks for keeping our repository healthy! |
Merged in #3237 Thank you @ridmagroupby |
Hi @ridmagroupby thank you once again for your PR and patience. Your change has been included with #3237 like Espen said. This should be out by the end of the week. |
Description
This PR introduces a new optional parameter,
no_wait_for_state_transition
, to themongodbatlas_search_deployment
resource. This new feature allows Terraform to update search nodes without waiting for all reindexing operations to complete.When creating search nodes, the default behavior is acceptable; however, when modifying search nodes that already have many indexes, waiting for the complete reindexing process can take a long time and incur significant pipeline costs. With this change, if
no_wait_for_state_transition
is set totrue
, Terraform will trigger the update and return immediately, allowing reindexing to continue in the background. The default value remainsfalse
to ensure backward compatibility.Link to any related issue(s): HELP-71754
Type of change
Required Checklist
make fmt
and formatted my code.Further comments
No further comments.