Skip to content

updating pulp limit to 1000#4842

Merged
abhishek-sa1 merged 1 commit into
dell:stagingfrom
Kratika-P:staging
Jul 14, 2026
Merged

updating pulp limit to 1000#4842
abhishek-sa1 merged 1 commit into
dell:stagingfrom
Kratika-P:staging

Conversation

@Kratika-P

@Kratika-P Kratika-P commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator

The original plan was to use --limit 0 to disable pagination, but the installed pulp CLI rejects 0 because --limit must be a positive integer (x>=1):

pulp rpm distribution list --field name,base_url --limit 0
Usage: pulp rpm distribution list [OPTIONS]
Try 'pulp rpm distribution list --help' for help.

Error: Invalid value for '--limit': 0 is not in the range x>=1.
Updating to 1000
image
image

Suggested Reviewers

@abhishek-sa1 @priti-parate @snarthan

@abhishek-sa1 abhishek-sa1 merged commit edce91f into dell:staging Jul 14, 2026
3 of 4 checks passed
@abhishek-sa1

Copy link
Copy Markdown
Collaborator

fixed for the issue #4842

@j0hnL

j0hnL commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator

Should we make this higher than 1000?

@abhishek-sa1

Copy link
Copy Markdown
Collaborator

@j0hnL should we keep 10000?

@j0hnL

j0hnL commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator

Can we do 100,000? We should also check if options exist to disable the pagination.

@abhishek-sa1

Copy link
Copy Markdown
Collaborator

Sure, we can update 10000. @Kratika-P can you check and update

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.

4 participants