From 2784641e3dbfe940aa62e111fc0ef794fbd9fa8d Mon Sep 17 00:00:00 2001 From: Jakub Libosvar Date: Tue, 1 Apr 2014 16:41:38 +0200 Subject: [PATCH] Use os.uname() instead of calling uname in subprocess Related-Bug: #1291535 Change-Id: I57e18c0b95904f318e3fff08cd18b32e28f48834 --- neutron/agent/linux/ovs_lib.py | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/neutron/agent/linux/ovs_lib.py b/neutron/agent/linux/ovs_lib.py index e1172234f8..b9670d5350 100644 --- a/neutron/agent/linux/ovs_lib.py +++ b/neutron/agent/linux/ovs_lib.py @@ -14,6 +14,7 @@ # under the License. import distutils.version as dist_version +import os import re from oslo.config import cfg @@ -480,13 +481,9 @@ def get_installed_ovs_klm_version(): def get_installed_kernel_version(): - args = ["uname", "-r"] try: - cmd = utils.execute(args) - for line in cmd.split('\n'): - ver = re.findall("\d+\.\d+\.\d+", line) - return ver[0] - except Exception: + return os.uname()[2].split('-', 1)[0] + except IndexError: LOG.exception(_("Unable to retrieve installed Linux kernel version."))