schnie commented on a change in pull request #8777:
URL: https://github.com/apache/airflow/pull/8777#discussion_r429394334
##########
File path: chart/values.yaml
##########
@@ -0,0 +1,438 @@
+# Default values for airflow.
+# This is a YAML-formatted file.
+# Declare variables to be passed into your templates.
+
+# User and group of airflow user
+uid: 100
+gid: 101
+
+# Select certain nodes for airflow pods.
+nodeSelector: {}
+affinity: {}
+tolerations: []
+
+# Add common labels to all objects and pods defined in this chart.
+labels: {}
+
+# Ingress configuration
+ingress:
+ # Enable ingress resource
+ enabled: false
+
+ # Enable for cert-manager or kube-lego
+ acme: false
+
+ # Name of tls secret to use on ingress
+ tlsSecretName: ~
+
+ # Annotations always injected when configuring webserver Ingress ojbect
+ webserverAnnotations: {}
+
+ # Annotations always injected when configuring Flower Ingress object
+ flowerAnnotations: {}
+
+ # Base domain for ingress vhosts
+ baseDomain: ~
+
+# Network policy configuration
+networkPolicies:
+ # Enabled network policies
+ enabled: false
+
+# Airflow home directory
+# Used for mount paths
+airflowHome: "/usr/local/airflow"
+
+# Extra annotations to apply to all
+# Airflow pods
+airflowPodAnnotations: {}
+
+# Enable RBAC (default on most clusters these days)
+rbacEnabled: true
+
+# Airflow executor
+# Options: SequentialExecutor, LocalExecutor, CeleryExecutor,
KubernetesExecutor
+executor: "KubernetesExecutor"
+
+# If this is true and using
LocalExecutor/SequentialExecutor/KubernetesExecutor, the scheudler's
+# service account will have access to communicate with the api-server and
launch pods.
+# If this is true and using the CeleryExecutor, the workers will be able to
launch pods.
+allowPodLaunching: true
+
+# Default airflow repository
+defaultAirflowRepository: astronomerinc/ap-airflow
+
+# Default airflow tag to deploy
+defaultAirflowTag: 1.10.7-alpine3.10
+
+# Images
+images:
+ airflow:
+ repository: astronomerinc/ap-airflow
+ tag: ~
+ pullPolicy: IfNotPresent
+ flower:
+ repository: astronomerinc/ap-airflow
Review comment:
Great point. @dimberman has some experience with this. I believe the
intention is to follow up this PR with a better integration into the official
image as well as Airflow 2.0. Any thoughts on this Daniel?
----------------------------------------------------------------
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]
