Merge pull request #606 from Swirrl/no-volumes-in-dockerfile

remove VOLUME from dockerfile -- it's a trap

Author: RickMoynihan

drafter/README.md

@@ -61,3 +61,15 @@ $ stardog query <database-name> <path-to-migration>
 If a version bump requires a migration to be run, it should be noted in the
 release notes.
 
+## Docker
+
+CI will build and push a docker image to
+
+```
+europe-west2-docker.pkg.dev/swirrl-devops-infrastructure-1/swirrl/drafter-pmd4
+```
+
+tagged with git tag, branch name, and commit sha.
+
+Consumers may want to mount volumes at `/app/config` to provide custom
+configuration, and at `/app/stasher-cache` to persist the cache.

drafter/build.clj

@@ -28,7 +28,6 @@
       :image-type (get opts :image-type :docker)
       :include {"/app/config" ["./resources/drafter-auth0.edn"]}
       :base-image "gcr.io/distroless/java:11"
-      :volumes #{"/app/config" "/app/stasher-cache"}
       ;; NOTE Not as documented!
       ;; The docstring states that these should be
       ;;     :to-registry {:username ... :password ...}

drafter/deps.edn

@@ -144,7 +144,7 @@
                   :main-opts ["-m" "mach.pack.alpha.skinny"]}
 
            :build {:deps {io.github.clojure/tools.build {:git/tag "v0.7.5" :git/sha "34727f7"}
-                          io.github.rickmoynihan/pack.alpha {:git/sha "e461cf1ad0e6aa1a9d93e924bcbad32ae52db783"}}
+                          io.github.juxt/pack.alpha {:git/sha "9fd4a63ddeec3bedcbc819422de5a40ccb9eb8c9"}}
                    :ns-default build}
 
            :java9 {:jvm-opts ["--add-modules" "java.xml.bind"]}