Story #2507
closed
Story #3693: Lazy for Pulp3
The streamer needs to try all RemoteArtifact entires related to a ContentArtifact
100%
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
Updated by bmbouter over 7 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.
Updated by jortel@redhat.com over 7 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.
Updated by bmbouter over 7 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.
Updated by amacdona@redhat.com over 5 years ago
- Parent issue 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.
Updated by bmbouter over 5 years 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.
Updated by dkliban@redhat.com over 5 years ago
- Related to Issue #4246: Lazy sync does not create additional RemoteArtifact objects added
Updated by bmbouter over 5 years ago
- Status changed from ASSIGNED to POST
PR available at: https://github.com/pulp/pulp/pull/3826
Added by bmbouter about 5 years ago
Added by bmbouter about 5 years ago
Revision b8a2a912 | View on GitHub
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.
Updated by bmbouter about 5 years ago
- Status changed from POST to MODIFIED
- % Done changed from 0 to 100
Applied in changeset pulp|b8a2a912ec3e4349778a89891d7c7fb4f0518bfc.
Updated by bherring about 5 years ago
- Copied to Test #4466: The streamer needs to try all RemoteArtifact entires related to a ContentArtifact added
Updated by bmbouter almost 5 years ago
- Tags deleted (
Pulp 3, Pulp 3 RC Blocker)
Updated by bmbouter over 4 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
.
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