Project

Profile

Help

Story #1176

As a user, I can remove Manifests from repositories

Added by rbarlow about 5 years ago. Updated over 1 year ago.

Status:
CLOSED - DUPLICATE
Priority:
Normal
Start date:
Due date:
% Done:

80%

Estimated time:
Platform Release:
Target Release - Docker:
Groomed:
No
Sprint Candidate:
Yes
Tags:
Pulp 2
Sprint:
Quarter:

Description

We need pulp-admin and API support for removing Manifests from repositories. Removing a Manifest should also remove the Blobs it references, unless other Manifests in the same repository also reference those Blobs.


Checklist


Related issues

Blocks Docker Support - Task #1048: Tracker for Docker v2 API and Manifest workCLOSED - CURRENTRELEASE

<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

History

#1 Updated by rbarlow about 5 years ago

  • Blocks Task #1048: Tracker for Docker v2 API and Manifest work added

#2 Updated by jortel@redhat.com about 5 years ago

There is a race condition concern with removing associated blobs when removing a manifest. The blob should only be removed only if not referenced by ANY manifest in the repository. Since associate/disassociate operations happen concurrently this can happen:

process-1: Remove manifest-123 which references blob-1
process-1: See if a blob-1 is referenced by any other manifests. result is - nope.
process-2: Add manifest-1 that references blob-1.
process-2: Add blob-1
process-1: Removes blob-1 (thinks it's not referenced)

Now manifest-1 is in the repository and blob-1 is not.

#3 Updated by jortel@redhat.com about 5 years ago

Included in #1203.

#4 Updated by jortel@redhat.com about 5 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to jortel@redhat.com
  • % Done changed from 0 to 80

#5 Updated by jortel@redhat.com about 5 years ago

Looks like unassociate tasks run with reservation which addresses concurrency concern.

#6 Updated by jortel@redhat.com about 5 years ago

  • Status changed from ASSIGNED to CLOSED - DUPLICATE

Included in #1203

#7 Updated by bmbouter over 1 year ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF