dzhigimont commented on code in PR #30603:
URL: https://github.com/apache/airflow/pull/30603#discussion_r1166502205


##########
chart/values.schema.json:
##########
@@ -1864,6 +1864,14 @@
                                         "string"
                                     ],
                                     "default": 1
+                                },
+                                "minAvailable": {
+                                    "description": "Min available pods for 
scheduler.",
+                                    "type": [
+                                        "integer",
+                                        "string"
+                                    ],
+                                    "default": 1

Review Comment:
   Helm doesn't take the default value into account at all, but I followed the 
style in the file. I think it can help developers see an example of possible 
value.  Could you explain why we can set default values in the schema only for 
specified properties in the values.yaml which already have explicit values?



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to