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

Increase default httpd recreate strategy timeout #187

Merged
merged 1 commit into from
Jul 25, 2017

Conversation

fbladilo
Copy link
Contributor

  • Increase default recreate strategy to 1200 (20 mins) from 600
  • Allows httpd to be more patient with frontend statefulset seeding

- Increase default recreate strategy to 1200 (20 mins) from 600
- Allows httpd to be more patient with frontend statefulset seeding
@fbladilo
Copy link
Contributor Author

@carbonin @bdunne Please review guys, more often than not, the httpd DC decides to error out as the frontend statefulset is still finishing seeding. The default strategy timeout is absolute, meaning that the deployment will error out and not even try again to restart/redeploy. The 20 minutes allows it to be more patient with frontend , thus allowing it to finish working or giving it another chance to retry finishing.

@fbladilo fbladilo requested a review from bdunne July 25, 2017 13:59
@fbladilo fbladilo requested a review from carbonin July 25, 2017 13:59
Copy link
Member

@bdunne bdunne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 This should help for now. In the longer term, we need to stop seeding all the time.

@bdunne bdunne merged commit 9be391a into ManageIQ:master Jul 25, 2017
@fbladilo
Copy link
Contributor Author

@bdunne Yes sir, I dream about it faster startups...

@bdunne bdunne added this to the Sprint 66 Ending Aug 7, 2017 milestone Jul 25, 2017
@bdunne bdunne self-assigned this Jul 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants