canonical / canonical/cos-coordinated-workers
nginx: enable health check at a different location than `/`
- Dominant language
- Python
- Stars
- 1
- Forks
- 2
- PR merge metrics
- No merged PRs in 30d
Description
### Enhancement Proposal
Currently, in https://github.com/canonical/cos-coordinated-workers/blob/main/src/coordinated_workers/nginx.py#L588, we set a "heath" endpoint on nginx at the root `/` location, which, IMO, is not ideal.
Perhaps we should put it in a `/health` and verify that this change doesn't break loki and mimir.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start in src/coordinated_workers/nginx.py around line 588, where the health endpoint is currently configured at /. Check how the nginx location is used by Loki and Mimir, then determine whether moving it to /health preserves their behavior. Done means the endpoint uses the new location without breaking Loki or Mimir.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- nginx, python
- Domain
- infrastructure, observability
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 42/100