canonical / canonical/cos-coordinated-workers

nginx: enable health check at a different location than `/`

Open
#86 0 comments 1 reaction 0 assignees View on GitHub
Status: Triage Type: Enhancement
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.