Skip to content

Commit 625bdab

Browse files
authored
fix(ci): Provide correct build info for release builds (#15939)
Then building a binary or image release for an upcoming version from a release branch, we provide the version using `--build-arg IMAGE_TAG=...` in the Docker build step of the CI workflow ([ref](https://github.com/grafana/loki/blob/3df08bd39f45bbf8314f31a59f35b955b9a31a5e/.github/workflows/minor-release-pr.yml#L456)). However, this argument was ignored in the Dockerfile and not further passed down to the `make ...` command inside build phase of the Dockerfile. Signed-off-by: Christian Haudum <christian.haudum@gmail.com>
1 parent 07ccf7c commit 625bdab

File tree

5 files changed

+11
-6
lines changed

5 files changed

+11
-6
lines changed

‎Makefile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ OPERATOR_IMAGE := $(IMAGE_PREFIX)/loki-operator:$(IMAGE_TAG)
6666

6767
# OCI (Docker) setup
6868
OCI_PLATFORMS := --platform=linux/amd64,linux/arm64
69-
OCI_BUILD_ARGS := --build-arg GO_VERSION=$(GO_VERSION) --build-arg BUILD_IMAGE=$(BUILD_IMAGE)
69+
OCI_BUILD_ARGS := --build-arg GO_VERSION=$(GO_VERSION) --build-arg BUILD_IMAGE=$(BUILD_IMAGE) --build-arg IMAGE_TAG=$(IMAGE_TAG)
7070
OCI_PUSH_ARGS := -o type=registry
7171
OCI_PUSH := docker push
7272
OCI_TAG := docker tag
@@ -612,7 +612,7 @@ loki-local-image: ## build the loki docker image locally (set LOCAL_ARCH=linux/a
612612

613613
# Canary image
614614
loki-canary-image: ## build the canary docker image
615-
$(OCI_BUILD) -t $(LOKI_CANARY_IMAGE) -f cmd/loki-canary/Dockerfile .
615+
$(OCI_BUILD) -t $(CANARY_IMAGE) -f cmd/loki-canary/Dockerfile .
616616
loki-canary-boringcrypto-image:
617617
$(OCI_BUILD) -t $(IMAGE_PREFIX)/loki-canary-boringcrypto:$(IMAGE_TAG) -f cmd/loki-canary-boringcrypto/Dockerfile .
618618

‎clients/cmd/promtail/Dockerfile

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,12 @@
11
ARG GO_VERSION=1.23
2+
ARG IMAGE_TAG
3+
24
FROM golang:${GO_VERSION}-bookworm as build
35

46
COPY . /src/loki
57
WORKDIR /src/loki
68
RUN apt-get update && apt-get install -qy libsystemd-dev
7-
RUN make clean && make BUILD_IN_CONTAINER=false PROMTAIL_JOURNAL_ENABLED=true promtail
9+
RUN make clean && make BUILD_IN_CONTAINER=false PROMTAIL_JOURNAL_ENABLED=true IMAGE_TAG=${IMAGE_TAG} promtail
810

911
# Promtail requires debian or ubuntu as the base image to support systemd journal reading
1012
FROM public.ecr.aws/ubuntu/ubuntu:noble

‎cmd/loki-canary-boringcrypto/Dockerfile

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,11 @@
11
ARG GO_VERSION=1.23
2+
ARG IMAGE_TAG
23
FROM golang:${GO_VERSION} as build
34

45
COPY . /src/loki
56
WORKDIR /src/loki
67
RUN go env GOARCH > /goarch
7-
RUN make clean && make GOARCH=$(cat /goarch) BUILD_IN_CONTAINER=true GOEXPERIMENT=boringcrypto loki-canary-boringcrypto
8+
RUN make clean && make GOARCH=$(cat /goarch) BUILD_IN_CONTAINER=true GOEXPERIMENT=boringcrypto IMAGE_TAG=${IMAGE_TAG} loki-canary-boringcrypto
89

910
FROM gcr.io/distroless/base-nossl:debug
1011
COPY --from=build /src/loki/cmd/loki-canary-boringcrypto/loki-canary-boringcrypto /usr/bin/loki-canary

‎cmd/loki-canary/Dockerfile

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,10 @@
11
ARG GO_VERSION=1.23
2+
ARG IMAGE_TAG
23
FROM golang:${GO_VERSION} AS build
34

45
COPY . /src/loki
56
WORKDIR /src/loki
6-
RUN make clean && make BUILD_IN_CONTAINER=false loki-canary
7+
RUN make clean && make BUILD_IN_CONTAINER=false IMAGE_TAG=${IMAGE_TAG} loki-canary
78

89
FROM gcr.io/distroless/static:debug
910

‎cmd/loki/Dockerfile

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
ARG GO_VERSION=1.23
2+
ARG IMAGE_TAG
23

34
# UI build stage
45
FROM node:20-alpine AS ui-builder
@@ -12,7 +13,7 @@ FROM golang:${GO_VERSION} AS build
1213
COPY . /src/loki
1314
COPY --from=ui-builder /src/loki/pkg/dataobj/explorer/dist /src/loki/pkg/dataobj/explorer/dist
1415
WORKDIR /src/loki
15-
RUN make clean && make BUILD_IN_CONTAINER=false loki
16+
RUN make clean && make BUILD_IN_CONTAINER=false IMAGE_TAG=${IMAGE_TAG} loki
1617

1718
# Final stage
1819
FROM gcr.io/distroless/static:debug

0 commit comments

Comments
 (0)