Task #3914
closedPort pulp_ansible to use DeclarativeVersion
100%
Description
Overview¶
Now that DeclarativeVersion is available the pulp_ansible plugin can port onto it. The progress reporting names will change, but otherwise this change will not be noticable to users, except maybe faster performance.
Implementation¶
The implementation will have two main parts, a metadata downloading page-by-page part, and then an object building part (i.e. Content unit, Artifact, DeclarativeArtifact, and DeclarativeContent). The metadata downloading part is the complicated part because in the paginated API you want several downloads running concurrently.
This is currently done in the pulp_ansible code. A master list is built here: https://github.com/pulp/pulp_ansible/blob/master/pulp_ansible/app/tasks/synchronizing.py#L153 Then 20 outstanding downloads happen here: https://github.com/pulp/pulp_ansible/blob/master/pulp_ansible/app/tasks/synchronizing.py#L155-L158
Then the "new_roles" data from a fetched page is parsed here https://github.com/pulp/pulp_ansible/blob/master/pulp_ansible/app/tasks/synchronizing.py#L168 This is where the Content units and Artifact objects should be encapsulated in DeclarativeContent and DeclarativeArtifact objects respectively. This is similar to what pulp_file does: https://github.com/pulp/pulp_file/blob/master/pulp_file/app/tasks/synchronizing.py#L74-L75
Related issues
Updated by daviddavis over 6 years ago
- Sprint/Milestone set to 0.1 Release
- Groomed changed from No to Yes
- Sprint Candidate changed from No to Yes
Updated by vdusek over 6 years ago
- Status changed from NEW to ASSIGNED
- Assignee set to vdusek
Updated by vdusek over 6 years ago
- Status changed from ASSIGNED to NEW
- Assignee deleted (
vdusek)
Updated by amacdona@redhat.com over 6 years ago
- Sprint Candidate changed from Yes to No
Updated by daviddavis about 6 years ago
- Related to Task #4078: Remove the Changeset code added
Updated by daviddavis about 6 years ago
As part of this fix, we'll need to unpin the versoin of pulpcore-plugin:
Updated by mdellweg almost 6 years ago
PR available https://github.com/pulp/pulp_ansible/pull/79
Updated by daviddavis almost 6 years ago
- Status changed from NEW to POST
- Assignee set to mdellweg
Added by mdellweg almost 6 years ago
Updated by mdellweg almost 6 years ago
- Status changed from POST to MODIFIED
- % Done changed from 0 to 100
Applied in changeset pulp_ansible|c9f3eec8c3a7899b4234b68938d59c4bc1cfdfbe.
Updated by bmbouter about 4 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Port synchronizing to stages api
closes #3914 closes #4155
https://pulp.plan.io/issues/3914 https://pulp.plan.io/issues/4155