Project

Profile

Help

Issue #3878

closed

Published metadata path is saved to None path

Added by daviddavis over 6 years ago. Updated almost 5 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Category:
-
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 41
Quarter:

Description

In task #3848, we converted ids from being UUIDs to integer IDs. The advantage of UUIDs is that they can be set before the model is saved. We rely on this here when setting the path to the published metadata:

https://github.com/pulp/pulp/blob/b3adca1dde4658f87c6dd9109fd5969617845203/pulpcore/pulpcore/app/models/storage.py#L144

However, now that we're using int ids (which are set after save), published metadata is getting saved to /var/lib/pulp/published/metadata/None/.

I believe we should be able to simply substitute model.pk with something like uuid4().

Actions #1

Updated by jortel@redhat.com over 6 years ago

wrote:

I believe we should be able to simply substitute model.pk with something like uuid4().

Seems like a good solution.

I don't know of any need for linkage back to the PublishedMetadata in the DB.

Actions #2

Updated by CodeHeeler over 6 years ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 40
Actions #3

Updated by jortel@redhat.com over 6 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to jortel@redhat.com
Actions #4

Updated by rchan over 6 years ago

  • Sprint changed from Sprint 40 to Sprint 41

Added by jortel@redhat.com over 6 years ago

Revision 5caa01f4 | View on GitHub

Fix storage path for published metadata. closes #3878

Added by jortel@redhat.com over 6 years ago

Revision 5caa01f4 | View on GitHub

Fix storage path for published metadata. closes #3878

Actions #5

Updated by jortel@redhat.com over 6 years ago

  • Status changed from ASSIGNED to POST
Actions #6

Updated by jortel@redhat.com over 6 years ago

  • Status changed from POST to MODIFIED
Actions #7

Updated by daviddavis over 5 years ago

  • Sprint/Milestone set to 3.0.0
Actions #8

Updated by bmbouter over 5 years ago

  • Tags deleted (Pulp 3)
Actions #9

Updated by bmbouter almost 5 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF