FAQ | This is a LIVE service | Changelog

Skip to content
Snippets Groups Projects

chore: remove /healthy endpoint comment that contained wrong information

Merged Sebastiaan ten Pas requested to merge fix-healthy-endpoint-comment into main

I think the comment that I removed in this MR was copied from https://gitlab.developers.cam.ac.uk/uis/devops/iam/deploy-identity/-/blob/master/staff_identity.tf#L107. I checked the /healthy endpoint in the ucam-observe remote server (https://gitlab.developers.cam.ac.uk/uis/devops/ucam-observe/ucam-observe-remote-server/-/blob/main/ucam_observe_remote_server/main.py?ref_type=heads#L118) and that one does not contain a check that the service can connect to the underlying database. So we can remove this comment to avoid confusion.

Edited by Sebastiaan ten Pas

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading