There is currently multiple functions to run command this duplication should be removed.
- some modules define their run command (_cmd)
- some modules use os.system
- some modules use subprocess and a variation (call, run, Popen, check_output, ...)
There is therefore not one way it is done and error messages are not consistently presented back to the user.
All the function call should be channelled via vyos.util where helper functions should be provided.
It will also allow to centrally log all command failure futher helping debugging.