The GitHub Actions job "Tests" on airflow.git/overhaul-task-logging has failed.
Run started by GitHub user ashb (triggered by ashb).

Head commit for run:
5c79c1358bd46ac4ea5de47c58d0213f7bf653e5 / Ash Berlin-Taylor <[email protected]>
Rework remote task log handling for the structlog era.

Previously this feature was built on top of the stdlib logging.Handler
interface, and it worked but had a few issues (even before we switched to
structlog for Task SDK):

- we had to use what is in many ways a hack with the "set_context" to get
  information down in to the task handler.
- Discovering of the configured task handler was somewhat baroque
- The whole thing is just complex due to the features of stdlib logging
  (loggers, propagate, handler levels etc etc.)
- The upload was triggered somewhat "automatically" inside close, which from
  an abstraction point of view is messy.

This changes things to have a more explicit interface purpose made for
uploading task log files and for reading them, and perhaps more crucially for
things like CloudWatch Logs, it (re)adds the ability to install a structlog
processor that will receive every log message as it happens.

The return types for the read et al functions were confusing the living
daylights out of me, so I've created type alias to give the return types
explicit names to reduce (my) confusion.

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

With regards,
GitHub Actions via GitBox


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

Reply via email to