Github user JonZeolla commented on a diff in the pull request:
https://github.com/apache/metron/pull/1261#discussion_r236250204
--- Diff:
metron-deployment/development/centos6_docker_build/build_and_run.sh ---
@@ -0,0 +1,144 @@
+#!/usr/bin/env bash
+
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements. See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to You under the Apache License, Version 2.0
+# (the "License"); you may not use this file except in compliance with
+# the License. You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+shopt -s nocasematch
+
+function help {
+ echo " "
+ echo "usage: ${0}"
+ echo " --skip-vagrant-up skip vagrant up"
+ echo " --force-docker-build force build docker machine"
+ echo " --skip-tags='tag,tag2,tag3' the ansible skip tags"
+ echo " -h/--help Usage information."
+ echo " "
+ echo "example: to skip vagrant up and force docker build with two tags"
+ echo " build_and_run.sh -skip-vagrant-up --force-docker-build
--skip-tags='solr,sensors'"
+ echo " "
+}
+
+SKIP_VAGRANT_UP=false
+FORCE_DOCKER_BUILD=false
+A_SKIP_TAGS="sensors,solr"
+
+# handle command line options
+for i in "$@"; do
+ case $i in
+ #
+ # SKIP_VAGRANT_UP
+ #
+ #
+ --skip-vagrant-up)
+ SKIP_VAGRANT_UP=true
+ shift # past argument
+ ;;
+
+ #
+ # FORCE_DOCKER_BUILD
+ #
+ # --force-docker-build
+ #
+ --force-docker-build)
+ FORCE_DOCKER_BUILD=true
+ shift # past argument
+ ;;
+
+ #
+ # SKIP_TAGS
+ #
+ # --skip-tags='foo,bar'
+ #
+ --skip-tags=*)
+ A_SKIP_TAGS="${i#*=}"
+ shift # past argument=value
+ ;;
+
+ #
+ # -h/--help
+ #
+ -h|--help)
+ help
+ exit 0
+ shift # past argument with no value
+ ;;
+
+ #
+ # Unknown option
+ #
+ *)
+ UNKNOWN_OPTION="${i#*=}"
+ echo "Error: unknown option: $UNKNOWN_OPTION"
+ help
+ ;;
+ esac
+done
+
+echo "Running with "
+echo "SKIP_VAGRANT_UP = $SKIP_VAGRANT_UP"
+echo "FORCE_DOCKER_BUILD = $FORCE_DOCKER_BUILD"
+echo "SKIP_TAGS = $A_SKIP_TAGS"
+echo "==================================================="
+
+if [[ "$SKIP_VAGRANT_UP" = false ]]; then
+ vagrant up
+ rc=$?; if [[ $rc != 0 ]]; then exit $rc; fi
+fi
+
+VAGRANT_PATH="$( cd "$( dirname "${BASH_SOURCE[0]}" )" >/dev/null && pwd )"
+ANSIBLE_PATH=${VAGRANT_PATH}/ansible
+VAGRANT_KEY_PATH=${VAGRANT_PATH}/.vagrant/machines/node1/virtualbox
+
+# move over to the docker area
+cd ../docker || exit 1
+
+# Give the option to not build the docker container, which can take some
time and not be necessary
+if [[ "$FORCE_DOCKER_BUILD" = true ]]; then
+ echo "docker build"
+ docker build -t metron-build-docker:latest .
+fi
+
+if [[ ! -d ~/.m2 ]]; then
+ mkdir ~/.m2
+fi
+
+DATE=`date`
+LOG_DATE=${DATE// /_}
+LOGNAME="metron-build-${LOG_DATE}.log"
+
+# get the node1 ip address so we can add it to the docker hosts
+NODE1_IP=$(awk '/^\s*hosts/{flag=1; next} /}]/{flag=0} flag'
${VAGRANT_PATH}/Vagrantfile | grep "^\s*ip:" | awk -F'"' '{print $2}')
+if [[ -z "${NODE1_IP}" ]]; then exit 1; fi
+echo "Using NODE1 IP ${NODE1_IP}"
+
+echo "===============Running Docker==============="
+docker run -it \
+ -v ${VAGRANT_PATH}/../../..:/root/metron \
+ -v ~/.m2:/root/.m2 \
+ -v ${VAGRANT_PATH}:/root/vagrant \
+ -v ${ANSIBLE_PATH}:/root/ansible_config \
+ -v ${VAGRANT_KEY_PATH}:/root/vagrant_key \
+ -v ${VAGRANT_PATH}/logs:/root/logs \
+ -e ANSIBLE_CONFIG='/root/ansible_config/ansible.cfg' \
+ -e ANSIBLE_LOG_PATH="/root/logs/${LOGNAME}" \
+ -e ANSIBLE_SKIP_TAGS=${A_SKIP_TAGS} \
--- End diff --
From shellcheck.
```
In build_and_run.sh line 130:
-v ${VAGRANT_PATH}/../../..:/root/metron \
^-- SC2086: Double quote to prevent globbing and word splitting.
In build_and_run.sh line 132:
-v ${VAGRANT_PATH}:/root/vagrant \
^-- SC2086: Double quote to prevent globbing and word splitting.
In build_and_run.sh line 133:
-v ${ANSIBLE_PATH}:/root/ansible_config \
^-- SC2086: Double quote to prevent globbing and word splitting.
In build_and_run.sh line 134:
-v ${VAGRANT_KEY_PATH}:/root/vagrant_key \
^-- SC2086: Double quote to prevent globbing and word splitting.
In build_and_run.sh line 135:
-v ${VAGRANT_PATH}/logs:/root/logs \
^-- SC2086: Double quote to prevent globbing and word splitting.
In build_and_run.sh line 138:
-e ANSIBLE_SKIP_TAGS=${A_SKIP_TAGS} \
^-- SC2086: Double quote to prevent globbing and word
splitting.
```
---
