potiuk commented on code in PR #30352: URL: https://github.com/apache/airflow/pull/30352#discussion_r1186003224
########## airflow/config_templates/default_airflow.cfg: ########## @@ -1073,10 +1073,19 @@ worker_precheck = False # ETA you're planning to use. # visibility_timeout is only supported for Redis and SQS celery brokers. # See: -# http://docs.celeryproject.org/en/master/userguide/configuration.html#std:setting-broker_transport_options +# https://docs.celeryq.dev/en/stable/getting-started/backends-and-brokers/redis.html#visibility-timeout # Example: visibility_timeout = 21600 # visibility_timeout = +# The sentinel_kwargs parameter allows passing additional options to the Sentinel client. +# In a typical scenario where Redis Sentinel is used as the broker and Redis servers are Review Comment: The pre-commit prevents you doing it and modifes it and ask you to commit the changes it has done when it add. You need to: 1) add changes to ,yml 2) commit and let pre-commit add it to the config 3) add the generated config changes it to the staging 4) commit again -- 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]
