Project

Profile

Help

Story #2507

Story #3693: Lazy for Pulp3

The streamer needs to try all RemoteArtifact entires related to a ContentArtifact

Added by jortel@redhat.com over 3 years ago. Updated 7 months ago.

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

100%

Estimated time:
Platform Release:
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Sprint 48

Description

The long term design for the streamer was to sort the entries newest to oldest. Then, try them in order and only fail when all entries have failed.


Related issues

Related to Pulp - Issue #4246: Lazy sync does not create additional RemoteArtifact objectsCLOSED - CURRENTRELEASE<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>
Copied to Pulp - Test #4466: The streamer needs to try all RemoteArtifact entires related to a ContentArtifactNEW<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

Associated revisions

Revision b8a2a912 View on GitHub
Added by bmbouter over 1 year ago

Content app tries all RemoteArtifacts

The Content app would raise an exception if there were multiple RemoteArtifacts. It also would not try additional ones if the first ones failed.

https://pulp.plan.io/issues/2507 closes #2507

Revision b8a2a912 View on GitHub
Added by bmbouter over 1 year ago

Content app tries all RemoteArtifacts

The Content app would raise an exception if there were multiple RemoteArtifacts. It also would not try additional ones if the first ones failed.

https://pulp.plan.io/issues/2507 closes #2507

History

#1 Updated by bmbouter over 3 years ago

  • Tags deleted (Pulp 3)

From IRC convo, it seemed this would not need to be ported to Pulp3 so I've removed that tag. Also this seems to be a bug not a feature so I've switched the type to Issue. Please revert either of these changes if I'm remiss. Thanks for filing.

#2 Updated by jortel@redhat.com over 3 years ago

  • Tags Pulp 3 added

bmbouter wrote:

From IRC convo, it seemed this would not need to be ported to Pulp3 so I've removed that tag. Also this seems to be a bug not a feature so I've switched the type to Issue. Please revert either of these changes if I'm remiss. Thanks for filing.

The proposed change for pulp2 is to fix the catalog entry sorting but does not include the streamer trying all of the catalog entries. I think given the risk involved in this change, it is more appropriate for pulp3.

#3 Updated by bmbouter over 3 years ago

@jortel: Designating this for Pulp3 sounds good; thanks for adding the label. If there is nothing changing for this issue in Pulp2, we should probably have a separate issue created with the status of WONTFIX. I mistook this issue for a change requested on the 2.y line.

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

  • Parent task set to #3693

Sounds like we can close this issue. In case we decide to keep it, I am making this a child of 3693 to make sure it is a part of that effort.

#5 Updated by bmbouter over 1 year ago

  • Subject changed from The streamer needs to try all catalog entries. to The streamer needs to try all RemoteArtifact entires related to a ContentArtifact
  • Description updated (diff)
  • Status changed from NEW to ASSIGNED
  • Assignee set to bmbouter
  • Sprint set to Sprint 46
  • Tags Pulp 3 RC Blocker added

Adding to sprint and RC blocker since it's a required feature for lazy.

#6 Updated by dkliban@redhat.com over 1 year ago

  • Related to Issue #4246: Lazy sync does not create additional RemoteArtifact objects added

#7 Updated by rchan over 1 year ago

  • Sprint changed from Sprint 46 to Sprint 47

#8 Updated by bmbouter over 1 year ago

  • Status changed from ASSIGNED to POST

#9 Updated by rchan over 1 year ago

  • Sprint changed from Sprint 47 to Sprint 48

#10 Updated by bmbouter over 1 year ago

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

#11 Updated by bherring over 1 year ago

  • Copied to Test #4466: The streamer needs to try all RemoteArtifact entires related to a ContentArtifact added

#12 Updated by daviddavis about 1 year ago

  • Sprint/Milestone set to 3.0.0

#13 Updated by bmbouter about 1 year ago

  • Tags deleted (Pulp 3, Pulp 3 RC Blocker)

#14 Updated by bmbouter 7 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF