[bigtop] branch master updated: BIGTOP-3408. docker-hadoop.sh generates broken site.yaml from config.yaml containing multiple components. (#674)

[se...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

sekikn pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/bigtop.git


The following commit(s) were added to refs/heads/master by this push:
     new 543f17e  BIGTOP-3408. docker-hadoop.sh generates broken site.yaml from config.yaml containing multiple components. (#674)
543f17e is described below

commit 543f17e4faa8312812b732fe8d99d7b366f74868
Author: Masatake Iwasaki <iw...@apache.org>
AuthorDate: Mon Sep 21 03:14:51 2020 +0900

    BIGTOP-3408. docker-hadoop.sh generates broken site.yaml from config.yaml containing multiple components. (#674)
---
 provisioner/docker/docker-hadoop.sh | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/provisioner/docker/docker-hadoop.sh b/provisioner/docker/docker-hadoop.sh
index 8931b1d..b15c14d 100755
--- a/provisioner/docker/docker-hadoop.sh
+++ b/provisioner/docker/docker-hadoop.sh
@@ -93,7 +93,7 @@ create() {
         repo=$(get-yaml-config repo)
     fi
     if [ -z ${components+x} ]; then
-        components="[`echo "$(get-yaml-config components)" | sed 's/ /, /g'`]"
+        components="[`echo $(get-yaml-config components) | sed 's/ /, /g'`]"
     fi
     if [ -z ${distro+x} ]; then
         distro=$(get-yaml-config distro)
@@ -191,7 +191,7 @@ provision() {
 smoke-tests() {
     hadoop_head_node=${NODES:0:12}
     if [ -z ${smoke_test_components+x} ]; then
-        smoke_test_components="`echo "$(get-yaml-config smoke_test_components)" | sed 's/ /,/g'`"
+        smoke_test_components="`echo $(get-yaml-config smoke_test_components) | sed 's/ /,/g'`"
     fi
     docker exec $hadoop_head_node bash -c "bash -x /bigtop-home/provisioner/utils/smoke-tests.sh $smoke_test_components"
 }