summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChristian Poessinger <christian@poessinger.com>2020-04-04 11:29:37 +0200
committerChristian Poessinger <christian@poessinger.com>2020-04-04 11:29:37 +0200
commit25bfece386756cbd40b83bc26141edddb0e05c72 (patch)
tree129825f3aba898b46352d15b228ebc53b2a43511
parent2e1562fb02e7a2b45e84c0b8d861d149bfa890d2 (diff)
downloadvyos-1x-25bfece386756cbd40b83bc26141edddb0e05c72.tar.gz
vyos-1x-25bfece386756cbd40b83bc26141edddb0e05c72.zip
vyos.util: use common subprocess_cmd wrapper
-rw-r--r--python/vyos/util.py8
-rwxr-xr-xsrc/conf_mode/interfaces-pppoe.py7
-rwxr-xr-xsrc/conf_mode/interfaces-wirelessmodem.py7
3 files changed, 10 insertions, 12 deletions
diff --git a/python/vyos/util.py b/python/vyos/util.py
index 3970b8bf1..5807c837d 100644
--- a/python/vyos/util.py
+++ b/python/vyos/util.py
@@ -17,6 +17,14 @@ import os
import re
import sys
+
+def subprocess_cmd(command):
+ """ execute arbitrary command via Popen """
+ from subprocess import Popen, PIPE
+ p = Popen(command, stdout=PIPE, shell=True)
+ p.communicate()
+
+
def read_file(path):
""" Read a file to string """
with open(path, 'r') as f:
diff --git a/src/conf_mode/interfaces-pppoe.py b/src/conf_mode/interfaces-pppoe.py
index 9c045534c..97b0ff0df 100755
--- a/src/conf_mode/interfaces-pppoe.py
+++ b/src/conf_mode/interfaces-pppoe.py
@@ -19,11 +19,10 @@ import os
from sys import exit
from copy import deepcopy
from jinja2 import Template
-from subprocess import Popen, PIPE
from vyos.config import Config
from vyos.ifconfig import Interface
-from vyos.util import chown_file, chmod_x_file
+from vyos.util import chown_file, chmod_x_file, subprocess_cmd
from vyos import ConfigError
from netifaces import interfaces
@@ -189,10 +188,6 @@ default_config_data = {
'vrf': ''
}
-def subprocess_cmd(command):
- p = Popen(command, stdout=PIPE, shell=True)
- p.communicate()
-
def get_config():
pppoe = deepcopy(default_config_data)
conf = Config()
diff --git a/src/conf_mode/interfaces-wirelessmodem.py b/src/conf_mode/interfaces-wirelessmodem.py
index bfefa5dd1..7a10b9c43 100755
--- a/src/conf_mode/interfaces-wirelessmodem.py
+++ b/src/conf_mode/interfaces-wirelessmodem.py
@@ -19,12 +19,11 @@ import os
from sys import exit
from copy import deepcopy
from jinja2 import FileSystemLoader, Environment
-from subprocess import Popen, PIPE
from netifaces import interfaces
from vyos.config import Config
-from vyos.util import chown_file, chmod_x_file
from vyos.defaults import directories as vyos_data_dir
+from vyos.util import chown_file, chmod_x_file, subprocess_cmd
from vyos import ConfigError
default_config_data = {
@@ -45,10 +44,6 @@ default_config_data = {
'vrf': ''
}
-def subprocess_cmd(command):
- p = Popen(command, stdout=PIPE, shell=True)
- p.communicate()
-
def check_kmod():
modules = ['option', 'usb_wwan', 'usbserial']
for module in modules: