Issue #3878
closed
Published metadata path is saved to None path
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:
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().
Updated by jortel@redhat.com over 6 years ago
daviddavis@redhat.com wrote:
I believe we should be able to simply substitute
model.pkwith something likeuuid4().
Seems like a good solution.
I don't know of any need for linkage back to the PublishedMetadata in the DB.
Updated by CodeHeeler over 6 years ago
- Triaged changed from No to Yes
- Sprint set to Sprint 40
Updated by jortel@redhat.com over 6 years ago
- Status changed from NEW to ASSIGNED
- Assignee set to jortel@redhat.com
Added by jortel@redhat.com over 6 years ago
Added by jortel@redhat.com over 6 years ago
Revision 5caa01f4 | View on GitHub
Fix storage path for published metadata. closes #3878
Updated by jortel@redhat.com over 6 years ago
- Status changed from ASSIGNED to POST
Updated by jortel@redhat.com over 6 years ago
- Status changed from POST to MODIFIED
Applied in changeset pulp|5caa01f46584e7d65187ba00b526c072bb847950.
Updated by bmbouter about 5 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
.
Fix storage path for published metadata. closes #3878