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.

Sign up for GitHub

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

Jump to bottom

Variables to override service start retries and delay #51

Merged
merged 2 commits into from
Jan 2, 2023
Merged

Variables to override service start retries and delay #51

merged 2 commits into from
Jan 2, 2023

Conversation

rpelisse
Copy link
Contributor

@rpelisse rpelisse commented Dec 28, 2022 •
edited by guidograzioli
Loading

A small tweak, but I suspect many users we'll need to fine tune this depending on their env; new variables allow to tune the health check after a service start/restart:

Variable Default Description
keycloak_service_start_delay 10 Expected delay in seconds before the service is expected to be available after start.
keycloak_service_start_retries 25 How many times should Ansible retry to connect to the service after it was started, before failing

@rpelisse rpelisse added the minor_changes New parameters added to modules, or non-breaking behavior changes to existing parameters; no bugfix label Dec 28, 2022
@rpelisse rpelisse force-pushed the tweak_start_retries_and_delay branch from 4471935 to 9196b23 Compare December 29, 2022 11:41
guidograzioli
guidograzioli approved these changes Jan 2, 2023
View reviewed changes
Copy link
Member

@guidograzioli guidograzioli left a comment

Choose a reason for hiding this comment

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

👍 👍

@guidograzioli guidograzioli merged commit 73d7a46 into main Jan 2, 2023
@guidograzioli guidograzioli changed the title Allow overrides for service start retries and delay Variables to override service start retries and delay Jan 2, 2023
@guidograzioli guidograzioli deleted the tweak_start_retries_and_delay branch April 18, 2024 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@guidograzioli guidograzioli guidograzioli approved these changes

Assignees
No one assigned
Labels
minor_changes New parameters added to modules, or non-breaking behavior changes to existing parameters; no bugfix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rpelisse @guidograzioli