Skip to content

Commit 900fad5

Browse files
committed
tests: Remove unused flag
Nothing is setting 'merge_stderr'. It looks like a carry-over from the legacy clients. Change-Id: I32b65830e11b27ba83dfba002bf996af561b0768 Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
1 parent 1678f87 commit 900fad5

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

  • openstackclient/tests/functional

openstackclient/tests/functional/base.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,12 +24,12 @@
2424
LOG = logging.getLogger(__name__)
2525

2626

27-
def execute(cmd, fail_ok=False, merge_stderr=False):
27+
def execute(cmd, *, fail_ok=False):
2828
"""Executes specified command for the given action."""
2929
LOG.debug('Executing: %s', cmd)
3030
cmdlist = shlex.split(cmd)
3131
stdout = subprocess.PIPE
32-
stderr = subprocess.STDOUT if merge_stderr else subprocess.PIPE
32+
stderr = subprocess.PIPE
3333

3434
proc = subprocess.Popen(cmdlist, stdout=stdout, stderr=stderr)
3535

0 commit comments

Comments
 (0)