You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by "ORuteMa (via GitHub)" <gi...@apache.org> on 2023/05/29 15:08:18 UTC

[GitHub] [airflow] ORuteMa commented on pull request #31028: Include rediss to the list of supported URL schemes

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

   > Hello @hussein-awala,
   > 
   > Thank you for pointing at this. CI allows to merge the pr though (hm). Actually the failure is from the pr [30352](https://github.com/apache/airflow/pull/30352) by @ORuteMa <img alt="Screenshot 2023-05-26 at 12 34 59 PM" width="777" src="https://user-images.githubusercontent.com/94494788/241308281-c6142326-826b-467b-9a57-ede3f08f8d1c.png">
   > 
   > TBH I don't understand why we create a new variable `broker_transport_options_for_celery` there
   > 
   > @uranusjr take a look, please
   > 
   > Thanks
   Hello, it used in the situation that you have sentinel and redis server both password-authed.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org