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

Current django health check in docker-compose.yml useless. #9917

Closed
afabiani opened this issue Aug 26, 2022 · 0 comments
Closed

Current django health check in docker-compose.yml useless. #9917

afabiani opened this issue Aug 26, 2022 · 0 comments
Assignees
Labels
3.3.x 4.0.x docker Issues specific to GeoNode docker or GeoNode SPC major A high priority issue which might affect a lot of people or large parts of the codebase master

Comments

@afabiani
Copy link
Member

References: GeoNode/geonode-project#363

In the docker-compose.yml a health for the django container is performed using a simple curl command inside in the container to check if the server is running:
test: "curl --fail --silent --write-out 'HTTP CODE : %{http_code}\n' --output /dev/null http://127.0.0.1:8000/"

But this isn't working anymore since in the default configuration an http-socket isn't configured, this results in curl exiting with code 52 as its getting an empty reply from the server.

This results in a permanent unhealthy status of the container and is misleading.

A solution would be to use uswgi_curl from the uwsi-tools package (pip). It does not offer the same options as curl but results in a correct exit code (0 working, 1 not working).

@afabiani afabiani self-assigned this Aug 26, 2022
@afabiani afabiani added master 4.0.x 3.3.x major A high priority issue which might affect a lot of people or large parts of the codebase docker Issues specific to GeoNode docker or GeoNode SPC labels Aug 26, 2022
afabiani added a commit that referenced this issue Aug 29, 2022
…ss. (#9918) (#9928)

Co-authored-by: Alessio Fabiani <alessio.fabiani@geosolutionsgroup.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.3.x 4.0.x docker Issues specific to GeoNode docker or GeoNode SPC major A high priority issue which might affect a lot of people or large parts of the codebase master
Projects
None yet
Development

No branches or pull requests

1 participant