Skip to content
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

kola: Use github container registry for test images #210

Merged
merged 2 commits into from
Aug 17, 2021

Conversation

jepio
Copy link
Member

@jepio jepio commented Aug 17, 2021

Use github container registry for test imagees

We're hitting dockerhub pull rate limits in our CI. Most frequently it occurs with busybox and nginx images so I've mirrored those first. The nginx one is probably going to be pulled most often: the kubeadm tests run for 3 CNIs and multiple nodes each.

The mirroring was done with the following script:

#!/bin/bash

image=$1

platforms=( amd64 arm64 )
tags=()

for platform in "${platforms[@]}"; do
  var=$(docker pull $image --platform=$platform -q)
  tag=ghcr.io/kinvolk/$image:latest-$platform
  docker tag $var $tag
  docker push $tag
  tags+=( $tag )
done

docker manifest create ghcr.io/kinvolk/$image:latest "${tags[@]}"
docker manifest push --purge ghcr.io/kinvolk/$image:latest

Testing done

./bin/kola run --channel=alpha cl.internet docker.selinux

Signed-off-by: Jeremi Piotrowski <jpiotrowski@microsoft.com>
Signed-off-by: Jeremi Piotrowski <jpiotrowski@microsoft.com>
@jepio jepio requested review from tormath1 and a team August 17, 2021 09:03
Copy link
Contributor

@tormath1 tormath1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM ! Thanks :)

@jepio jepio merged commit a8649de into flatcar-master Aug 17, 2021
@jepio jepio deleted the jepio/change-container-mirror branch August 17, 2021 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants