potiuk commented on code in PR #35160:
URL: https://github.com/apache/airflow/pull/35160#discussion_r1375471350


##########
tests/jobs/test_triggerer_job.py:
##########
@@ -110,6 +112,9 @@ def create_trigger_in_db(session, trigger, operator=None):
     return dag_model, run, trigger_orm, task_instance
 
 
+# Quarantined because this test is failing the first time it is run. The 
second time succeeds
+# Seems to be connected with SOME race condition and needs to be investigated
+@pytest.mark.quarantined

Review Comment:
   Well. not really.. There is some reason why we get empty string but I think 
the test has a bit different (combined) assertion:
   
   WHEN we receive `test_dag/test_run/sensitive_arg_task/-1/1 (ID 1) starting` 
make sure that "some_password" is not there (it would be there if not masked). 
   
   So I think if we just test for "some password" - we at most checj that we 
have not "yet" received sensitive information :D



-- 
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-unsubscr...@airflow.apache.org

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

Reply via email to