vincbeck commented on PR #31484:
URL: https://github.com/apache/airflow/pull/31484#issuecomment-1563226703

   > > It happened this morning to another of our system test that uses 
`RedshiftClusterSensor`. Somehow, Redshift takes more time now to spin clusters 
up. Thus, I updated all our system tests using RedshiftClusterSensor to 
set/increase the timeout.
   > 
   > Outside the scope of this PR: We should change the default value for that 
sensor, if the default is doomed to fail with a timeout then it's not a good 
user experience.
   
   The default value is actually the default value set by Airflow (1 week), see 
[here](https://github.com/apache/airflow/blob/main/airflow/sensors/base.py#L126).
 So no need to update the sensor, we were just setting too low timeouts in our 
system tests


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