Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

document liveness and readiness probes #2982

Merged
merged 4 commits into from
Apr 4, 2024

Conversation

gerardsn
Copy link
Member

@gerardsn gerardsn commented Mar 25, 2024

closes #2193

changed the contents of this PR to documentation only

Copy link
Member

@woutslakhorst woutslakhorst left a comment

Choose a reason for hiding this comment

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

I don't think this is the right way.

@gerardsn gerardsn changed the title switch healthcheck to /health endpoint document liveness and readiness probes Mar 27, 2024
docs/pages/deployment/monitoring.rst Outdated Show resolved Hide resolved
docs/pages/deployment/monitoring.rst Outdated Show resolved Hide resolved
@gerardsn gerardsn merged commit 1a84edf into master Apr 4, 2024
9 checks passed
@gerardsn gerardsn deleted the feature/use-health-endpoint-for-health-check branch April 4, 2024 08:38
reinkrul pushed a commit that referenced this pull request Apr 5, 2024
* switch healthcheck to /health endpoint

* Revert "switch healthcheck to /health endpoint"

This reverts commit d3bae8d.

* document liveness/readiness probes

* pr feedback
rolandgroen added a commit that referenced this pull request Apr 8, 2024
* master: (21 commits)
  IAM: Remove OpenID4VP PoC code (#3022)
  Bump golang.org/x/crypto from 0.21.0 to 0.22.0 (#3021)
  VDR: Support root did:web DIDs (#2900)
  extended policy mapping for user/organization (#2977)
  IAM: fix loading of server state (#3011)
  Bump github.com/nats-io/nats.go from 1.34.0 to 1.34.1 (#3018)
  Bump github.com/prometheus/client_model from 0.6.0 to 0.6.1 (#3017)
  Bump google.golang.org/grpc from 1.62.1 to 1.63.0 (#3016)
  Update golang.org/x/net@v0.23.0 (#3014)
  Ignore allowUntrustedIssuer for did:web and clarify 'revocation' in vcr/search (#2992)
  document liveness and readiness probes (#2982)
  IAM: Allow user details to be passed in requestUserAccessToken (#2990)
  Policy: allow multiple policy files in local PDP (#3010)
  Bump github.com/amacneil/dbmate/v2 from 2.13.0 to 2.14.0 (#2999)
  Docker: don't run as root user (#2917)
  Add missing copyright notices (#3000)
  Set SQL logging to TRACE instead of DEBUG (#3002)
  fix invalid swagger (#2988)
  add credentialStatus to common ssi_types (#2989)
  Auth: IAM client should not use PKI TLS config (#2998)
  ...
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.

Change default Dockerfile health check from /status to /health
2 participants