Project

General

Profile

Actions

Bug #36538

closed

Removing manifest referenced by tag doesn't remove it from pulp but deletes it from katello

Added by Samir Jha 12 months ago. Updated 9 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Repositories
Target version:
Fixed in Releases:
Found in Releases:

Description

We allow deletions of Manifests from repository in Katello today if it is not part of any manifest lists. However, if a manifest has a tag, we allow users to delete it from the UI. Probably from API and hammer too.

Pulp skips deletion of Manifests with tags which leaves Katello DB out of sync with pulp when such manifests are removed.

Proposed solution:
We should not allow deletion of Manifests in Katello which are referenced by tags. Alternatively, we should delete the tag for the manifest in pulp. However it should be clearly specified to user that the operation will delete more than what's selected on the UI if we do so.

Alternatively, we can disallow deletion of manifests that are tagged like Pulp does. Follow the same pattern as manifests that are part of manifest lists.

Actions #1

Updated by Samir Jha 12 months ago

  • Target version set to Katello 4.10.0
  • Triaged changed from No to Yes
Actions #2

Updated by Samir Jha 12 months ago

  • Category changed from Container to Repositories
Actions #3

Updated by The Foreman Bot 10 months ago

  • Status changed from New to Ready For Testing
  • Assignee set to Samir Jha
  • Pull request https://github.com/Katello/katello/pull/10711 added
Actions #4

Updated by The Foreman Bot 10 months ago

  • Pull request https://github.com/Katello/katello/pull/10712 added
Actions #5

Updated by Samir Jha 10 months ago

  • Pull request deleted (https://github.com/Katello/katello/pull/10712)
Actions #6

Updated by The Foreman Bot 10 months ago

  • Fixed in Releases Katello 4.11.0 added
Actions #7

Updated by Samir Jha 10 months ago

  • Status changed from Ready For Testing to Closed
Actions #8

Updated by The Foreman Bot 9 months ago

  • Pull request https://github.com/Katello/katello/pull/10759 added
Actions #9

Updated by The Foreman Bot 9 months ago

  • Fixed in Releases Katello 4.10.0 added
Actions

Also available in: Atom PDF