potiuk commented on PR #30352: URL: https://github.com/apache/airflow/pull/30352#issuecomment-1523716169
> @potiuk I have added some docs to describe sentinel_kwargs, I am not sure if `version_added` is appropriate. Thanks for your time~ It should be 2.7.0 - we do not add features in patchlevel versions. Also - one small request, can you mention that Redis sentinel is supported in https://airflow.apache.org/docs/apache-airflow/stable/core-concepts/executor/celery.html and maybe add an example configuration for it there? That's what I was looking for as "discoverabiliyt" by having just parameter decribing kwargs, it's not obvious that sentinel is supported and if we describe it in celery docs with example it might be much easier for people to find out -- 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]
