diff options
| author | Christian Poessinger <christian@poessinger.com> | 2021-05-18 19:13:34 +0200 |
|---|---|---|
| committer | Christian Poessinger <christian@poessinger.com> | 2021-05-18 21:30:44 +0200 |
| commit | 432e5048db19ddf5a7505866421f77bb01635d08 (patch) | |
| tree | 93c29638f3cbe8a808a1895950191c307719df73 /op-mode-definitions/flow-accounting-op.xml.in | |
| parent | 6816d37501aa495d71a441a8e391a1249c4d13f6 (diff) | |
| download | vyos-1x-432e5048db19ddf5a7505866421f77bb01635d08.tar.gz vyos-1x-432e5048db19ddf5a7505866421f77bb01635d08.zip | |
revert: "util: T2467: add autosudo as an option to command"
Implementing a wrapper which will - based on the command - add a sudo prefix
to the execution string seemed to be a nice idea but unfortunately it did not
only not get momentum but also the codebase somethis added an implicit "sudo"
call a second time.
This resulted in a call: "sudo sudo systemctl"
The entire functionality was removed again and if an op-mode script requires
root priviledges it must be explicitly called with them - no black magic.
(cherry picked from commit b76931e6d3a4bd0cdb74ca447a746ab6855fcf55)
Diffstat (limited to 'op-mode-definitions/flow-accounting-op.xml.in')
| -rw-r--r-- | op-mode-definitions/flow-accounting-op.xml.in | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/op-mode-definitions/flow-accounting-op.xml.in b/op-mode-definitions/flow-accounting-op.xml.in index b847338f9..7aaae5974 100644 --- a/op-mode-definitions/flow-accounting-op.xml.in +++ b/op-mode-definitions/flow-accounting-op.xml.in @@ -57,7 +57,7 @@ <properties> <help>Restart (net)flow accounting process</help> </properties> - <command>${vyos_op_scripts_dir}/flow_accounting_op.py --action restart</command> + <command>sudo ${vyos_op_scripts_dir}/flow_accounting_op.py --action restart</command> </leafNode> </children> </node> |