Project

Profile

Help

Task #3428

Update content unit creation so it takes only a relative_path and artifact href

Added by daviddavis over 3 years ago. Updated almost 2 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 33
Quarter:

Description

There were a number of bugs around creating file content units:

https://pulp.plan.io/issues/3425 - Path and ContentArtifact.relative_path could be different
https://pulp.plan.io/issues/3422 - Content units could be created with multiple artifacts
https://pulp.plan.io/issues/3423 - Anything could be specified for digest

To resolve these bugs, we decided to only accept path and artifact href when creating file content units.


Related issues

Has duplicate File Support - Issue #3425: Path on content unit doesn't seem to do anythingCLOSED - DUPLICATE<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>
Has duplicate File Support - Issue #3422: It's possible to create a file content unit with multiple artifactsCLOSED - DUPLICATE<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>
Has duplicate File Support - Issue #3423: No validation or documentation for digest field on file contentCLOSED - DUPLICATE<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

Associated revisions

Revision 30afa7cc View on GitHub
Added by daviddavis over 3 years ago

Limit FileContent api fields to path and artifact

This solves a number of issues such as differences between digest on FileContent vs Artifact.sha256, and FileContent.relative_path vs ContentArtifact.relative_path.

This also renames FileContent.path to FileContent.relative_path

fixes #3428, #3410 https://pulp.plan.io/issues/3428 https://pulp.plan.io/issues/3410

History

#1 Updated by daviddavis over 3 years ago

  • Subject changed from Update content unit creation so it takes only a path and artifact href to Update content unit creation so it takes only a relative_path and artifact href

#2 Updated by daviddavis over 3 years ago

  • Has duplicate Issue #3425: Path on content unit doesn't seem to do anything added

#3 Updated by daviddavis over 3 years ago

  • Has duplicate Issue #3422: It's possible to create a file content unit with multiple artifacts added

#4 Updated by daviddavis over 3 years ago

  • Has duplicate Issue #3423: No validation or documentation for digest field on file content added

#5 Updated by daviddavis over 3 years ago

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

#6 Updated by daviddavis over 3 years ago

  • Status changed from ASSIGNED to POST
  • Tags Pulp 3, Pulp 3 MVP added

#7 Updated by daviddavis over 3 years ago

  • Sprint/Milestone set to 56

Setting the sprint to 33 since the fix fixes this bug which is on sprint 33:

https://pulp.plan.io/issues/3410

#8 Updated by daviddavis over 3 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100

#9 Updated by bmbouter over 3 years ago

  • Sprint set to Sprint 33

#10 Updated by bmbouter over 3 years ago

  • Sprint/Milestone deleted (56)

#11 Updated by bmbouter over 2 years ago

  • Tags deleted (Pulp 3, Pulp 3 MVP)

#12 Updated by bmbouter almost 2 years ago

  • Sprint/Milestone set to 0.1.0

#13 Updated by bmbouter almost 2 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF