Skip to content

Commit eb0112e

Browse files
authored
Merge pull request #1140 from hakman/automated-cherry-pick-of-#1122-#1123-#1139-upstream-release-0.8
Automated cherry pick of #1122: Use cross-compile for Docker builds #1123: Update Cloud Build config #1139: Update Go to v1.24.7
2 parents 89ba541 + 6bd1b9c commit eb0112e

File tree

5 files changed

+12
-26
lines changed

5 files changed

+12
-26
lines changed

Dockerfile

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,15 +19,15 @@
1919
# Must override builder-base, not builder, since the latter is referred to later in the file and so must not be
2020
# directly replaced. See here, and note that "stage" parameter mentioned there has been renamed to
2121
# "build-context": https://github.com/docker/buildx/pull/904#issuecomment-1005871838
22-
FROM golang:1.24.6-bookworm@sha256:bdc7cfd953b2701fcd95fd591ea3d788f41e4b74f21f1787b9f9843a28e72196 AS builder-base
23-
FROM builder-base AS builder
22+
FROM golang:1.24.7-bookworm@sha256:3ce988c30fa67dc966ca716ee0ce7ad08d7330573e808cb68ada7e419bdf23de AS builder-base
23+
FROM --platform=$BUILDPLATFORM builder-base AS builder
2424

2525
ARG TARGETARCH
2626

2727
ENV GOPATH=/gopath/
2828
ENV PATH=$GOPATH/bin:$PATH
2929

30-
RUN apt-get update --fix-missing && apt-get --yes install libsystemd-dev gcc-aarch64-linux-gnu
30+
RUN apt-get update --fix-missing && apt-get --yes install libsystemd-dev
3131
RUN go version
3232

3333
COPY . /gopath/src/k8s.io/node-problem-detector/

Makefile

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ DOCKER_PLATFORMS=linux/amd64,linux/arm64
2929
PLATFORMS=$(LINUX_PLATFORMS) windows_amd64
3030

3131
# VERSION is the version of the binary.
32-
VERSION?=$(shell if [ -d .git ]; then echo `git describe --tags --dirty`; else echo "UNKNOWN"; fi)
32+
VERSION?=$(shell git describe --tags --always --dirty)
3333

3434
# TAG is the tag of the container image, default to binary version.
3535
TAG?=$(VERSION)
@@ -275,10 +275,6 @@ build-in-docker: clean docker-builder
275275
-c 'cd /gopath/src/k8s.io/node-problem-detector/ && make build-binaries'
276276

277277
push-container: build-container
278-
# So we can push to docker hub by setting REGISTRY
279-
ifneq (,$(findstring gcr.io,$(REGISTRY)))
280-
gcloud auth configure-docker
281-
endif
282278
# Build should be cached from build-container
283279
docker buildx build --push --platform $(DOCKER_PLATFORMS) -t $(IMAGE) --build-arg LOGCOUNTER=$(LOGCOUNTER) .
284280

cloudbuild.yaml

Lines changed: 6 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -5,22 +5,12 @@ timeout: 3600s
55
options:
66
# job builds a multi-arch docker image for amd64 and arm64
77
machineType: E2_HIGHCPU_8
8+
substitution_option: ALLOW_LOOSE
89
steps:
9-
- name: 'gcr.io/k8s-staging-test-infra/gcb-docker-gcloud:v20230623-56e06d7c18'
10-
entrypoint: bash
11-
env:
12-
- PROW_GIT_TAG=$_GIT_TAG
13-
- PULL_BASE_REF=$_PULL_BASE_REF
14-
- VERSION=$_PULL_BASE_REF
15-
- DOCKER_CLI_EXPERIMENTAL=enabled
10+
- name: 'gcr.io/cloud-builders/docker'
11+
entrypoint: make
1612
args:
17-
- -c
18-
- |
19-
echo "Building/Pushing NPD containers"
20-
apk add musl-dev gcc
21-
make push-container
13+
- push-container
2214
substitutions:
23-
# _GIT_TAG will be filled with a git-based tag for the image, of the form vYYYYMMDD-hash, and
24-
# can be used as a substitution
25-
_GIT_TAG: 'PLACE_HOLDER'
26-
_PULL_BASE_REF: 'master'
15+
_GIT_TAG: 'dev'
16+
_PULL_BASE_REF: 'dev'

go.mod

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
module k8s.io/node-problem-detector
22

3-
go 1.24.6
3+
go 1.24.7
44

55
require (
66
cloud.google.com/go/compute/metadata v0.8.0

test/go.mod

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
module k8s.io/node-problem-detector/test
22

3-
go 1.24.6
3+
go 1.24.7
44

55
replace k8s.io/node-problem-detector => ../.
66

0 commit comments

Comments
 (0)