You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/01/05 22:30:23 UTC

[GitHub] [airflow] ashb commented on a change in pull request #12634: Allow customization of probes path and host

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:
users@infra.apache.org