Project

Profile

Help

Refactor #4178

closed

Update sync to use ContentSaver Stage

Added by amacdona@redhat.com over 3 years ago. Updated over 2 years 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
Quarter:

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 - CURRENTRELEASEdkliban@redhat.comActions
Blocked by Container Support - Refactor #4173: Change the multilayered design to use Futures to handle nested contentCLOSED - CURRENTRELEASEipanova@redhat.com

Actions
Copied from Container Support - Refactor #4177: Update sync to use ArtifactSaver StageCLOSED - CURRENTRELEASEipanova@redhat.com

Actions
Actions #1

Updated by amacdona@redhat.com over 3 years ago

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

Updated by amacdona@redhat.com over 3 years ago

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

Updated by amacdona@redhat.com over 3 years ago

  • Tags Pulp 3 RC Blocker added

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

Actions #4

Updated by ipanova@redhat.com over 3 years 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
Actions #5

Updated by rchan over 3 years ago

  • Sprint changed from Sprint 46 to Sprint 47
Actions #6

Updated by ipanova@redhat.com over 3 years 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

Actions #7

Updated by amacdona@redhat.com over 3 years ago

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

Updated by ipanova@redhat.com over 3 years ago

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

Updated by ipanova@redhat.com over 3 years ago

  • Status changed from ASSIGNED to POST
Actions #10

Updated by rchan over 3 years ago

  • Sprint changed from Sprint 50 to Sprint 51
Actions #11

Updated by ipanova@redhat.com over 3 years ago

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

Applied in changeset commit:pulp_docker|568c4a350964a91cb8c4fedd86e255e29537c19e.

Actions #12

Updated by bmbouter over 3 years ago

  • Tags deleted (Pulp 3)
Actions #13

Updated by ipanova@redhat.com over 2 years ago

  • Project changed from Docker Support to Container Support
Actions #14

Updated by bmbouter over 2 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Actions #15

Updated by ipanova@redhat.com over 2 years ago

  • Sprint/Milestone set to 1.0.0

Also available in: Atom PDF