Skip to content

Commit

Permalink
Do not use VOLUMES because it fails in OpenShift Online
Browse files Browse the repository at this point in the history
It's rationalized here:
#30

Ideally this should be reverted once OpenShift Online is fixed.
  • Loading branch information
hhorak authored and omron93 committed Dec 8, 2017
1 parent fa29e0a commit 0591134
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 6 deletions.
6 changes: 4 additions & 2 deletions 2.4/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,9 @@ RUN /usr/libexec/httpd-prepare && rpm-file-permissions

USER 1001

VOLUME ["${HTTPD_DATA_PATH}"]
VOLUME ["${HTTPD_LOG_PATH}"]
# Not using VOLUME statement since it's not working in OpenShift Online:
# https://github.com/sclorg/httpd-container/issues/30
# VOLUME ["${HTTPD_DATA_PATH}"]
# VOLUME ["${HTTPD_LOG_PATH}"]

CMD ["/usr/bin/run-httpd"]
6 changes: 4 additions & 2 deletions 2.4/Dockerfile.fedora
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,9 @@ RUN /usr/libexec/httpd-prepare

USER 1001

VOLUME ["${HTTPD_DATA_PATH}"]
VOLUME ["${HTTPD_LOG_PATH}"]
# Not using VOLUME statement since it's not working in OpenShift Online:
# https://github.com/sclorg/httpd-container/issues/30
# VOLUME ["${HTTPD_DATA_PATH}"]
# VOLUME ["${HTTPD_LOG_PATH}"]

CMD ["/usr/bin/run-httpd"]
6 changes: 4 additions & 2 deletions 2.4/Dockerfile.rhel7
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,9 @@ RUN /usr/libexec/httpd-prepare && rpm-file-permissions

USER 1001

VOLUME ["${HTTPD_DATA_PATH}"]
VOLUME ["${HTTPD_LOG_PATH}"]
# Not using VOLUME statement since it's not working in OpenShift Online:
# https://github.com/sclorg/httpd-container/issues/30
# VOLUME ["${HTTPD_DATA_PATH}"]
# VOLUME ["${HTTPD_LOG_PATH}"]

CMD ["/usr/bin/run-httpd"]

0 comments on commit 0591134

Please sign in to comment.