Skip to content

wip: Cross repository push#2697

Closed
dmcgowan wants to merge 3 commits intocontainerd:masterfrom
dmcgowan:cross-repo-push
Closed

wip: Cross repository push#2697
dmcgowan wants to merge 3 commits intocontainerd:masterfrom
dmcgowan:cross-repo-push

Conversation

@dmcgowan
Copy link
Copy Markdown
Member

@dmcgowan dmcgowan commented Oct 3, 2018

Add label on fetch to manifest to describe source in format containerd.io/distribution.source.<host> = <repo>[,<repo>]
Update authorizer to limit parallel requests
Store token by scope
Use source annotation to set cross repository query parameters

Depends on #2690

@estesp
Copy link
Copy Markdown
Member

estesp commented Oct 3, 2018

Hmm: failed to copy: context canceled: time="2018-10-03T08:18:53.470099229Z"

@dmcgowan dmcgowan changed the title Cross repository push wip: Cross repository push Oct 3, 2018
@dmcgowan
Copy link
Copy Markdown
Member Author

dmcgowan commented Oct 3, 2018

Marking this as WIP for now (some UI weirdness I need to fix), this can be used though to test shallow pulling/pushing

$ ctr content fetch --all-platforms --shallow <your image>
$ ctr images push -u `cat <credential file>` <new image on same registry> <your image>
``

@dmcgowan dmcgowan force-pushed the cross-repo-push branch 2 times, most recently from 077e2be to e597b8a Compare October 11, 2018 00:00
@dmcgowan dmcgowan added this to the 1.3 milestone Oct 18, 2018
Add label on fetch to manifest to describe source
Update authorizer to limit parallel requests
Store token by scope
Use source annotation to set cross repository query parameters

Signed-off-by: Derek McGowan <derek@mcgstyle.net>
Signed-off-by: Derek McGowan <derek@mcgstyle.net>
Use a slash to make the label key more clear which is
the key and which is the host

Signed-off-by: Derek McGowan <derek@mcgstyle.net>
@tonistiigi
Copy link
Copy Markdown
Member

I tried to use this PR but it didn't work. First push was ok but as soon as there were some blobs already in target repo it just deadlocks to whole push forever. This PR also regresses the regular push without any --shallow.

@ehazlett
Copy link
Copy Markdown
Member

@dmcgowan is this still in progress?

@dmcgowan
Copy link
Copy Markdown
Member Author

Yes and no, we still want this for 1.3 but I am not actively working on it. Let me create an issue and replace this

@dmcgowan dmcgowan mentioned this pull request Jan 29, 2019
@dmcgowan
Copy link
Copy Markdown
Member Author

See #2964

@dmcgowan dmcgowan closed this Jan 29, 2019
@dmcgowan dmcgowan mentioned this pull request Feb 25, 2019
@dmcgowan dmcgowan deleted the cross-repo-push branch March 13, 2024 01:55
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