From 0c5c43bfa1550209f94df5ae71d4943b4956d0cd Mon Sep 17 00:00:00 2001 From: Dmitriy Rabotyagov Date: Wed, 28 Apr 2021 12:54:55 +0300 Subject: [PATCH] Make journal_dump py3.6 compatable Currently journal_dump passes `capture_output` argument to subprocess.run which is valid only since py3.7 while we're running systems with default py3.6. We replace capture_output with old proven subprocess.PIPE for compatability with older Python versions. Change-Id: Id7713d8eeec86a910739ac64d3b0815fcfeb506d --- scripts/journal_dump.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/scripts/journal_dump.py b/scripts/journal_dump.py index f5e5572802..0c94ef514a 100755 --- a/scripts/journal_dump.py +++ b/scripts/journal_dump.py @@ -34,7 +34,7 @@ def get_container_journals(): journals = [] try: - s = subprocess.run(['lxc-ls', '-1'], capture_output=True) + s = subprocess.run(['lxc-ls', '-1'], stdout=subprocess.PIPE) except FileNotFoundError: return journals @@ -44,9 +44,9 @@ def get_container_journals(): info = {} info['name'] = container_name info['subdir'] = "openstack" - s = subprocess.run(['lxc-info', '--pid', '--no-humanize', container_name], capture_output=True) + s = subprocess.run(['lxc-info', '--pid', '--no-humanize', container_name], stdout=subprocess.PIPE) info['pid'] = s.stdout.decode('utf-8').strip() - + if(len(info['pid']) == 0): continue