Re: [PR] Introduce new config variable to control whether DAG processor outputs to stdout [airflow]

2024-04-08 Thread via GitHub
dimon222 commented on PR #37439: URL: https://github.com/apache/airflow/pull/37439#issuecomment-2043662173 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. -- This is an automa

Re: [PR] Introduce new config variable to control whether DAG processor outputs to stdout [airflow]

2024-02-22 Thread via GitHub
potiuk merged PR #37439: URL: https://github.com/apache/airflow/pull/37439 -- 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.a

Re: [PR] Introduce new config variable to control whether DAG processor outputs to stdout [airflow]

2024-02-21 Thread via GitHub
Owen-CH-Leung commented on PR #37439: URL: https://github.com/apache/airflow/pull/37439#issuecomment-1958581109 > CAn you resolve conflicts and rebase please? Sure & rebased -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

Re: [PR] Introduce new config variable to control whether DAG processor outputs to stdout [airflow]

2024-02-21 Thread via GitHub
potiuk commented on PR #37439: URL: https://github.com/apache/airflow/pull/37439#issuecomment-1958145958 CAn you resolve conflicts and rebase please? -- 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

Re: [PR] Introduce new config variable to control whether DAG processor outputs to stdout [airflow]

2024-02-20 Thread via GitHub
Owen-CH-Leung commented on code in PR #37439: URL: https://github.com/apache/airflow/pull/37439#discussion_r1495775381 ## airflow/config_templates/config.yml: ## @@ -857,6 +857,13 @@ logging: type: string example: ~ default: "{AIRFLOW_HOME}/logs/dag_process

Re: [PR] Introduce new config variable to control whether DAG processor outputs to stdout [airflow]

2024-02-19 Thread via GitHub
eladkal commented on code in PR #37439: URL: https://github.com/apache/airflow/pull/37439#discussion_r1494671554 ## airflow/config_templates/config.yml: ## @@ -857,6 +857,13 @@ logging: type: string example: ~ default: "{AIRFLOW_HOME}/logs/dag_processor_man

Re: [PR] Introduce new config variable to control whether DAG processor outputs to stdout [airflow]

2024-02-16 Thread via GitHub
Owen-CH-Leung commented on code in PR #37439: URL: https://github.com/apache/airflow/pull/37439#discussion_r1492152298 ## tests/dag_processing/test_job_runner.py: ## @@ -1460,3 +1460,20 @@ def test_launch_process(self): processor_agent._process.join() assert

Re: [PR] Introduce new config variable to control whether DAG processor outputs to stdout [airflow]

2024-02-15 Thread via GitHub
dirrao commented on code in PR #37439: URL: https://github.com/apache/airflow/pull/37439#discussion_r1490899402 ## tests/dag_processing/test_job_runner.py: ## @@ -1460,3 +1460,20 @@ def test_launch_process(self): processor_agent._process.join() assert os.path