Merge pull request #170 from rhuss/master

fix: Set working directory back to /home/jboss
fabric8io-images · Aug 10, 2018 · 7d1d6d2 · 7d1d6d2
2 parents 58b5c6f + 5aa0dfc
commit 7d1d6d2
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 1 deletion.
diff --git a/java/images.yml b/java/images.yml
@@ -3,7 +3,7 @@ fish-pepper:
   params:
     - base
   name: "fabric8/s2i-java"
-  build: "2.3.0"
+  build: "2.3.1"
 
 config:
   base:

diff --git a/java/images/centos/Dockerfile b/java/images/centos/Dockerfile
@@ -107,6 +107,9 @@ RUN mkdir -p /deployments/data \
  && chown -R jboss:root /opt/jboss \
  && chmod 664 /etc/passwd
 
+# S2I scripts rely on /opt/jboss as working directory
+WORKDIR /opt/jboss
+
 # S2I requires a numeric, non-0 UID. This is the UID for the jboss user in the base image
 
 USER 1000

diff --git a/java/images/rhel/Dockerfile b/java/images/rhel/Dockerfile
@@ -96,6 +96,9 @@ RUN mkdir -p /deployments/data \
  && chown -R jboss:root /home/jboss \
  && chmod 664 /etc/passwd
 
+# S2I scripts rely on /home/jboss as working directory
+WORKDIR /home/jboss
+
 # S2I requires a numeric, non-0 UID. This is the UID for the jboss user in the base image
 
 USER 185

diff --git a/java/templates/Dockerfile b/java/templates/Dockerfile
@@ -98,6 +98,9 @@ RUN mkdir -p /deployments/data \
  && chown -R jboss:root {{= fp.config.base.home }} \
  && chmod 664 /etc/passwd
 
+# S2I scripts rely on {{= fp.config.base.home }} as working directory
+WORKDIR {{= fp.config.base.home }}
+
 # S2I requires a numeric, non-0 UID. This is the UID for the jboss user in the base image
 {{? fp.param.base === "rhel"}}
 USER 185