Skip to content

Commit

Permalink
Stop pulling Antrea images in ci/kind/test-e2e-kind.sh (antrea-io#5297)
Browse files Browse the repository at this point in the history
The images should not be pulled, as we want to use the ones we have
built locally. The reason why this has not been an issue so far for CI
is that all jobs us the "coverage" images, which are not available in
the Docker registry (but the logs show error messages because of the
failed docker pull commands).

Signed-off-by: Antonin Bas <[email protected]>
  • Loading branch information
antoninbas authored Jul 21, 2023
1 parent 396cbd6 commit 84879ee
Showing 1 changed file with 13 additions and 9 deletions.
22 changes: 13 additions & 9 deletions ci/kind/test-e2e-kind.sh
Original file line number Diff line number Diff line change
Expand Up @@ -183,15 +183,25 @@ FLOW_VISIBILITY_IMAGE_LIST=("projects.registry.vmware.com/antrea/ipfix-collector
"projects.registry.vmware.com/antrea/clickhouse-operator:0.21.0" \
"projects.registry.vmware.com/antrea/metrics-exporter:0.21.0" \
"projects.registry.vmware.com/antrea/clickhouse-server:23.4")
if $proxy_all; then
COMMON_IMAGES_LIST+=("registry.k8s.io/echoserver:1.10")
fi
# Silence CLI suggestions.
export DOCKER_CLI_HINTS=false
for image in "${COMMON_IMAGES_LIST[@]}"; do
for i in `seq 3`; do
docker pull $image && break
sleep 1
done
done

# The Antrea images should not be pulled, as we want to use the local build.
if $coverage; then
manifest_args="$manifest_args --coverage"
COMMON_IMAGES_LIST+=("antrea/antrea-ubuntu-coverage:latest")
else
COMMON_IMAGES_LIST+=("antrea/antrea-ubuntu:latest")
fi
if $proxy_all; then
COMMON_IMAGES_LIST+=("registry.k8s.io/echoserver:1.10")
fi
if $flow_visibility; then
COMMON_IMAGES_LIST+=("${FLOW_VISIBILITY_IMAGE_LIST[@]}")
if $coverage; then
Expand All @@ -200,12 +210,6 @@ if $flow_visibility; then
COMMON_IMAGES_LIST+=("antrea/flow-aggregator:latest")
fi
fi
for image in "${COMMON_IMAGES_LIST[@]}"; do
for i in `seq 3`; do
docker pull $image && break
sleep 1
done
done

printf -v COMMON_IMAGES "%s " "${COMMON_IMAGES_LIST[@]}"

Expand Down

0 comments on commit 84879ee

Please sign in to comment.