Merge lp://staging/~jelmer/loggerhead/health into lp://staging/loggerhead
Proposed by
Jelmer Vernooij
| Status: | Merged |
|---|---|
| Approved by: | Colin Watson |
| Approved revision: | 520 |
| Merge reported by: | Otto Co-Pilot |
| Merged at revision: | not available |
| Proposed branch: | lp://staging/~jelmer/loggerhead/health |
| Merge into: | lp://staging/loggerhead |
| Diff against target: |
34 lines (+8/-1) 2 files modified
loggerhead/apps/__init__.py (+5/-0) loggerhead/apps/branch.py (+3/-1) |
| To merge this branch: | bzr merge lp://staging/~jelmer/loggerhead/health |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Colin Watson (community) | Approve | ||
|
Review via email:
|
|||
Commit message
Add a /health target.
Description of the change
Add a /health target.
This is convenient for health checking in e.g. kubernetes.
At the moment it just returns okay, but it could return something else in the future.
To post a comment you must log in.
Revision history for this message
| Colin Watson (cjwatson) | # |
review:
Approve
Revision history for this message
| Otto Co-Pilot (otto-copilot) wrote | # |
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
A commit message must be set
https:/