Project

Profile

Help

Actions
Send by e-mail Copy link

Refactor #4178

closed

Update sync to use ContentSaver Stage

Added by amacdona@redhat.com about 6 years ago. Updated almost 5 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
ipanova@redhat.com
Sprint/Milestone:
1.0.0
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
Copy link
 #1

Updated by amacdona@redhat.com about 6 years ago

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

Updated by amacdona@redhat.com about 6 years ago

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

Updated by amacdona@redhat.com about 6 years ago

  • Tags Pulp 3 RC Blocker added

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

Actions
Copy link
 #4

Updated by ipanova@redhat.com almost 6 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
Copy link
 #5

Updated by rchan almost 6 years ago

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

Updated by ipanova@redhat.com almost 6 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
Copy link
 #7

Updated by amacdona@redhat.com almost 6 years ago

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

Updated by ipanova@redhat.com almost 6 years ago

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

Updated by ipanova@redhat.com almost 6 years ago

  • Status changed from ASSIGNED to POST
Actions
Copy link
 #10

Updated by rchan over 5 years ago

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

Updated by ipanova@redhat.com over 5 years ago

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

Applied in changeset commit:pulp_docker|568c4a350964a91cb8c4fedd86e255e29537c19e.

Actions
Copy link
 #12

Updated by bmbouter over 5 years ago

  • Tags deleted (Pulp 3)
Actions
Copy link
 #13

Updated by ipanova@redhat.com about 5 years ago

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

Updated by bmbouter about 5 years ago

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

Updated by ipanova@redhat.com almost 5 years ago

  • Sprint/Milestone set to 1.0.0
Actions
Send by e-mail Copy link

Also available in: Atom PDF