Skip to content

Issue #4833: Disable Pulp pagination for distribution list commands#4838

Merged
abhishek-sa1 merged 1 commit into
dell:q3_mainfrom
j0hnL:issue-4833
Jul 14, 2026
Merged

Issue #4833: Disable Pulp pagination for distribution list commands#4838
abhishek-sa1 merged 1 commit into
dell:q3_mainfrom
j0hnL:issue-4833

Conversation

@j0hnL

@j0hnL j0hnL commented Jul 13, 2026

Copy link
Copy Markdown
Collaborator

Description

This PR fixes issue #4833 — the pulp rpm distribution list command uses Pulp's default pagination, which truncates results when the number of repositories exceeds the default page size. This causes downstream packages (e.g. Slurm) to not be discovered or installed because their repos fall beyond the first page of results.

Changes

Adds --limit 0 to the pulp rpm distribution list commands in both architecture-specific fetch_pulp_repos.yml files. The --limit 0 flag disables Pulp pagination entirely, ensuring all distributions are returned regardless of count.

Files modified

  • src/playbooks/build_image_aarch64/roles/fetch_packages/tasks/fetch_pulp_repos.yml
  • src/playbooks/build_image_x86_64/roles/fetch_packages/tasks/fetch_pulp_repos.yml

How to verify

  1. Configure Pulp with more repos than the default page size (typically >100)
  2. Run the build_image playbook for either aarch64 or x86_64
  3. Confirm all distributions are returned and downstream packages (e.g. Slurm) are found

This PR closes #4833

Add --limit 0 to pulp rpm distribution list commands to disable
pagination and return all distributions. Without this flag, Pulp's
default pagination truncates results when the number of repos
exceeds the page size, causing downstream packages (e.g. Slurm)
to not be found during image builds.

This PR closes issue dell#4833

Signed-off-by: John Lockman <jlockman3@gmail.com>
@abhishek-sa1

Copy link
Copy Markdown
Collaborator

@j0hnL can we raise same PR in staging, issue was part of staging as well.

@abhishek-sa1 abhishek-sa1 merged commit 935771f into dell:q3_main Jul 14, 2026
9 checks passed
@abhishek-sa1

Copy link
Copy Markdown
Collaborator

@j0hnL raised PR to staging to update the same - #4842
limit 0 is showing as invalid option in pulp

@j0hnL j0hnL deleted the issue-4833 branch July 14, 2026 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants