diff options
author | Arne H Juul <arnej27959@users.noreply.github.com> | 2020-03-31 13:27:20 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-03-31 13:27:20 +0200 |
commit | 451173e78f50c4db14f0def7a12eb9881720b94a (patch) | |
tree | 56b432934450eff87e4208ee714bf2a0fc5360ba | |
parent | 292960179e19b3325076e9cc9d84806f47fcc8d2 (diff) | |
parent | 20a374f0d734fbeb87fa98e93837954b7888c8eb (diff) |
Merge pull request #12776 from vespa-engine/arnej/override-log-control
override log control file for verify-ranksetup
-rw-r--r-- | config-model/src/main/java/com/yahoo/vespa/model/application/validation/RankSetupValidator.java | 2 | ||||
-rw-r--r-- | vespajlib/src/main/java/com/yahoo/system/ProcessExecuter.java | 12 |
2 files changed, 13 insertions, 1 deletions
diff --git a/config-model/src/main/java/com/yahoo/vespa/model/application/validation/RankSetupValidator.java b/config-model/src/main/java/com/yahoo/vespa/model/application/validation/RankSetupValidator.java index b4246171277..b6f7ab4ff62 100644 --- a/config-model/src/main/java/com/yahoo/vespa/model/application/validation/RankSetupValidator.java +++ b/config-model/src/main/java/com/yahoo/vespa/model/application/validation/RankSetupValidator.java @@ -138,7 +138,7 @@ public class RankSetupValidator extends Validator { private boolean execValidate(String configId, SearchCluster sc, String sdName, DeployLogger deployLogger) { String job = String.format("%s %s", binaryName, configId); - ProcessExecuter executer = new ProcessExecuter(); + ProcessExecuter executer = new ProcessExecuter(true); try { Pair<Integer, String> ret = executer.exec(job); if (ret.getFirst() != 0) { diff --git a/vespajlib/src/main/java/com/yahoo/system/ProcessExecuter.java b/vespajlib/src/main/java/com/yahoo/system/ProcessExecuter.java index cceac7e84bb..c455929bf51 100644 --- a/vespajlib/src/main/java/com/yahoo/system/ProcessExecuter.java +++ b/vespajlib/src/main/java/com/yahoo/system/ProcessExecuter.java @@ -16,6 +16,14 @@ import com.yahoo.collections.Pair; */ public class ProcessExecuter { + private final boolean override_log_control; + public ProcessExecuter(boolean override_log_control) { + this.override_log_control = override_log_control; + } + public ProcessExecuter() { + this(false); + } + /** * Executes the given command synchronously without timeout. * @@ -39,6 +47,10 @@ public class ProcessExecuter { ProcessBuilder pb = new ProcessBuilder(command); StringBuilder ret = new StringBuilder(); pb.environment().remove("VESPA_LOG_TARGET"); + if (override_log_control) { + pb.environment().remove("VESPA_LOG_CONTROL_FILE"); + pb.environment().put("VESPA_SERVICE_NAME", "exec-" + command[0]); + } pb.redirectErrorStream(true); Process p = pb.start(); InputStream is = p.getInputStream(); |