Merge pull request #849 from houz42/master

use two-stage building for conductor:server

docker/server/Dockerfile

@@ -1,6 +1,20 @@
 #
 # conductor:server - Netflix conductor server
 #
+
+# 0. Builder stage
+FROM openjdk:8-jdk AS builder
+
+MAINTAINER Netflix OSS <[email protected]>
+
+# Copy the project directly onto the image
+COPY . /conductor
+WORKDIR /conductor
+
+# Build the server on run
+RUN ./gradlew build -x test
+
+# 1. Bin stage
 FROM openjdk:8-jre-alpine
 
 MAINTAINER Netflix OSS <[email protected]>
@@ -9,9 +23,9 @@ MAINTAINER Netflix OSS <[email protected]>
 RUN mkdir -p /app/config /app/logs /app/libs
 
 # Copy the project directly onto the image
-COPY ./docker/server/bin /app
-COPY ./docker/server/config /app/config
-COPY ./server/build/libs/conductor-server-*-all.jar /app/libs
+COPY --from=builder /conductor/docker/server/bin /app
+COPY --from=builder /conductor/docker/server/config /app/config
+COPY --from=builder /conductor/server/build/libs/conductor-server-*-all.jar /app/libs
 
 # Copy the files for the server into the app folders
 RUN chmod +x /app/startup.sh

docker/server/README.md

@@ -6,8 +6,6 @@ This Dockerfile create the conductor:server image
 
 Run the following commands from the project root.
 
-`docker build -f docker/server/Dockerfile.build -t conductor:server-build .`
-`docker run -v $(pwd):/conductor conductor:server-build`
 `docker build -f docker/server/Dockerfile -t conductor:server .`
 
 ## Running the conductor server