Project

Profile

Help

Issue #5494

closed

pulp assuming a pagesize of 10 when syncing from the ansible galaxy api

Added by jsherril@redhat.com over 4 years ago. Updated over 3 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Sprint/Milestone:
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Platform Release:
OS:
Triaged:
No
Groomed:
No
Sprint Candidate:
No
Tags:
Katello
Sprint:
Sprint 74
Quarter:

Description

pulp is assuming a pagesize of 10 when syncing from galaxy: https://github.com/pulp/pulp_ansible/blob/335b6742d3c3c11d9307ae55a1f67547b5309aa9/pulp_ansible/app/tasks/collections.py#L58

it should be more intelligent about it. This breaks syncing from another pulp3 server because pulp3 isn't defaulting to a pagesize of 10

Actions #1

Updated by daviddavis over 4 years ago

From my reading of the code, it looks like we assume that the page size is 10. Instead, we should request a certain page when syncing. So like:

https://galaxy.ansible.com/api/v2/collections/?page_size=1

I think 100 would be a good size to use.

Actions #2

Updated by ggainey almost 4 years ago

  • Tags Katello added
  • Tags deleted (Katello-P2)
Actions #3

Updated by bmbouter almost 4 years ago

  • Sprint set to Sprint 74
Actions #4

Updated by fao89 almost 4 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to fao89
Actions #5

Updated by pulpbot almost 4 years ago

  • Status changed from ASSIGNED to POST

Added by Fabricio Aguiar almost 4 years ago

Revision d568c8c2 | View on GitHub

Make default page size equals to 100

https://pulp.plan.io/issues/5494 closes #5494

Actions #7

Updated by Anonymous almost 4 years ago

  • Status changed from POST to MODIFIED
Actions #8

Updated by fao89 almost 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Actions #9

Updated by fao89 over 3 years ago

  • Sprint/Milestone set to 0.2.0

Also available in: Atom PDF