ORuteMa commented on PR #30352:
URL: https://github.com/apache/airflow/pull/30352#issuecomment-1529601574

   > > @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
   
   Sure. I will add a example for sentinel. 


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