Project

Profile

Help

Story #5179

closed

As a user I can recursively remove docker content units.

Added by ipanova@redhat.com over 5 years ago. Updated almost 5 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Sprint/Milestone:
Start date:
Due date:
% Done:

100%

Estimated time:
Platform Release:
Groomed:
Yes
Sprint Candidate:
No
Tags:
Sprint:
Sprint 58
Quarter:

Description

Workflow

Removal of docker content unit would allow removal of the content unit in question if no content unit on a higher level references it. If it can be removed, all (otherwise unused) content units on lower levels are removed recursively.

  • When a docker image manifest is removed, all its blobs( not referenced by other image manifests) are removed as well. If docker manifest is referenced by tags, or by manifest lists, removal is not possible.
  • When a docker manifest list is removed, all its manifests( not referenced by other manifests lists and not tagged) are removed as well. Furthermore, same story with the blobs. If docker manifest list is referenced by tags, removal is not possible unless force flag is specified.
  • (Behavior change from pulp 2) When a tag is removed, all unreferenced content is also removed. If a tagged manifest/manifest_list has other tags or other references, only the tag is removed.
  • Removal of blobs will not be possible unless it has no references.

Endpoint
POST v3/docker/recursive-remove content_hrefs=[] repo/repo_version=X

Notes
When removal is not possible because of higher level references, a list of _hrefs of those references will be returned along with the user friendly message.


Related issues

Related to Container Support - Story #5232: As a user I can remove docker content units with a force flagCLOSED - WONTFIX

Actions
Related to Container Support - Story #5322: As a user I have retuned a list of references that prevent recursive removalCLOSED - WONTFIX

Actions
Actions #1

Updated by ipanova@redhat.com over 5 years ago

  • Description updated (diff)
Actions #2

Updated by ipanova@redhat.com over 5 years ago

  • Subject changed from As a user I can remove and forcely remove docker content units. to As a user I can remove and forcefully remove docker content units.
  • Description updated (diff)
Actions #3

Updated by ipanova@redhat.com over 5 years ago

  • Description updated (diff)
Actions #4

Updated by gmbnomis over 5 years ago

Thanks for writing this up! I have just two comments:

  • For the sake of completeness, we should specify what happens on blob removal. Proposal: without "force" it works analogously to the other artifact types, i.e. removal is not possible if any manifest refers to it. "force" is not supported for blobs(?) (I think this operation does not make much sense)
  • For the regular case (not forced): If the removal of an artifact is not possible because there are references to the artifact, what information about these references do we give back to the caller? (no information at all, single reference that caused the error, all references that point to the artifact. I have no idea about the overhead created by these options)

Question

When we remove tag with force flag do we want to recursively remove content units on lower levels?

I think we don't. This functionality looks unrelated to "force". For all other artifact type "force" says "delete on higher levels as well". If there is a need for recursive delete for tags, I tend to introduce another flag for it.

Actions #5

Updated by ipanova@redhat.com over 5 years ago

gmbnomis wrote:

Thanks for writing this up! I have just two comments:

  • For the sake of completeness, we should specify what happens on blob removal. Proposal: without "force" it works analogously to the other artifact types, i.e. removal is not possible if any manifest refers to it. "force" is not supported for blobs(?) (I think this operation does not make much sense)

Force flag will not be applicable for blobs. In general the plan is to disallow removal of blobs, unless they are orphans.

  • For the regular case (not forced): If the removal of an artifact is not possible because there are references to the artifact, what information about these references do we give back to the caller? (no information at all, single reference that caused the error, all references that point to the artifact. I have no idea about the overhead created by these options)

Probably just the message that it is referenced in other manifests lists or that it has tags, therefore regular removal is not possible. Will it be useful to the user to see the list of digests the artifact is referenced in?

We can keep this in mind, and see that's the overhead whenever we get to the implementation.

Question

When we remove tag with force flag do we want to recursively remove content units on lower levels?

I think we don't. This functionality looks unrelated to "force". For all other artifact type "force" says "delete on higher levels as well". If there is a need for recursive delete for tags, I tend to introduce another flag for it.

Actions #6

Updated by gmbnomis over 5 years ago

wrote:

gmbnomis wrote:

  • For the regular case (not forced): If the removal of an artifact is not possible because there are references to the artifact, what information about these references do we give back to the caller? (no information at all, single reference that caused the error, all references that point to the artifact. I have no idea about the overhead created by these options)

Probably just the message that it is referenced in other manifests lists or that it has tags, therefore regular removal is not possible. Will it be useful to the user to see the list of digests the artifact is referenced in?

I was thinking about returning the _href to the artifacts, not the digests. For example, I think this could help the user to make an informed decision whether repeating the removal with "force" is a sensible thing to do.

I agree, that makes sense to me.

Actions #7

Updated by ipanova@redhat.com over 5 years ago

  • Description updated (diff)
Actions #8

Updated by ipanova@redhat.com over 5 years ago

  • Subject changed from As a user I can remove and forcefully remove docker content units. to As a user I can recursively remove and forcefully remove docker content units.
  • Description updated (diff)
Actions #9

Updated by ipanova@redhat.com over 5 years ago

  • Description updated (diff)
Actions #10

Updated by ipanova@redhat.com over 5 years ago

  • Related to Story #5232: As a user I can remove docker content units with a force flag added
Actions #11

Updated by ipanova@redhat.com over 5 years ago

  • Subject changed from As a user I can recursively remove and forcefully remove docker content units. to As a user I can recursively remove docker content units.
Actions #12

Updated by ipanova@redhat.com over 5 years ago

  • Description updated (diff)
Actions #13

Updated by amacdona@redhat.com over 5 years ago

  • Description updated (diff)
  • Groomed changed from No to Yes
Actions #14

Updated by amacdona@redhat.com over 5 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to amacdona@redhat.com
Actions #15

Updated by ipanova@redhat.com over 5 years ago

  • Sprint set to Sprint 57
Actions #16

Updated by amacdona@redhat.com over 5 years ago

  • Status changed from ASSIGNED to POST
Actions #17

Updated by ipanova@redhat.com over 5 years ago

  • Related to Story #5322: As a user I have retuned a list of references that prevent recursive removal added
Actions #18

Updated by rchan over 5 years ago

  • Sprint changed from Sprint 57 to Sprint 58
Actions #19

Updated by amacdona@redhat.com about 5 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100

Applied in changeset commit:pulp_docker|b2d9900247a6bb8a83f44af05f7dffd66d7288dc.

Actions #20

Updated by ipanova@redhat.com about 5 years ago

  • Project changed from Docker Support to Container Support
Actions #21

Updated by ipanova@redhat.com almost 5 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Actions #22

Updated by ipanova@redhat.com almost 5 years ago

  • Sprint/Milestone set to 1.0.0

Also available in: Atom PDF