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

Delete tags with API v2 (fixes #106) #167

Open
wants to merge 2 commits into
base: v2
Choose a base branch
from
Open

Conversation

xgin
Copy link

@xgin xgin commented May 1, 2017

Use API v2 to delete tags.
In particular GET /manifest by tag to obtain the correct digest to delete and then DELETE /blob by the digest. The sequence is described here: https://docs.docker.com/registry/spec/api/#deleting-an-image

Note that the API does not remove the image from file system, you have to execute garbage-collector, e.g. with crontab

I did not find how to delete repository, because the API DELETE /repository does not work.

Issue for reference #106

@xgin xgin changed the title Delete tags with API v2 Delete tags with API v2 (fixes #106) May 1, 2017
@eidam
Copy link

eidam commented May 3, 2017

any chance to get this merged? 😄

@pkcakeout
Copy link

pkcakeout commented May 4, 2017

Travis tells us that NPM does not want us to have nice things... :-( Good effort though @xgin 👍 !

We probably need to wait until somebody can help out and fix the NPM dependencies before this can be merged...

@xgin
Copy link
Author

xgin commented May 4, 2017

Travis is waiting for #162 to be merged.
Feel free to checkout my fork https://github.com/xgin/docker-registry-frontend and image https://store.docker.com/community/images/ekazakov/docker-registry-frontend

@thiagowfx
Copy link

thiagowfx commented Jul 6, 2017

#162 has just been merged. How can we force a travis rebuild through the github UI? Oh, actually, we'd need to rebase first.

@odoucet
Copy link

odoucet commented Jul 13, 2017

it is now ready to be merged :) Can it be done ? This is a very awaited feature !

@slavniyteo
Copy link

Why it isn't merged yet? Lot of people is waiting for this merge =)

@stijndehaes
Copy link

Still no progress?

@quangnhut123
Copy link

Is it still in process ?

@skygragon
Copy link

Guys, are we blocking on review for this?

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Very important feature! PR looks good 👍

@healermagnus
Copy link

hoping to see this soon

@Nitrosito
Copy link

Why not merged yet? :(
This is a important feature in V2

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.

None yet