potiuk commented on pull request #16404:
URL: https://github.com/apache/airflow/pull/16404#issuecomment-861789074


   > > I believe the conclusion of our discussion was that in case of 
`get_config` use where Secrets Backend is missing-in-action we should raise an 
Exception. I gues It should be added here (including tests?)
   > 
   > @potiuk For `get_config` currently it causes airflow to crash if it cannot 
reach the Secrets Backend. This happens when `get_config` is called. If I 
understood properly we considered this behavior as expected so I left it alone.
   
   But I think we should add a better message in this case and a test to cover 
this scenario explicitly. Then no-one will ever "fix it" thinking it is a bug.


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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to