Skip to content

Commit

Permalink
Revert "fix broken Dockerfile due to ARG (huh?! but fixes #206)"
Browse files Browse the repository at this point in the history
This reverts commit a9bd456.

see #206
  • Loading branch information
vorburger committed Dec 6, 2018
1 parent 80d07d6 commit c23d43f
Show file tree
Hide file tree
Showing 5 changed files with 13 additions and 13 deletions.
6 changes: 3 additions & 3 deletions java/images/centos/Dockerfile
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
FROM centos:7.5.1804

ARG EXPOSE_PORT=8080
EXPOSE ${EXPOSE_PORT}

ENV JOLOKIA_VERSION="1.6.0" \
PROMETHEUS_JMX_EXPORTER_VERSION="0.3.1" \
PATH=$PATH:"/usr/local/s2i" \
Expand Down Expand Up @@ -121,6 +124,3 @@ COPY settings.xml /opt/jboss/.m2/settings.xml
# Use the run script as default since we are working as an hybrid image which can be
# used directly to. (If we were a plain s2i image we would print the usage info here)
CMD [ "/usr/local/s2i/run" ]

# https://github.com/fabric8io-images/s2i/issues/115
EXPOSE 8080
6 changes: 3 additions & 3 deletions java/images/fedora-java11/Dockerfile
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
FROM fedora:29

ARG EXPOSE_PORT=8080
EXPOSE ${EXPOSE_PORT}

ENV JOLOKIA_VERSION="1.6.0" \
PROMETHEUS_JMX_EXPORTER_VERSION="0.3.1" \
PATH=$PATH:"/usr/local/s2i" \
Expand Down Expand Up @@ -121,6 +124,3 @@ COPY settings.xml /opt/jboss/.m2/settings.xml
# Use the run script as default since we are working as an hybrid image which can be
# used directly to. (If we were a plain s2i image we would print the usage info here)
CMD [ "/usr/local/s2i/run" ]

# https://github.com/fabric8io-images/s2i/issues/115
EXPOSE 8080
6 changes: 3 additions & 3 deletions java/images/rhel/Dockerfile
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
FROM jboss/openjdk18-rhel7:1.1-7

ARG EXPOSE_PORT=8080
EXPOSE ${EXPOSE_PORT}

ENV JOLOKIA_VERSION="1.5.0" \
PROMETHEUS_JMX_EXPORTER_VERSION="0.10" \
PATH=$PATH:"/usr/local/s2i" \
Expand Down Expand Up @@ -109,6 +112,3 @@ COPY settings.xml /home/jboss/.m2/settings.xml
# Use the run script as default since we are working as an hybrid image which can be
# used directly to. (If we were a plain s2i image we would print the usage info here)
CMD [ "/usr/local/s2i/run" ]

# https://github.com/fabric8io-images/s2i/issues/115
EXPOSE 8080
6 changes: 3 additions & 3 deletions java/templates/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,9 @@
var mavenVersion = fp.config.base.version.maven;
}}FROM {{= fp.config.base.from }}

ARG EXPOSE_PORT=8080
EXPOSE ${EXPOSE_PORT}

ENV JOLOKIA_VERSION="{{= fp.config.base.version.jolokia }}" \
PROMETHEUS_JMX_EXPORTER_VERSION="{{= fp.config.base.version.jmxexporter }}" \
PATH=$PATH:"/usr/local/s2i" \
Expand Down Expand Up @@ -116,6 +119,3 @@ COPY settings.xml /opt/jboss/.m2/settings.xml
# Use the run script as default since we are working as an hybrid image which can be
# used directly to. (If we were a plain s2i image we would print the usage info here)
CMD [ "/usr/local/s2i/run" ]

# https://github.com/fabric8io-images/s2i/issues/115
EXPOSE 8080
2 changes: 1 addition & 1 deletion test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -123,5 +123,5 @@ function test_image() {
# ==================================================================================

cd java ; fish-pepper ; cd ..
test_image "java/images/fedora-java11/" "s2i-java-11"
test_image "java/images/fedora-java11/" "s2i-java"
test_image "java/images/centos/" "s2i-java"

0 comments on commit c23d43f

Please sign in to comment.