This is an automated email from the ASF dual-hosted git repository.
kaxilnaik pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new c0fbe3a Fix broken static check on main (#18351)
c0fbe3a is described below
commit c0fbe3ad12eae4dc408a0ef103f2a359762f73e3
Author: Kaxil Naik <[email protected]>
AuthorDate: Sun Sep 19 02:03:18 2021 +0100
Fix broken static check on main (#18351)
This got broken in https://github.com/apache/airflow/pull/18349
---
airflow/utils/process_utils.py | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/airflow/utils/process_utils.py b/airflow/utils/process_utils.py
index f8587bb..3c03917 100644
--- a/airflow/utils/process_utils.py
+++ b/airflow/utils/process_utils.py
@@ -163,7 +163,12 @@ def execute_interactive(cmd: List[str], **kwargs):
try:
# use os.setsid() make it run in a new process group, or bash job
control will not be enabled
with subprocess.Popen(
- cmd, stdin=secondary_fd, stdout=secondary_fd, stderr=secondary_fd,
universal_newlines=True, **kwargs
+ cmd,
+ stdin=secondary_fd,
+ stdout=secondary_fd,
+ stderr=secondary_fd,
+ universal_newlines=True,
+ **kwargs,
) as proc:
while proc.poll() is None:
readable_fbs, _, _ = select.select([sys.stdin, primary_fd],
[], [])