]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
qa/vstart_runner: fix mutiple 'timeout' option conflicting 34799/head
authorXiubo Li <xiubli@redhat.com>
Thu, 5 Mar 2020 10:31:14 +0000 (05:31 -0500)
committerNathan Cutler <ncutler@suse.com>
Tue, 28 Apr 2020 17:26:42 +0000 (19:26 +0200)
commit84004e85d0d1efeae20cce8cbb08033b2ed8dfee
tree328149124fa405f1630ec21d0f26f397d971d5f0
parentfcb0536921a652f58e87fc2b3304d30b9d6253c0
qa/vstart_runner: fix mutiple 'timeout' option conflicting

The 'timeout' option in the environment may will conflict with the
ones in some ceph commands, like:

$ timeout 120 ./bin/ceph daemon mds.b session config 8718 timeout 45

And the old code will also give us incorrect result like:

['adjust-ulimits', 'ceph-coverage', 'timeout', '120', 'ceph', 'fs', 'dump']
will be transfered to:
['adjust-ulimits', 'ceph-coverage', '120', 'ceph', 'fs', 'dump']

The '120' is left behind.

Fixes: https://tracker.ceph.com/issues/44437
Signed-off-by: Xiubo Li <xiubli@redhat.com>
(cherry picked from commit 1bcd319496f507f76864a972c1de63439dd78d4a)
qa/tasks/vstart_runner.py