ci.jpl: Remove not useful and confusing set +x/-x commands
This partially reverts commit d091f811a12. That commit intended to avoid to
"print duplicated Python commands in CI logs", but the actual result of it
is that logs now contain stuff like:
[Pipeline] sh
00:25:11 + set +x
[Pipeline] sh
00:25:12 + set +x
[Pipeline] sh
00:25:12 + set +x
[Pipeline] sh
00:25:13 + set +x
[Pipeline] sh
00:25:13 + set +x
[Pipeline] sh
00:25:14 + set +x
[Pipeline] sh
00:25:14 + set +x
[Pipeline] sh
00:25:15 + set +x
(may be repeated 100 of times).
It's much better to print the actual Python command. Actually, it's
important to print the actual commands being run, that's immense help
to debug any issues with CI jobs (which are many, and even when they
won't be many, they will still happen).
Signed-off-by: Paul Sokolovsky <paul.sokolovsky@linaro.org>
Change-Id: I5c66ae7cc831f0a5d403700d3b1260061e09fedb
diff --git a/jenkins/ci.jpl b/jenkins/ci.jpl
index 318274f..82e99b9 100644
--- a/jenkins/ci.jpl
+++ b/jenkins/ci.jpl
@@ -45,9 +45,7 @@
def listConfigs(config_list, filter_group) {
def build_config_list_raw = sh(script:
"""
- set +x\n
python3 ./tf-m-ci-scripts/configs.py -g ${filter_group.replace(" ", " -g ")}\n
- set -x
""", returnStdout: true).trim()
def build_config_list = build_config_list_raw.tokenize('\n')
config_list.addAll(build_config_list)
@@ -57,9 +55,7 @@
def build_params = [:]
build_config_params = sh(script:
"""
- set +x\n
python3 ./tf-m-ci-scripts/configs.py ${config}\n
- set -x
""", returnStdout: true).trim()
def lines = build_config_params.tokenize('\n')
for (String line : lines) {