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

Add wait_for_port and wait_for_log systemd unit logic #199

Merged
merged 1 commit into from
Apr 24, 2024

Conversation

guidograzioli
Copy link
Member

@guidograzioli guidograzioli commented Apr 23, 2024

New parameters allow to wait for healthy status in the systemd unit:

Variable Description Default
keycloak_quarkus_systemd_wait_for_port Whether systemd unit should wait for keycloak port before returning {{ keycloak_quarkus_ha_enabled }}
keycloak_quarkus_systemd_wait_for_log Whether systemd unit should wait for service to be up in logs false
keycloak_quarkus_systemd_wait_for_timeout How long to wait for service to be alive (seconds) 60
keycloak_quarkus_systemd_wait_for_delay Activation delay for service systemd unit (seconds) 10

Both keycloak_quarkus_systemd_wait_for_port and keycloak_quarkus_systemd_wait_for_log can be active at the same time, but mind they will be executed post-exec in serial, potentially adding up to the startup time.

Part of #182

@guidograzioli guidograzioli added the minor_changes New parameters added to modules, or non-breaking behavior changes to existing parameters; no bugfix label Apr 23, 2024
@guidograzioli guidograzioli changed the title Add wait_for systemd logic Add wait_for_port and wait_for_log systemd unit logic Apr 24, 2024
Copy link
Contributor

@rpelisse rpelisse left a comment

Choose a reason for hiding this comment

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

Misc comments, otherwise looks good to me.

roles/keycloak_quarkus/handlers/main.yml Show resolved Hide resolved
roles/keycloak_quarkus/tasks/restart.yml Show resolved Hide resolved
@rpelisse rpelisse merged commit 2925ea8 into ansible-middleware:main Apr 24, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor_changes New parameters added to modules, or non-breaking behavior changes to existing parameters; no bugfix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants