Project

Profile

Help

Task #7172

closed

Delete a distribution along with a push repository

Added by lmjachky over 4 years ago. Updated over 4 years ago.

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

100%

Estimated time:
Platform Release:
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Sprint 79
Quarter:

Description

When a push repository is going to be deleted, a distribution that references this repository needs to be deleted as well.

Question remains: Should we DELETE a push repository via the pulp api? We should enable users to remove repos however there might be a race condition - a push to the repo X can happen while in the middle of push operation repo X can be deleted.


Related issues

Related to Container Support - Story #7013: Disallow update/partial_update of a push repo via pulp apiCLOSED - CURRENTRELEASElmjachky

Actions
Actions #1

Updated by lmjachky over 4 years ago

  • Related to Story #7013: Disallow update/partial_update of a push repo via pulp api added
Actions #2

Updated by lmjachky over 4 years ago

  • Description updated (diff)
Actions #3

Updated by dkliban@redhat.com over 4 years ago

Yes, we should only allow users to delete Push repositories via Pulp's REST API. If the delete occurs while a user is pushing to this repository, at some point the user will start getting a 404 response from the registry API.

Actions #4

Updated by ipanova@redhat.com over 4 years ago

  • Sprint set to Sprint 78
Actions #5

Updated by mdellweg over 4 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to mdellweg
Actions #6

Updated by pulpbot over 4 years ago

  • Status changed from ASSIGNED to POST
Actions #7

Updated by rchan over 4 years ago

  • Sprint changed from Sprint 78 to Sprint 79

Added by mdellweg over 4 years ago

Revision 9cf3853f | View on GitHub

Delete push repositories with their distribution

Both entities can be used to trigger the deletion, and the deletions are coordinated in an atomic transaction.

fixes #7172 https://pulp.plan.io/issues/7172

Added by mdellweg over 4 years ago

Revision 9cf3853f | View on GitHub

Delete push repositories with their distribution

Both entities can be used to trigger the deletion, and the deletions are coordinated in an atomic transaction.

fixes #7172 https://pulp.plan.io/issues/7172

Actions #8

Updated by mdellweg over 4 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100
Actions #9

Updated by pulpbot over 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF