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 2021/09/20 15:48:16 UTC
[GitHub] [airflow] collinmcnulty opened a new pull request #18377: Make Kubernetes job description fit on one log line
collinmcnulty opened a new pull request #18377:
URL: https://github.com/apache/airflow/pull/18377
Currently, when the Kubernetes executor creates a pod it prints a dictionary description of the pod across many lines (can easily be 20+ lines). This is fine if you're reading the log in a stream in a text file, but throws off log search tools like Kibana. A better practice would be to print the whole pod description on a single line. It is quite easy to prettify a dictionary if one wants to see it back in a more human-friendly form with the newlines.
This update simply forces the log from this command into a single line.
--
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] kaxil merged pull request #18377: Make Kubernetes job description fit on one log line
Posted by GitBox <gi...@apache.org>.
kaxil merged pull request #18377:
URL: https://github.com/apache/airflow/pull/18377
--
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] collinmcnulty commented on pull request #18377: Make Kubernetes job description fit on one log line
Posted by GitBox <gi...@apache.org>.
collinmcnulty commented on pull request #18377:
URL: https://github.com/apache/airflow/pull/18377#issuecomment-924034972
These tests are passing when I run in Breeze.
--
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] github-actions[bot] commented on pull request #18377: Make Kubernetes job description fit on one log line
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #18377:
URL: https://github.com/apache/airflow/pull/18377#issuecomment-923160182
The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the risk. If they don't merge it quickly - please rebase it to the latest main at your convenience, or amend the last commit of the PR, and push it with --force-with-lease.
--
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