Project

Profile

Help

Refactor #4178

Update sync to use ContentSaver Stage

Added by amacdona@redhat.com over 1 year ago. Updated 6 months ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Sprint/Milestone:
Start date:
Due date:
% Done:

100%

Estimated time:
Platform Release:
Groomed:
Yes
Sprint Candidate:
Yes
Tags:
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.CLOSED - CURRENTRELEASE<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>
Blocked by Container Support - Refactor #4173: Change the multilayered design to use Futures to handle nested contentCLOSED - CURRENTRELEASE

<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>
Copied from Container Support - Refactor #4177: Update sync to use ArtifactSaver StageCLOSED - CURRENTRELEASE

<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

History

#1 Updated by amacdona@redhat.com over 1 year ago

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

#2 Updated by amacdona@redhat.com over 1 year ago

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

#3 Updated by amacdona@redhat.com over 1 year 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 over 1 year 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 over 1 year ago

  • Sprint changed from Sprint 46 to Sprint 47

#6 Updated by ipanova@redhat.com over 1 year 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 over 1 year ago

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

#8 Updated by ipanova@redhat.com over 1 year ago

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

#9 Updated by ipanova@redhat.com over 1 year ago

  • Status changed from ASSIGNED to POST

#10 Updated by rchan over 1 year ago

  • Sprint changed from Sprint 50 to Sprint 51

#11 Updated by ipanova@redhat.com over 1 year ago

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

Applied in changeset commit:pulp_docker|568c4a350964a91cb8c4fedd86e255e29537c19e.

#12 Updated by bmbouter about 1 year ago

  • Tags deleted (Pulp 3)

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

  • Project changed from Docker Support to Container Support

#14 Updated by bmbouter 7 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

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

  • Sprint/Milestone set to 1.0.0

Please register to edit this issue

Also available in: Atom PDF