Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Introduce new config variable to control whether DAG processor outputs to stdout #37439

Conversation

Owen-CH-Leung
Copy link
Contributor

fixes #37122

This PR adds a new config variable under logging called dag_processor_manager_log_stdout, which is set to False by default. If being set to True, the DagFileProcessorAgent will also output logs to the stdout of airflow scheduler, in addition to writing logs to the designated log file defined by dag_processor_manager_log_location

@Owen-CH-Leung Owen-CH-Leung marked this pull request as ready for review February 16, 2024 09:16
@eladkal eladkal added this to the Airflow 2.8.2 milestone Feb 19, 2024
@eladkal eladkal added the type:bug-fix Changelog: Bug Fixes label Feb 19, 2024
@jedcunningham jedcunningham added type:new-feature Changelog: New Features and removed type:bug-fix Changelog: Bug Fixes labels Feb 19, 2024
@potiuk
Copy link
Member

potiuk commented Feb 21, 2024

CAn you resolve conflicts and rebase please?

@Owen-CH-Leung
Copy link
Contributor Author

CAn you resolve conflicts and rebase please?

Sure & rebased

@potiuk potiuk merged commit 3c9863c into apache:main Feb 22, 2024
59 checks passed
sudiptob2 pushed a commit to Satoshi-Sh/airflow that referenced this pull request Feb 22, 2024
abhishekbhakat pushed a commit to abhishekbhakat/my_airflow that referenced this pull request Mar 5, 2024
@dimon222
Copy link
Contributor

dimon222 commented Apr 8, 2024

How to select output ONLY to stdout? If I write to both locations I risk getting OOM in container environment and I don't want to persist it perpetually in PVC.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:new-feature Changelog: New Features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow Dag Processor Manager to log to stdout
6 participants