Project

Profile

Help

Issue #7431

Removing distribution tree cause error on repository version

Added by ppicka about 2 months ago. Updated 16 days ago.

Status:
MODIFIED
Priority:
Normal
Assignee:
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Sprint 83
Quarter:

Description

How to reproduce:

  1. sync repository with dist tree e.g. fixtures[0] (got the repo_version and added dist_tree)
  2. remove dist_tree
  3. look (e.g. http GET) for repo version
AttributeError at /pulp/api/v3/repositories/rpm/rpm/6c56dc14-63f5-4ee1-ac1f-a542add7061a/versions/1/
'NoneType' object has no attribute 'cast'

[0] https://fixtures.pulpproject.org/rpm-distribution-tree/

Associated revisions

Revision 5fb7bee6 View on GitHub
Added by ppicka 16 days ago

Comps and disttree removal

User should not be able to remove distribution trees and comps if they are used in repository. If they are not in repo they will be removed with orphan cleanup as other content.

closes: #7431 https://pulp.plan.io/issues/7431

History

#1 Updated by ttereshc about 2 months ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 80

#2 Updated by ppicka about 2 months ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to ppicka

#3 Updated by rchan about 2 months ago

  • Sprint changed from Sprint 80 to Sprint 81

#4 Updated by fao89 about 2 months ago

I'm not sure about the use case for pulp_rpm, but I believe it needs a fix at pulpcore side, we cannot call cast without assuring we have an object

#5 Updated by ppicka about 1 month ago

That is true, but in the time it should be casted, pulp assume content is there and it should. We should not allow delete content when it is used in repository.

#6 Updated by rchan about 1 month ago

  • Sprint changed from Sprint 81 to Sprint 82

#7 Updated by pulpbot 23 days ago

  • Status changed from ASSIGNED to POST

#8 Updated by rchan 21 days ago

  • Sprint changed from Sprint 82 to Sprint 83

#9 Updated by ppicka 16 days ago

  • Status changed from POST to MODIFIED

Please register to edit this issue

Also available in: Atom PDF