Project

Profile

Help

Task #3914

Port pulp_ansible to use DeclarativeVersion

Added by bmbouter about 1 year ago. Updated 7 months ago.

Status:
MODIFIED
Priority:
Normal
Assignee:
Sprint/Milestone:
Start date:
Due date:
% Done:

100%

Platform Release:
Blocks Release:
Backwards Incompatible:
No
Groomed:
Yes
Sprint Candidate:
No
Tags:
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:

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

Related to Pulp - Task #4078: Remove the Changeset code MODIFIED Actions

Associated revisions

History

#1 Updated by daviddavis about 1 year ago

  • Sprint/Milestone set to 0.1 Release
  • Groomed changed from No to Yes
  • Sprint Candidate changed from No to Yes

#2 Updated by vdusek about 1 year ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to vdusek

#3 Updated by vdusek about 1 year ago

  • Status changed from ASSIGNED to NEW
  • Assignee deleted (vdusek)

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

  • Sprint Candidate changed from Yes to No

#5 Updated by daviddavis about 1 year ago

  • Related to Task #4078: Remove the Changeset code added

#6 Updated by daviddavis about 1 year ago

As part of this fix, we'll need to unpin the versoin of pulpcore-plugin:

https://github.com/pulp/pulp_ansible/pull/66

#8 Updated by daviddavis 8 months ago

  • Status changed from NEW to POST
  • Assignee set to mdellweg

#9 Updated by mdellweg 8 months ago

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

#10 Updated by bmbouter 7 months ago

  • Tags deleted (Pulp 3)

Please register to edit this issue

Also available in: Atom PDF