Skip to content

Commit

Permalink
fix: increase timeout for Authzed DB health probes (#3800)
Browse files Browse the repository at this point in the history
The timeout in k8s was increasesd in a recent PR. But this increases the
timeout of the CLI we call in the Authzed DB image to actuall do the
health checks. And since this CLI had a default timeout of 1s it was
timing out even before the k8s timeout could take effect. We noticed
that Authzed on renkulab.io was in a restart loop because of this.
  • Loading branch information
olevski authored Oct 8, 2024
1 parent f792d99 commit 2cf4645
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 0 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.rst
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,7 @@ Internal Changes
- **Data services**: Order resource classes by GPU, CPU, RAM and storage
- **Data services**: Following redirects when sending requests to git repositories
- **Data services**: Allow unsetting secrets for cloud storage
- **Helm chart**: Increase the connection timeout for the Authzed database health checks

Individual Components
~~~~~~~~~~~~~~~~~~~~~
Expand Down
4 changes: 4 additions & 0 deletions helm-chart/renku/templates/authz/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -108,6 +108,8 @@ spec:
- -addr=127.0.0.1:50051
- -tls
- -tls-server-name={{ template "renku.fullname" . }}-authz
- -connect-timeout
- "3s"
timeoutSeconds: 3
periodSeconds: 10
readinessProbe:
Expand All @@ -118,6 +120,8 @@ spec:
- -addr=127.0.0.1:50051
- -tls
- -tls-server-name={{ template "renku.fullname" . }}-authz
- -connect-timeout
- "3s"
timeoutSeconds: 3
periodSeconds: 10
resources:
Expand Down

0 comments on commit 2cf4645

Please sign in to comment.