-
Notifications
You must be signed in to change notification settings - Fork 362
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Avoid rate limit issue on kind test bed #6676
base: main
Are you sure you want to change the base?
Conversation
c819cb8
to
7fe3f36
Compare
For e2e tests on kind there are also common set of images which are pulled every-time like registry.k8s.io/e2e-test-images/agnhost:2.40" |
476df1c
to
71166a5
Compare
a5bf1cb
to
b169ffc
Compare
/test-kind-all |
b169ffc
to
07935e8
Compare
/test-kind-ipv6-e2e |
@@ -201,8 +201,13 @@ function clean_antrea { | |||
for antrea_yml in ${WORKDIR}/*.yml; do | |||
kubectl delete -f $antrea_yml --ignore-not-found=true || true | |||
done | |||
docker images --format "{{.Repository}}:{{.Tag}}" | grep 'antrea'| xargs -r docker rmi -f || true | |||
docker images | grep '<none>' | awk '{print $3}' | xargs -r docker rmi || true | |||
if [[ $TESTBED_TYPE == "kind-flexible-ipam" || $TESTBED_TYPE == "kind" ]]; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is the cleanup logic different based on the testbed type?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same question, is it related to the rate limit issue?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@antoninbas @XinShuYang yes, no need to delete base images from the kind testbed. during run it will use images available on kind testbed no need to pull it again from docker..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It feels like such a change does not need to be specific to Kind tests?
However, one thing to keep in mind is that we always try to build the Antrea image with the latest available version of the base images (be it ubuntu or golang) in CI. To achieve that, a docker pull
is essentially required and it may be subject to rate limiting, regardless of whether the correct version is already present or not - but maybe we get penalized less if the image is already present (fewer requests to the docker API?).
Signed-off-by: KMAnju-2021 <[email protected]>
07935e8
to
979830f
Compare
No description provided.