flvndh commented on a change in pull request #10664:
URL: https://github.com/apache/airflow/pull/10664#discussion_r484338059



##########
File path: chart/values.schema.json
##########
@@ -0,0 +1,1250 @@
+{
+    "$schema": "http://json-schema.org/draft-07/schema";,
+    "description": "Default values for airflow. Declare variables to be passed 
into your templates.",
+    "type": "object",
+    "properties": {
+        "uid": {
+            "description": "User of airflow user.",
+            "type": "integer"
+        },
+        "gid": {
+            "description": "Group of airflow user.",
+            "type": "integer"
+        },
+        "airflowHome": {
+            "description": "Airflow home directory. Used for mount paths.",
+            "type": "string"
+        },
+        "defaultAirflowRepository": {
+            "description": "Default airflow repository. Overrides all the 
specific images below.",
+            "type": "string"
+        },
+        "defaultAirflowTag": {
+            "description": "Default airflow tag to deploy.",
+            "type": "string"
+        },
+        "nodeSelector": {
+            "description": "Select certain nodes for airflow pods.",
+            "type": "object"

Review comment:
       You are right. The 
[doc](https://kubernetes.io/docs/concepts/scheduling-eviction/assign-pod-node/#nodeselector)
 for `nodeSelector` says:
   > `nodeSelector` is the simplest recommended form of node selection 
constraint. `nodeSelector` is a field of PodSpec. It specifies a map of 
key-value pairs. 




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


Reply via email to