You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/08/17 04:43:37 UTC
[GitHub] [airflow] jhtimmins opened a new pull request, #25754: Add option of sending DAG parser logs to stdout.
jhtimmins opened a new pull request, #25754:
URL: https://github.com/apache/airflow/pull/25754
This is a continuation of https://github.com/apache/airflow/pull/25528, with the addition of tests.
There were issues getting #25528 to run locally, so I duplicated the PR.
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] ashb commented on pull request #25754: Add option of sending DAG parser logs to stdout.
Posted by GitBox <gi...@apache.org>.
ashb commented on PR #25754:
URL: https://github.com/apache/airflow/pull/25754#issuecomment-1239088293
> @uranusjr are you referring to this error? https://github.com/apache/airflow/runs/8220641637?check_suite_focus=true#step:13:3875
No that is something else (but likely not this PR) https://github.com/apache/airflow/runs/8215672242?check_suite_focus=true#step:9:12013 is the error he's talking about I think
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] uranusjr commented on pull request #25754: Add option of sending DAG parser logs to stdout.
Posted by GitBox <gi...@apache.org>.
uranusjr commented on PR #25754:
URL: https://github.com/apache/airflow/pull/25754#issuecomment-1238425730
Test failures not related, it’s introduced into main by another PR.
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] jhtimmins merged pull request #25754: Add option of sending DAG parser logs to stdout.
Posted by GitBox <gi...@apache.org>.
jhtimmins merged PR #25754:
URL: https://github.com/apache/airflow/pull/25754
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on pull request #25754: Add option of sending DAG parser logs to stdout.
Posted by GitBox <gi...@apache.org>.
potiuk commented on PR #25754:
URL: https://github.com/apache/airflow/pull/25754#issuecomment-1228992063
Restarting is easiest way to try @jhtimmins. I did it now, but you should be able to do it when things fail as well.
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] jhtimmins commented on pull request #25754: Add option of sending DAG parser logs to stdout.
Posted by GitBox <gi...@apache.org>.
jhtimmins commented on PR #25754:
URL: https://github.com/apache/airflow/pull/25754#issuecomment-1217456085
@kaxil this includes the tests you requested for the linked PR
@uranusjr can you take another look at this PR? It's the same as the linked one, with the addition of tests.
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] jhtimmins commented on pull request #25754: Add option of sending DAG parser logs to stdout.
Posted by GitBox <gi...@apache.org>.
jhtimmins commented on PR #25754:
URL: https://github.com/apache/airflow/pull/25754#issuecomment-1239067962
@uranusjr are you referring to this error? https://github.com/apache/airflow/runs/8220641637?check_suite_focus=true#step:13:3875
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] jhtimmins commented on pull request #25754: Add option of sending DAG parser logs to stdout.
Posted by GitBox <gi...@apache.org>.
jhtimmins commented on PR #25754:
URL: https://github.com/apache/airflow/pull/25754#issuecomment-1220166449
@ashb @uranusjr looks like An unrelated file in the same test directory keeps failing on MSSQL2017. Is it well known that this is a problematic test environment? Unsure if I should disregard.
https://github.com/apache/airflow/runs/7892800524?check_suite_focus=true
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org