Skip to content

Conversation

@mtrmac
Copy link
Collaborator

@mtrmac mtrmac commented Jul 23, 2021

... to avoid docker.io pull limits.

NewImageSource and NewImage actually access the remote server (because NewImageSource is looking for a working mirror); update NewImageDestination basically for symmetry.

@rhatdan
Copy link
Member

rhatdan commented Jul 23, 2021

LGTM

@cevich
Copy link
Member

cevich commented Jul 23, 2021

LGTM also. Those images we copied to quay are used by nearly all of our CI everywhere and everybody understands not to touch them :D

The test-failures here will go away after we synchronize the "skopeo_test" task to the new c/skopeo version...

@cevich
Copy link
Member

cevich commented Jul 23, 2021

...rebase and this will fix the skopeo tests: #1309

@TomSweeneyRedHat
Copy link
Member

LGTM
Looks like the skopeo test is unhappy, but it's calling for a rebase. Perhaps the rebase will pick that up?

@cevich
Copy link
Member

cevich commented Jul 23, 2021

Perhaps the rebase will pick that up?

Ya it's a known issue, rebase will indeed fix it.

... to avoid docker.io pull limits.

NewImageSource and NewImage actually access the remote
server (because NewImageSource is looking for a working mirror);
update NewImageDestination basically for symmetry.

Signed-off-by: Miloslav Trmač <[email protected]>
@mtrmac mtrmac merged commit c3649c8 into containers:main Jul 23, 2021
@mtrmac mtrmac deleted the docker.io branch July 23, 2021 15:41
@mtrmac mtrmac mentioned this pull request Jul 30, 2021
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.

4 participants