Project

Profile

Help

Refactor #4178

Update sync to use ContentSaver Stage

Added by amacdona@redhat.com 12 months ago. Updated 7 months ago.

Status:
MODIFIED
Priority:
Normal
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
% Done:

100%

Platform Release:
Blocks Release:
Target Release - Docker:
Groomed:
Yes
Sprint Candidate:
Yes
Tags:
Complexity:
Verified:
No
Verification Required:
No
Sprint:
Sprint 51

Description

We can't use the vanilla ContentSaver due to #4085.

Use of that stage should replace the use of the SerialContentSave stage, which should be deleted:
https://github.com/pulp/pulp_docker/blob/master/pulp_docker/app/tasks/dedupe_save.py#L69

After completion of this task, the entire dedupe_save will probably be able to be removed.


Related issues

Related to Pulp - Issue #4085: ContentUnitSaver stage is vulnerable to race conditions. MODIFIED Actions
Blocked by Docker Support - Refactor #4173: Change the multilayered design to use Futures to handle nested content MODIFIED Actions
Copied from Docker Support - Refactor #4177: Update sync to use ArtifactSaver Stage MODIFIED Actions

Associated revisions

History

#1 Updated by amacdona@redhat.com 12 months ago

  • Copied from Refactor #4177: Update sync to use ArtifactSaver Stage added

#2 Updated by amacdona@redhat.com 12 months ago

  • Related to Issue #4085: ContentUnitSaver stage is vulnerable to race conditions. added

#3 Updated by amacdona@redhat.com 12 months ago

  • Tags Pulp 3 RC Blocker added

Adding RC blocker since this is the plugin that made the updated ContentSaver stage necessary.

#4 Updated by ipanova@redhat.com 11 months ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to ipanova@redhat.com
  • Groomed changed from No to Yes
  • Sprint Candidate changed from No to Yes
  • Sprint set to Sprint 46

#5 Updated by rchan 11 months ago

  • Sprint changed from Sprint 46 to Sprint 47

#6 Updated by ipanova@redhat.com 10 months ago

  • Status changed from ASSIGNED to NEW
  • Assignee deleted (ipanova@redhat.com)
  • Sprint deleted (Sprint 47)
  • Tags deleted (Pulp 3 RC Blocker)

https://github.com/pulp/pulp_docker/pull/294 closing this PR for now.
We cannot use ContentUnitSaver stage in docker plugin, because it expects all the artifacts downloaded and saved.
Current docker design skips the contents units if the artifacts have not been downloaded/saved yet https://github.com/pulp/pulp_docker/pull/294/files#diff-db9e748d164d25b2078d659910c68878L42

#7 Updated by amacdona@redhat.com 9 months ago

  • Blocked by Refactor #4173: Change the multilayered design to use Futures to handle nested content added

#8 Updated by ipanova@redhat.com 8 months ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to ipanova@redhat.com
  • Sprint set to Sprint 50

#9 Updated by ipanova@redhat.com 8 months ago

  • Status changed from ASSIGNED to POST

#10 Updated by rchan 8 months ago

  • Sprint changed from Sprint 50 to Sprint 51

#11 Updated by ipanova@redhat.com 8 months ago

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

#12 Updated by bmbouter 7 months ago

  • Tags deleted (Pulp 3)

Please register to edit this issue

Also available in: Atom PDF