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