public_git
/
democratic-csi
mirror of https://github.com/democratic-csi/democratic-csi.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Apply sudo only to targetcli bin, not entire command

This commit is contained in:
Jan Sušnik 2022-05-26 18:00:34 +02:00
parent a0d664abfa
commit f626a93f57
1 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/driver/controller-zfs-generic/index.js
View File

@ -476,19 +476,19 @@ delete ${iscsiName}
let command = "sh"; let command = "sh";
let args = ["-c"]; let args = ["-c"];
let taregetCliCommand = [];
taregetCliCommand.push(`echo "${data}"`.trim());
taregetCliCommand.push("|");
taregetCliCommand.push("targetcli");
let targetCliArgs = ["targetcli"];
if ( if (
_.get(this.options, "iscsi.shareStrategyTargetCli.sudoEnabled", false) _.get(this.options, "iscsi.shareStrategyTargetCli.sudoEnabled", false)
) { ) {
command = "sudo"; targetCliArgs.unshift("sudo");
args.unshift("sh");
} }
args.push("'" + taregetCliCommand.join(" ") + "'"); let targetCliCommand = [];
targetCliCommand.push(`echo "${data}"`.trim());
targetCliCommand.push("|");
targetCliCommand.push(targetCliArgs.join(" "));
args.push("'" + targetCliCommand.join(" ") + "'");
let logCommandTmp = command + " " + args.join(" "); let logCommandTmp = command + " " + args.join(" ");
let logCommand = ""; let logCommand = "";