-
Notifications
You must be signed in to change notification settings - Fork 196
Add support for Testing nginx-1.26 in CentOS Stream 10 #315
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
4 commits
Select commit
Hold shift + click to select a range
a34c170
Create Dockerfile.c10s based on Dockerfile.c9s
phracek 5d48447
Update Dockerfile.c10s to use CentOS Stream 10.
phracek f3d0e7e
Update README's with CentOS Stream 10 support
phracek 32379c3
Update build-and-push action with pushing to quay.io 1.26 version
phracek File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,88 @@ | ||
FROM quay.io/sclorg/s2i-core-c10s:c10s | ||
|
||
EXPOSE 8080 | ||
EXPOSE 8443 | ||
|
||
ENV NAME=nginx \ | ||
NGINX_VERSION=1.26 \ | ||
NGINX_SHORT_VER=126 \ | ||
VERSION=0 | ||
|
||
ENV SUMMARY="Platform for running nginx $NGINX_VERSION or building nginx-based application" \ | ||
DESCRIPTION="Nginx is a web server and a reverse proxy server for HTTP, SMTP, POP3 and IMAP \ | ||
protocols, with a strong focus on high concurrency, performance and low memory usage. The container \ | ||
image provides a containerized packaging of the nginx $NGINX_VERSION daemon. The image can be used \ | ||
as a base image for other applications based on nginx $NGINX_VERSION web server. \ | ||
Nginx server image can be extended using source-to-image tool." | ||
|
||
LABEL summary="${SUMMARY}" \ | ||
description="${DESCRIPTION}" \ | ||
io.k8s.description="${DESCRIPTION}" \ | ||
io.k8s.display-name="Nginx ${NGINX_VERSION}" \ | ||
io.openshift.expose-services="8080:http" \ | ||
io.openshift.expose-services="8443:https" \ | ||
io.openshift.tags="builder,${NAME},${NAME}-${NGINX_SHORT_VER}" \ | ||
com.redhat.component="${NAME}-${NGINX_SHORT_VER}-container" \ | ||
name="sclorg/${NAME}-${NGINX_SHORT_VER}-c10s" \ | ||
version="1" \ | ||
com.redhat.license_terms="https://www.redhat.com/en/about/red-hat-end-user-license-agreements#UBI" \ | ||
maintainer="SoftwareCollections.org <[email protected]>" \ | ||
help="For more information visit https://github.com/sclorg/${NAME}-container" \ | ||
usage="s2i build <SOURCE-REPOSITORY> quay.io/sclorg/${NAME}-${NGINX_SHORT_VER}-c10s:latest <APP-NAME>" | ||
|
||
ENV NGINX_CONFIGURATION_PATH=${APP_ROOT}/etc/nginx.d \ | ||
NGINX_CONF_PATH=/etc/nginx/nginx.conf \ | ||
NGINX_DEFAULT_CONF_PATH=${APP_ROOT}/etc/nginx.default.d \ | ||
NGINX_CONTAINER_SCRIPTS_PATH=/usr/share/container-scripts/nginx \ | ||
NGINX_APP_ROOT=${APP_ROOT} \ | ||
NGINX_LOG_PATH=/var/log/nginx \ | ||
NGINX_PERL_MODULE_PATH=${APP_ROOT}/etc/perl | ||
|
||
# Modules does not exist | ||
RUN INSTALL_PKGS="nss_wrapper bind-utils gettext hostname nginx nginx-mod-stream nginx-mod-http-perl" && \ | ||
dnf install -y --setopt=tsflags=nodocs $INSTALL_PKGS && \ | ||
rpm -V $INSTALL_PKGS && \ | ||
nginx -v 2>&1 | grep -qe "nginx/$NGINX_VERSION\." && echo "Found VERSION $NGINX_VERSION" && \ | ||
dnf -y clean all --enablerepo='*' | ||
|
||
# Copy the S2I scripts from the specific language image to $STI_SCRIPTS_PATH | ||
COPY $NGINX_VERSION/s2i/bin/ $STI_SCRIPTS_PATH | ||
|
||
# Copy extra files to the image. | ||
COPY $NGINX_VERSION/root/ / | ||
|
||
# Changing ownership and user rights to support following use-cases: | ||
# 1) running container on OpenShift, whose default security model | ||
# is to run the container under random UID, but GID=0 | ||
# 2) for working root-less container with UID=1001, which does not have | ||
# to have GID=0 | ||
# 3) for default use-case, that is running container directly on operating system, | ||
# with default UID and GID (1001:0) | ||
# Supported combinations of UID:GID are thus following: | ||
# UID=1001 && GID=0 | ||
# UID=<any>&& GID=0 | ||
# UID=1001 && GID=<any> | ||
RUN sed -i -f ${NGINX_APP_ROOT}/nginxconf.sed ${NGINX_CONF_PATH} && \ | ||
mkdir -p ${NGINX_APP_ROOT}/etc/nginx.d/ && \ | ||
mkdir -p ${NGINX_APP_ROOT}/etc/nginx.default.d/ && \ | ||
mkdir -p ${NGINX_APP_ROOT}/src/nginx-start/ && \ | ||
mkdir -p ${NGINX_CONTAINER_SCRIPTS_PATH}/nginx-start && \ | ||
mkdir -p ${NGINX_LOG_PATH} && \ | ||
mkdir -p ${NGINX_PERL_MODULE_PATH} && \ | ||
chown -R 1001:0 ${NGINX_CONF_PATH} && \ | ||
chown -R 1001:0 ${NGINX_APP_ROOT}/etc && \ | ||
chown -R 1001:0 ${NGINX_APP_ROOT}/src/nginx-start/ && \ | ||
chown -R 1001:0 ${NGINX_CONTAINER_SCRIPTS_PATH}/nginx-start && \ | ||
chown -R 1001:0 /var/lib/nginx /var/log/nginx /run && \ | ||
chmod ug+rw ${NGINX_CONF_PATH} && \ | ||
chmod -R ug+rwX ${NGINX_APP_ROOT}/etc && \ | ||
chmod -R ug+rwX ${NGINX_APP_ROOT}/src/nginx-start/ && \ | ||
chmod -R ug+rwX ${NGINX_CONTAINER_SCRIPTS_PATH}/nginx-start && \ | ||
chmod -R ug+rwX /var/lib/nginx /var/log/nginx /run && \ | ||
rpm-file-permissions | ||
|
||
USER 1001 | ||
|
||
STOPSIGNAL SIGQUIT | ||
|
||
CMD $STI_SCRIPTS_PATH/usage |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,9 +1,5 @@ | ||
FROM quay.io/fedora/s2i-core:40 | ||
|
||
# nginx 1.26 image. | ||
# | ||
# Volumes: | ||
# * /var/log/nginx/ - Storage for logs | ||
|
||
EXPOSE 8080 | ||
EXPOSE 8443 | ||
|
@@ -27,8 +23,8 @@ LABEL summary="${SUMMARY}" \ | |
io.k8s.display-name="Nginx ${NGINX_VERSION}" \ | ||
io.openshift.expose-services="8080:http" \ | ||
io.openshift.expose-services="8443:https" \ | ||
io.openshift.tags="builder,${NAME},${NAME}${NGINX_SHORT_VER}" \ | ||
com.redhat.component="${NAME}" \ | ||
io.openshift.tags="builder,${NAME},${NAME}-${NGINX_SHORT_VER}" \ | ||
com.redhat.component="${NAME}-${NGINX_SHORT_VER}" \ | ||
name="fedora/${NAME}-${NGINX_SHORT_VER}" \ | ||
version="${VERSION}" \ | ||
maintainer="SoftwareCollections.org <[email protected]>" \ | ||
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this intended to differ from the C10S entry?
com.redhat.component="${NAME}-${NGINX_SHORT_VER}-container" \
vs
com.redhat.component="${NAME}-${NGINX_SHORT_VER}" \