tf-ci-gateway, tf-a-builder: Pass test description in a clearer way
Despite the fact that the tf-a-builder had a *string* parameter named
TEST_DESC, what actually was posted there by the tf-ci-gateway job
was a *file*. It turned out that both file *name* and *content* was
significant, even though this fact (especially about the filename)
wasn't clear/obvious. Besides overall confusion about the flow
between 2 jobs, that also had user experience issues, as the
TEST_DESC value wasn't shown directly fot a tf-a-builder instance,
instead requiring extra clicks to expose - why the test description
is arguably the most important parameter of tf-a-builder, which
should be readily visible.
Change this by making tf-a-builder accepting 2 explicit string
parameters: TEST_DESC (previously encoded by the filename) and
TEST_CONFIG (previously encoded by file contents). These parameters
are posted by tf-ci-gateway from a .fileprop file, and this file
is in turn generated by the updated gen_test_desc.py from the
tf-a-ci-scripts repository. This change thus depends on the
separate patch submitted against that repo.
Signed-off-by: Paul Sokolovsky <paul.sokolovsky@linaro.org>
Change-Id: Ia745d0f6967eaed116028d6e5af38494c6f7fe1e
diff --git a/tf-a-builder/builders.sh b/tf-a-builder/builders.sh
index 7618c10..8137f5b 100755
--- a/tf-a-builder/builders.sh
+++ b/tf-a-builder/builders.sh
@@ -163,7 +163,7 @@
test_group="$(echo "$lhs" | awk -F% '{print $2}')"
build_config="$(echo "$lhs" | awk -F% '{print $3}')"
run_config="${rhs%.test}"
-test_config="$(cat $WORKSPACE/TEST_DESC)"
+test_config="${TEST_CONFIG}"
export BUILD_CONFIG="$build_config"
export RUN_CONFIG="$run_config"