Skip to content

Commit

Permalink
Ensure method to override rancher project monitoring version still works
Browse files Browse the repository at this point in the history
  • Loading branch information
mallardduck committed Jan 28, 2025
1 parent 5197c6f commit 23d0064
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/prom-fed-e2e-ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@ jobs:
-
name: Perform pre-e2e image build
run: |
EMBEDED_CHART_VERSION=0.3.4 REPO=${REPO} TAG=${TAG} make build;
REPO=${REPO} TAG=${TAG} make build;
REPO=${REPO} TAG=${TAG} make package;
-
name : Install k3d
Expand Down
4 changes: 2 additions & 2 deletions package/Dockerfile-prometheus-federator
Original file line number Diff line number Diff line change
Expand Up @@ -21,10 +21,10 @@ FROM registry.suse.com/bci/golang:1.22 AS builder

# Allow chart version config
ARG TARGETPLATFORM
ARG EMBEDDED_CHART_VERSION=0.3.4
ARG RANCHER_PROJECT_MONITORING=''
ARG TAG=''
ARG REPO=''
ENV EMBEDDED_CHART_VERSION=$EMBEDDED_CHART_VERSION TAG=$TAG REPO=$REPO
ENV RANCHER_PROJECT_MONITORING=$RANCHER_PROJECT_MONITORING TAG=$TAG REPO=$REPO

WORKDIR /usr/src/app
COPY --from=helm ./helm/bin/helm /usr/local/bin/
Expand Down
4 changes: 2 additions & 2 deletions scripts/package
Original file line number Diff line number Diff line change
Expand Up @@ -12,9 +12,9 @@ DOCKERFILE=package/Dockerfile${DOCKER_TARGET}
echo "Building \`${IMAGE}\` from \`${DOCKERFILE}\`:";

if [[ ${USE_DOCKER_BUILDX} -eq 1 ]]; then
docker buildx build --platform linux/amd64,linux/arm64 -f "${DOCKERFILE}" . -t "${IMAGE}"
docker buildx build --platform linux/amd64,linux/arm64 -f "${DOCKERFILE}" --build-arg RANCHER_PROJECT_MONITORING=$RANCHER_PROJECT_MONITORING -t "${IMAGE}" .
else
docker build -f "${DOCKERFILE}" -t "${IMAGE}" .
docker build -f "${DOCKERFILE}" --build-arg RANCHER_PROJECT_MONITORING=$RANCHER_PROJECT_MONITORING -t "${IMAGE}" .
fi

echo "Completed building ${IMAGE} container image"

0 comments on commit 23d0064

Please sign in to comment.