The GitHub Actions job "Tests (AMD)" on 
airflow.git/fix/triggerer-spurious-remote-log-warning has failed.
Run started by GitHub user anmolxlight (triggered by anmolxlight).

Head commit for run:
9cfa7341b17f982429107336c83bbea3be506ff6 / anmol mishra <[email protected]>
fix(logging): gate remote log warning on expected state, not just config flag

The previous code checked remote_logging config in upload_to_remote(),
but that's narrower than what actually decides whether a handler exists.
A custom logging_config_class exporting REMOTE_TASK_LOG, or the fallback
discover_remote_log_handler() path, both produce a handler without
remote_logging=True.

Compute remote_logging_expected at load time based on whether a custom
logging_config_class is set or remote_logging=True, store it in
_ActiveLoggingConfig, and gate the warning on that flag instead of
re-deriving intent at the upload call site.

Addresses review feedback from @kaxil on #68239.

Report URL: https://github.com/apache/airflow/actions/runs/27173887512

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to