Skip to content

Commit

Permalink
Merge pull request #3 from signalsciences/bbucher-23072-remove-build-…
Browse files Browse the repository at this point in the history
…number

BUILD_NUMBER is no longer part of the filename
  • Loading branch information
bbucher-sigsci authored Apr 17, 2020
2 parents 10e3fdf + 87ce46a commit 6afa6fa
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions Dockerfile
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
ARG NGINX_INGRESS_VERSION=${NGINX_INGRESS_VERSION:-latest}
ARG NGINX_INGRESS_VERSION=${NGINX_INGRESS_VERSION:-0.30.0}
FROM quay.io/kubernetes-ingress-controller/nginx-ingress-controller:${NGINX_INGRESS_VERSION}
ARG PKGNAME=${PKGNAME:-nginx-module-sigsci-nxo}
ARG BUILD_NUMBER=146

# Change to the root user to update the container
USER root
Expand All @@ -19,7 +18,7 @@ RUN apk update && apk add --no-cache gnupg wget --virtual ./build_deps \
# Get the latest version of the sigsci nginx native module
&& MODULE_VERSION=$(wget -O- -q https://dl.signalsciences.net/sigsci-module-nginx-native/VERSION) \
# Get the correct sigsci nginx native module based on alpine version, nginx version, and module version
&& wget -O /tmp/nginx-module-sigsci-nxo_${NGXVERSION}-${BUILD_NUMBER}-alpine${ALPINE_RELEASE}.tar.gz https://dl.signalsciences.net/sigsci-module-nginx-native/${MODULE_VERSION}/alpine/alpine${ALPINE_RELEASE}/nginx-module-sigsci-nxo_${NGXVERSION}-${BUILD_NUMBER}-alpine${ALPINE_RELEASE}.tar.gz \
&& wget -O /tmp/nginx-module-sigsci-nxo_${NGXVERSION}-alpine${ALPINE_RELEASE}.tar.gz https://dl.signalsciences.net/sigsci-module-nginx-native/${MODULE_VERSION}/alpine/alpine${ALPINE_RELEASE}/nginx-module-sigsci-nxo_${NGXVERSION}-alpine${ALPINE_RELEASE}.tar.gz \
# Manually install the sigsci native nginx module and update nginx.conf
&& tar xvfz /tmp/nginx-module-sigsci-nxo_${NGXVERSION}-${BUILD_NUMBER}-alpine${ALPINE_RELEASE}.tar.gz -C /tmp || : \
&& mkdir -p /usr/lib/nginx/modules \
Expand Down

0 comments on commit 6afa6fa

Please sign in to comment.