Project

Profile

Help

Story #1176

closed

As a user, I can remove Manifests from repositories

Added by rbarlow over 7 years ago. Updated almost 4 years 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.


Related issues

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

Actions
Actions #1

Updated by rbarlow over 7 years ago

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

Updated by jortel@redhat.com over 7 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.

Actions #3

Updated by jortel@redhat.com over 7 years ago

Included in #1203.

Actions #4

Updated by jortel@redhat.com over 7 years ago

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

Updated by jortel@redhat.com over 7 years ago

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

Actions #6

Updated by jortel@redhat.com over 7 years ago

  • Status changed from ASSIGNED to CLOSED - DUPLICATE

Included in #1203

Actions #7

Updated by bmbouter almost 4 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF