Skip to content
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

Solr may take some time now to figure out Lock file #55

Merged
merged 3 commits into from
Sep 6, 2022

Conversation

nickumia-reisys
Copy link
Contributor

@nickumia-reisys nickumia-reisys commented Aug 12, 2022

Related to

With the new logic of waiting for the lock file to be deleted, the solr service might take some time to get ready...
- The default timeout for a terraform resource is 20 mins..
- https://www.terraform.io/plugin/sdkv2/resources/retries-and-customizable-timeouts#default-timeouts-and-deadline-exceeded-errors
- The default timeout for the ecs service resource is 10 mins..
~ - hashicorp/terraform-provider-aws#16012 (comment)

We are setting the timeout to be a little higher; while also deregistering the old instance faster (allowing for a quicker turnaround between restarts); we need the 'wait_for_steady_state' in order to setup the admin username/password properly

Quicken deregistration_delay

  • This will allow old ECS tasks to be deregistered from the load balancer quicker which will make restarts faster.

With the new logic of waiting for the lock file to be deleted, the solr service might take some time to get ready...
- The default timeout for a terraform resource is 20 mins..
  - https://www.terraform.io/plugin/sdkv2/resources/retries-and-customizable-timeouts#default-timeouts-and-deadline-exceeded-errors
- The default timeout for the ecs service resource is ~10 mins..
  - hashicorp/terraform-provider-aws#16012 (comment)

We are setting the timeout to be a little higher; while also deregistering the old instance faster (allowing for a quicker turnaround between restarts); we need the 'wait_for_steady_state' in order to setup the admin username/password properly
The target group needs to deregister the ecs task before it can be promptly restarted, so quicken that interval
@nickumia-reisys
Copy link
Contributor Author

@FuhuXia It looks like this isn't possible for this resource 😞

@nickumia-reisys nickumia-reisys merged commit bed2cbf into main Sep 6, 2022
@nickumia-reisys nickumia-reisys deleted the wait-for-solr-start branch September 6, 2022 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants