korex-f commented on PR #66633: URL: https://github.com/apache/airflow/pull/66633#issuecomment-5149814844
> Hi @korex-f, > > Thanks for follow-up on this. We need to do the following items before merge. > > * Add unit test to exercise the patch itself > > * Fix the CI failure > > * Rephrase the PR title and the description > > > Thanks. Added a regression test in test_log.py that verifies load_remote_log_handler() is called only after the inner configure_logging() (dictConfig) returns. The CI failure in test_inject_parent_job_info_with_resume_on_retry is unrelated, it's a Glue operator test making a real network call that times out, last touched in #64513. Happy to re-trigger CI to confirm it's a flake. -- 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]
