Skip to content

Commit 30bb532

Browse files
committed
fixes for building and testing arm locally
1 parent e9d59d0 commit 30bb532

File tree

3 files changed

+16
-12
lines changed

3 files changed

+16
-12
lines changed

Makefile

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -30,12 +30,12 @@ container-dockerfile-dev:
3030
.PHONY: builder
3131
builder:
3232
ifneq ($(BUILD_BUILDER_IMAGE), false)
33-
docker buildx build --load --platform ${PLATFORM} \
34-
-t quay.io/stackrox-io/collector-builder:$(COLLECTOR_BUILDER_TAG) \
33+
docker buildx build --load --platform $(PLATFORM) \
34+
-t $(COLLECTOR_BUILDER_IMAGE) \
3535
-f "$(CURDIR)/builder/Dockerfile" \
3636
"$(CURDIR)/builder"
3737
else
38-
docker pull --platform ${PLATFORM} quay.io/stackrox-io/collector-builder:$(COLLECTOR_BUILDER_TAG)
38+
docker pull --platform $(PLATFORM) $(COLLECTOR_BUILDER_IMAGE)
3939
endif
4040

4141
collector: check-builder
@@ -59,7 +59,7 @@ build-drivers:
5959

6060
image: collector unittest
6161
make -C collector txt-files
62-
docker buildx build --load --platform ${PLATFORM} \
62+
docker buildx build --load --platform $(PLATFORM) \
6363
--build-arg COLLECTOR_VERSION="$(COLLECTOR_TAG)" \
6464
--build-arg MODULE_VERSION="$(MODULE_VERSION)" \
6565
-f collector/container/Dockerfile \
@@ -68,7 +68,8 @@ image: collector unittest
6868

6969
image-dev: collector unittest container-dockerfile-dev
7070
make -C collector txt-files
71-
docker build --build-arg collector_version="$(COLLECTOR_TAG)" \
71+
docker buildx build --load --platform $(PLATFORM) \
72+
--build-arg collector_version="$(COLLECTOR_TAG)" \
7273
--build-arg BUILD_TYPE=devel \
7374
--build-arg MODULE_VERSION="$(MODULE_VERSION)" \
7475
-f collector/container/Dockerfile.dev \
@@ -77,7 +78,7 @@ image-dev: collector unittest container-dockerfile-dev
7778

7879
image-dev-full: image-dev build-drivers
7980
docker tag quay.io/stackrox-io/collector:$(COLLECTOR_TAG) quay.io/stackrox-io/collector:$(COLLECTOR_TAG)-slim
80-
docker build \
81+
docker buildx build --load --platform $(PLATFORM) \
8182
--target=probe-layer-1 \
8283
--tag quay.io/stackrox-io/collector:$(COLLECTOR_TAG)-full \
8384
--build-arg collector_repo=quay.io/stackrox-io/collector \
@@ -111,11 +112,12 @@ endif
111112
start-builder: builder teardown-builder
112113
docker run -d \
113114
--name $(COLLECTOR_BUILDER_NAME) \
115+
--platform $(PLATFORM) \
114116
-v $(CURDIR):$(CURDIR) \
115117
$(if $(LOCAL_SSH_PORT),-p $(LOCAL_SSH_PORT):22 )\
116118
-w $(CURDIR) \
117119
--cap-add sys_ptrace \
118-
quay.io/stackrox-io/collector-builder:$(COLLECTOR_BUILDER_TAG)
120+
$(COLLECTOR_BUILDER_IMAGE)
119121

120122
.PHONY: check-builder
121123
check-builder:
@@ -128,6 +130,7 @@ teardown-builder:
128130
.PHONY: clean
129131
clean:
130132
rm -rf cmake-build/
133+
rm -rf $(CMAKE_BASE_DIR)
131134
make -C collector clean
132135

133136
.PHONY: shfmt-check

Makefile-constants.mk

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,15 +11,19 @@ COLLECTOR_TAG := $(CIRCLE_TAG)
1111
endif
1212
endif
1313

14+
PLATFORM ?= $(shell docker system info --format {{.OSType}}/{{.Architecture}})
15+
1416
USE_VALGRIND ?= false
1517
ADDRESS_SANITIZER ?= false
1618
CMAKE_BUILD_TYPE ?= Release
19+
CMAKE_BASE_DIR = cmake-build-$(shell echo $(CMAKE_BUILD_TYPE) | tr A-Z a-z)-$(subst /,-,$(PLATFORM))
1720
COLLECTOR_APPEND_CID ?= false
18-
PLATFORM ?= linux/amd64
1921
TRACE_SINSP_EVENTS ?= false
2022
DISABLE_PROFILING ?= false
2123

2224
COLLECTOR_BUILD_CONTEXT = collector/
23-
COLLECTOR_BUILDER_NAME ?= collector_builder
25+
COLLECTOR_BUILDER_NAME ?= collector_builder_$(subst /,_,$(PLATFORM))
26+
COLLECTOR_BUILDER_REPO=quay.io/stackrox-io/collector-builder
27+
COLLECTOR_BUILDER_IMAGE="$(COLLECTOR_BUILDER_REPO):$(COLLECTOR_BUILDER_TAG)"
2428

2529
export COLLECTOR_PRE_ARGUMENTS

collector/Makefile

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ include ../Makefile-constants.mk
33

44
NPROCS ?= $(shell nproc)
55

6-
CMAKE_BASE_DIR = cmake-build
76
CMAKE_DIR= $(BASE_PATH)/$(CMAKE_BASE_DIR)
87
COLLECTOR_BIN_DIR = $(CMAKE_DIR)/collector
98
LIBSINSP_BIN_DIR = $(CMAKE_DIR)/collector/EXCLUDE_FROM_DEFAULT_BUILD/libsinsp
@@ -67,8 +66,6 @@ txt-files:
6766
clean:
6867
docker rm -fv build_collector || true
6968
docker rm -fv container-stats benchmark collector grpc-server || true
70-
rm -rf falcosecurity-libs/cmake-build
71-
rm -rf falcosecurity-libs/falcosecurity-libs-build
7269
rm -rf container/bin
7370
rm -rf container/driver
7471
rm -rf container/libs

0 commit comments

Comments
 (0)