Project

Profile

Help

Task #3914

Port pulp_ansible to use DeclarativeVersion

Added by bmbouter about 2 years ago. Updated about 1 month ago.

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

100%

Estimated time:
Platform Release:
Groomed:
Yes
Sprint Candidate:
No
Tags:
Sprint:
Quarter:

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 codeCLOSED - CURRENTRELEASE

<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

Associated revisions

History

#1 Updated by daviddavis about 2 years 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 2 years ago

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

#3 Updated by vdusek about 2 years ago

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

#4 Updated by amacdona@redhat.com about 2 years ago

  • Sprint Candidate changed from Yes to No

#5 Updated by daviddavis about 2 years ago

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

#6 Updated by daviddavis about 2 years 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 over 1 year ago

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

#9 Updated by mdellweg over 1 year ago

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

#10 Updated by bmbouter over 1 year ago

  • Tags deleted (Pulp 3)

#11 Updated by bmbouter about 1 month ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF