jscheffl commented on code in PR #63187: URL: https://github.com/apache/airflow/pull/63187#discussion_r2908094402
########## chart/values.yaml: ########## @@ -1716,9 +1716,9 @@ migrateDatabaseJob: apiServer: enabled: true - # Number of Airflow API servers in the deployment - # This setting is ignored when HPA (Horizontal Pod Autoscaler) is enabled, - # as HPA will automatically manage the number of replicas based on the configured metrics. + # Number of Airflow API servers in the deployment. + # Set to ~ (null) when HPA is enabled so the replicas field is omitted from the Deployment, Review Comment: Just one remark, in other to make it consistent you also need to adjust `chart/values.schema.json` as the replicas field is marked as type "integer" there which makes it mandatory. So the values schema does not allow `null` 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]
