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 code server healthcheck #376

Merged
merged 3 commits into from
Feb 7, 2025
Merged

Add code server healthcheck #376

merged 3 commits into from
Feb 7, 2025

Conversation

joecorall
Copy link
Contributor

@joecorall joecorall commented Feb 7, 2025

Since the image base is drupal, the code server container had a healthcheck from nginx (and was failing). Instead, we should hit the /healthz endpoint in the codeserver.

@joecorall joecorall force-pushed the code-server-healthcheck branch from b77f11e to a5ee02f Compare February 7, 2025 13:07
@joecorall joecorall force-pushed the code-server-healthcheck branch from a5ee02f to 676339d Compare February 7, 2025 13:08
@joecorall
Copy link
Contributor Author

joecorall commented Feb 7, 2025

I am going to go ahead and merge this since the main tag currently sourced from the main branch will cause dev environments to break with code server having a bad healthcheck (that causes traefik to drop the codeserver)

Glad we have end to end nightly tests to catch these sorts of errors :). Was alerted in slack when the nightly isle-site-template build failed https://github.com/Islandora-Devops/isle-site-template/actions/runs/13198877336

@joecorall joecorall merged commit 50fa27d into main Feb 7, 2025
77 checks passed
@joecorall joecorall deleted the code-server-healthcheck branch February 7, 2025 13:24
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.

1 participant