Skip to content

Commit

Permalink
fix(chart): set ttlSecondsAfterFinished on certgen job to 30 by default
Browse files Browse the repository at this point in the history
This is painless workaround for ArgoCD bug:
argoproj/argo-cd#6880

Signed-off-by: JuniorJPDJ <git@juniorjpdj.pl>
  • Loading branch information
JuniorJPDJ committed Apr 15, 2024
1 parent 9f00be3 commit 98f4b62
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion charts/gateway-helm/values.tmpl.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ certgen:
job:
annotations: {}
resources: {}
ttlSecondsAfterFinished: 0
ttlSecondsAfterFinished: 30
rbac:
annotations: {}
labels: {}
2 changes: 1 addition & 1 deletion site/content/en/latest/install/api.md
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ The Helm chart for Envoy Gateway
|-----|------|---------|-------------|
| certgen.job.annotations | object | `{}` | |
| certgen.job.resources | object | `{}` | |
| certgen.job.ttlSecondsAfterFinished | int | `0` | |
| certgen.job.ttlSecondsAfterFinished | int | `30` | |
| certgen.rbac.annotations | object | `{}` | |
| certgen.rbac.labels | object | `{}` | |
| config.envoyGateway.gateway.controllerName | string | `"gateway.envoyproxy.io/gatewayclass-controller"` | |
Expand Down
2 changes: 1 addition & 1 deletion test/helm/default.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -524,4 +524,4 @@ spec:
runAsNonRoot: true
runAsUser: 65534
serviceAccountName: eg-gateway-helm-certgen
ttlSecondsAfterFinished: 0
ttlSecondsAfterFinished: 30

0 comments on commit 98f4b62

Please sign in to comment.