Skip to content

feat: Added liveness and readyness probes - #4

Merged
adskyiproger merged 2 commits into
developfrom
probes
Apr 9, 2025
Merged

adskyiproger merged 2 commits into
developfrom
probes

Conversation

@adskyiproger

@adskyiproger adskyiproger commented Apr 9, 2025 •

Copy link
Copy Markdown
Contributor

This PR is part of bigger changes from issue opencrvs/opencrvs-core#9154

Goal of this PR is to supply liveness and readiness probes to kubernetes helm chart.

Identified issues

  • Gateway ping takes around 3 seconds
  • Few services don't have /ping endpoint:
    • config
    • countryconfig
    • events
  • /ping endpoint doesn't reflect real health of the service, E/g service might not be connected to database, but nodejs could be started.

More information on liveness and readiness probes can be found at https://kubernetes.io/docs/tasks/configure-pod-container/configure-liveness-readiness-startup-probes/#configure-probes

Testing

Application is deployed at: https://k8s.opencrvs.dev

Configuration can be checked at: https://console.cloud.google.com/kubernetes/deployment/europe-west1-b/gke-test/opencrvs-dev/auth/yaml/view?invt=AbuSDQ&project=opencrvs-on-k8s (auth service)

# tag: develop
tag: 50b3e14
tag: develop
# tag: 50b3e14

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this on purpose? What is the tag: 50b3e14 for exactly?

@adskyiproger
adskyiproger merged commit d47fbac into develop Apr 9, 2025
@adskyiproger
adskyiproger deleted the probes branch August 6, 2025 08:17
adskyiproger pushed a commit that referenced this pull request Apr 10, 2026
adskyiproger pushed a commit that referenced this pull request May 18, 2026
add reference data related secrets and mappings
adskyiproger pushed a commit that referenced this pull request Aug 3, 2026
feat: run e2e directly from opencrvs-core dispatch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants