]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
qa/workunits/mon/test_mon_config_key: use subprocess.run() instead of proc.communicate() 42221/head
authorKefu Chai <kchai@redhat.com>
Fri, 16 Apr 2021 04:09:14 +0000 (12:09 +0800)
committerNeha Ojha <nojha@redhat.com>
Tue, 6 Jul 2021 18:59:30 +0000 (18:59 +0000)
commita280edbe82dbca8d3a9639fd8ceee38e09a3f33a
tree325f527a009955adae04626b3a4d33b83c7e8e93
parent468a1be6bb5dc20af2dd5bb916071db5e960845b
qa/workunits/mon/test_mon_config_key: use subprocess.run() instead of proc.communicate()

the loop of proc.communicate() on python3.6, where we always are able to
get something out of stdout and/or stderr PIPEs. and the `stdout` and
`stderr` keep growing until out of memory. and teuthology considers
the command crashed after a while.

Fixes: https://tracker.ceph.com/issues/50393
Signed-off-by: Kefu Chai <kchai@redhat.com>
(cherry picked from commit f2e0770c276e6fffa01a0bcb2d969f02ee1aafef)
qa/workunits/mon/test_mon_config_key.py