jaegertracing / jaeger

CNCF Jaeger, a Distributed Tracing Platform

Home Page:https://www.jaegertracing.io/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

[test] Crossdock tests do not need to build Docker images for all architectures

yurishkuro opened this issue · comments

I just noticed in the crossdock CI test logs:

#17 [linux/arm64 cert 2/2] RUN apk add --update --no-cache ca-certificates mailcap
#17 2.550 Executing ca-certificates-20240226-r0.trigger
#17 ...
#18 [linux/ppc64le stage-1 2/3] COPY --from=cert /etc/ssl/certs/ca-certificates.crt /etc/ssl/certs/ca-certificates.crt
#18 DONE 0.0s
#19 [linux/ppc64le stage-1 3/3] COPY --from=cert /etc/mime.types /etc/mime.types
#19 DONE 0.0s
#20 [linux/s390x cert 2/2] RUN apk add --update --no-cache ca-certificates mailcap

It means we're building images for arm64, ppc64le, s390x, etc. while we only need arm64 linux images for this test. We already have a shortcut for not building other architectures in the docker-images CI workflow for PRs, we should do the same for crossdock in all situations.

run: bash scripts/build-upload-docker-images.sh pr-only