ashb commented on a change in pull request #12634:
URL: https://github.com/apache/airflow/pull/12634#discussion_r552237800
##########
File path: chart/README.md
##########
@@ -208,10 +208,14 @@ The following tables lists the configurable parameters of
the Airflow chart and
| `scheduler.serviceAccountAnnotations` | Annotations to add
to scheduler kubernetes service account
| `{}` |
| `scheduler.extraVolumes` | Mount additional
volumes into scheduler
| `[]` |
| `scheduler.extraVolumeMounts` | Mount additional
volumes into scheduler
| `[]` |
+| `webserver.livenessProbe.path` | Webserver
LivenessProbe path (need to match your webserver base_url)
| `/health` |
+| `webserver.livenessProbe.host` | Webserver
LivenessProbe host header (need to match your webserver base_url)
| `~` |
| `webserver.livenessProbe.initialDelaySeconds` | Webserver
LivenessProbe initial delay
| `15` |
| `webserver.livenessProbe.timeoutSeconds` | Webserver
LivenessProbe timeout seconds
| `30` |
| `webserver.livenessProbe.failureThreshold` | Webserver
LivenessProbe failure threshold
| `20` |
| `webserver.livenessProbe.periodSeconds` | Webserver
LivenessProbe period seconds
| `5` |
+| `webserver.readinessProbe.path` | Webserver
ReadinessProbe path (need to match your webserver base_url)
| `/health` |
+| `webserver.readinessProbe.host` | Webserver
ReadinessProbe host header (need to match your webserver base_url)
| `~` |
Review comment:
Do we ever need different values here -- they are controlled by the same
airflow.cfg so these two values should always be the same, right?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
