Task #3763
closed
Story #3693: Lazy for Pulp3
Add "policy" attribute to Remote
100%
Description
A choice field with three choices: on_demand and cache_only, and immediate. The default is "immediate".
When set to "on_demand", any time the Streamer downloads that ContentArtifact it will save the Artifact and associate it correctly so that it won't be re-downloaded.
Related issues
Updated by bmbouter almost 6 years ago
- Has duplicate Story #3526: As a user, I can disable lazy_sync from storing packages added
Updated by CodeHeeler over 5 years ago
- Status changed from NEW to ASSIGNED
- Assignee set to CodeHeeler
Updated by CodeHeeler over 5 years ago
- Assignee changed from CodeHeeler to bmbouter
Updated by bmbouter over 5 years ago
- Subject changed from Add "policy" attribute to ContentArtifact to Add "policy" attribute to Remote
- Description updated (diff)
This is supposed to be on the Remote since that is where the user configures it. Also it's a choice of 3 things, immediate, on_demand, and cache_only. This was from the mailing list: https://www.redhat.com/archives/pulp-dev/2018-June/msg00016.html
Updated by kersom over 5 years ago
- Related to Test #4126: Test sync with different download policies added
Updated by bmbouter over 5 years ago
- Status changed from ASSIGNED to POST
PR available at: https://github.com/pulp/pulp/pull/3738
Updated by bmbouter over 5 years ago
- Sprint set to Sprint 46
Adding to sprint because it's a sub-story of the streamer work which is also on sprint.
Updated by kersom over 5 years ago
- Related to Test #4182: Test Remote "policy" attribute added
Updated by amacdona@redhat.com over 5 years ago
- Has duplicate Story #3320: Support On_demand and Background downloaders added
Added by bmbouter over 5 years ago
Added by bmbouter over 5 years ago
Revision 3daf3bf7 | View on GitHub
Adds 'policy' to Remote
Required PR: https://github.com/pulp/pulp_file/pull/132
This includes a serializer that validates input correctly.
This also switches the finalize, write_data, and handle_headers callbacks to be coroutines themselves. This is required because within those callbacks the streamer needs to await on coroutines and that can only be done from withing a coroutine itself.
This also adjusts the Remote.get_downloader() method signature to take
url or remote_artifact as kwargs instead of url as a positional
parameter. This allows plugin writers (and the streamer) to get
downloaders configured with digest and size validation automatically
from remote_artifacts.
Updated by bmbouter over 5 years ago
- Status changed from POST to MODIFIED
- % Done changed from 0 to 100
Applied in changeset pulp|3daf3bf7e48b6dcb99046b518e17ac226cb10245.
Updated by kersom over 5 years ago
- Related to Test #4220: Test update download policy with invalid field added
Updated by bmbouter over 4 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
.
Adds 'policy' to Remote
Required PR: https://github.com/pulp/pulp_file/pull/132
This includes a serializer that validates input correctly.
This also switches the finalize, write_data, and handle_headers callbacks to be coroutines themselves. This is required because within those callbacks the streamer needs to await on coroutines and that can only be done from withing a coroutine itself.
This also adjusts the Remote.get_downloader() method signature to take
urlorremote_artifactas kwargs instead ofurlas a positional parameter. This allows plugin writers (and the streamer) to get downloaders configured with digest and size validation automatically from remote_artifacts.https://pulp.plan.io/issues/3763 closes #3763