[GitHub] [airflow] kaxil commented on a change in pull request #17743: Add Triggerer to Helm Chart

[GitBox <gi...@apache.org>]

kaxil commented on a change in pull request #17743:
URL: https://github.com/apache/airflow/pull/17743#discussion_r693272681



##########
File path: chart/values.yaml
##########
@@ -715,6 +715,113 @@ webserver:
             topologyKey: "kubernetes.io/hostname"
   tolerations: []
 
+# Airflow Triggerer Config
+triggerers:
+  # Number of airflow triggerers in StatefulSet
+  replicas: 1
+
+  # Command to use when running Airflow triggerers (templated).
+  command: ~
+  # Args to use when running Airflow triggerers (templated).
+  args:
+    - "bash"
+    - "-c"
+    # The format below is necessary to get `helm lint` happy
+    - |-
+      exec \
+      airflow triggerer
+
+  # Update Strategy when triggerers are deployed as a StatefulSet
+  updateStrategy: ~
+  # Update Strategy when triggerers are deployed as a Deployment
+  strategy:
+    rollingUpdate:
+      maxSurge: "100%"
+      maxUnavailable: "50%"
+
+  # Create ServiceAccount
+  serviceAccount:
+    # Specifies whether a ServiceAccount should be created
+    create: true
+    # The name of the ServiceAccount to use.
+    # If not set and create is true, a name is generated using the release name
+    name: ~
+
+    # Annotations to add to triggerer kubernetes service account.
+    annotations: {}
+
+  persistence:
+    # Enable persistent volumes
+    enabled: false

Review comment:
       I made it a deployment -- there isn't any state on Triggers




-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org