potiuk commented on a change in pull request #8393:
URL: https://github.com/apache/airflow/pull/8393#discussion_r412073486
##########
File path: tests/sensors/test_timeout_sensor.py
##########
@@ -72,6 +74,7 @@ def setUp(self):
}
self.dag = DAG(TEST_DAG_ID, default_args=args)
+ @pytest.mark.quarantined
Review comment:
Yes. I saw it failed. For now I want to make all the "suspicious" cases
and possibly have some false negatives". The idea is that I do not want to do
all the investigation myself but split it between committers so that everyone
will take care about some of the "flaky" tests, observe it's stability and move
it out of quarantine when it is stable (and move it back if not). I think all
or most committers should take part of the job on their shoulders if we wan to
keep stability for everyone.
##########
File path: tests/sensors/test_timeout_sensor.py
##########
@@ -72,6 +74,7 @@ def setUp(self):
}
self.dag = DAG(TEST_DAG_ID, default_args=args)
+ @pytest.mark.quarantined
Review comment:
Yes. I saw it failed. For now I want to make all the "suspicious" cases
and possibly have some "false negatives". The idea is that I do not want to do
all the investigation myself but split it between committers so that everyone
will take care about some of the "flaky" tests, observe it's stability and move
it out of quarantine when it is stable (and move it back if not). I think all
or most committers should take part of the job on their shoulders if we wan to
keep stability for everyone.
----------------------------------------------------------------
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]
