Merge pull request #115 from NerdLogic/103-output-is-not-a-tty

If container exists but EULA not signed, reinstall

Makefile

@@ -68,7 +68,7 @@ vertica-stop: ## Stop and delete the vertica container.
 .PHONY: vertica-start
 vertica-start: etc/vertica-demo.conf ## start/restart the vertica container.
 	@source etc/vertica-demo.conf; \
-	if (($$(docker ps -a --no-trunc -q -f NAME="$$VERTICA_CONTAINER_NAME" | wc -l)==0)); then \
+	if [[ -z $$VERTICA_EULA ]] || [[ -z $$(docker ps -a --no-trunc -q -f NAME="$$VERTICA_CONTAINER_NAME") ]]; then \
 	    $(MAKE) vertica-install; \
 	else \
 	    bin/vertica-start; \

bin/vertica-install

@@ -62,7 +62,8 @@ docker network create "$DOCKER_NETWORK" 2>/dev/null
 VERTICA_RUNARGS+=( --network "$DOCKER_NETWORK" )
 
 # clear out old container in case it had issues
-docker container rm $VERTICA_CONTAINER_NAME 2>/dev/null
+docker stop $VERTICA_CONTAINER_NAME >/dev/null 2>&1
+docker container rm $VERTICA_CONTAINER_NAME >/dev/null 2>&1
 
 docker run --detach "${VERTICA_RUNARGS[@]}" "${VERTICA_ENV[@]}" "$VERTICA_DOCKER_IMAGE" || abort $? "Could not create docker"