Skip to content

Commit

Permalink
Merge pull request #5 from trautmane/master
Browse files Browse the repository at this point in the history
fix MONGO_HOST check when configuring docker container
  • Loading branch information
fcollman authored Apr 16, 2018
2 parents c96e6a0 + 5c70ac0 commit bbde286
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -32,8 +32,8 @@ FROM build_environment as builder
COPY . /var/www/render/
RUN mvn clean

# use -T 1C option to multi-thread maven, using 1 thread per available core
RUN mvn -T 1C -Dproject.build.sourceEncoding=UTF-8 package && \
# note: cannot safely multi-thread maven for now because some unit tests are (but should not be) order dependent
RUN mvn -Dproject.build.sourceEncoding=UTF-8 package && \
mkdir -p /root/render-lib && \
mv */target/*.*ar /root/render-lib && \
printf "\nsaved the following build artifacts:\n\n" && \
Expand Down
2 changes: 1 addition & 1 deletion render-ws/src/main/scripts/docker/render-config.sh
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ JETTY_BASE=$(stripQuotes $JETTY_BASE)
RENDER_DB_PROPERTIES="${JETTY_BASE}/resources/render-db.properties"


if [ -z "${MONGO_HOST}" ] & [ -z "${MONGO_CONNECTION_STRING}" ]; then
if [ -z "${MONGO_HOST}" ] && [ -z "${MONGO_CONNECTION_STRING}" ]; then
echo "ERROR: either MONGO_HOST or MONGO_CONNECTION_STRING must be defined"
exit 1
fi
Expand Down

0 comments on commit bbde286

Please sign in to comment.