Actions
Bug #14530
closedhammer queries defaults to 1000 items per page
Difficulty:
Triaged:
Bugzilla link:
Pull request:
Team Backlog:
Description
Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1247781
Description of problem:
hammer queries default to 1000 items per page which makes hammer very slow
Version-Release number of selected component (if applicable):
Sat 6.1 GA Snap 14
How reproducible:
Always
- cat /etc/hammer/cli_config.yml
:ui:
:interactive: true
:per_page: 20
:history_file: '~/.foreman/history'
- enable/disable color output of logger in Clamp commands
:watch_plain: false
:log_dir: '~/.foreman/log'
:log_level: 'error'
:log_api_calls: false
#:log_owner: 'foreman'
#:log_group: 'foreman'
#:log_size: 5 #MB
- hammer
u admin -p changeme repository list --organization-id=3|-------------------------------------------------------|---------------------------------|--------------|--------------------------------------------------------------------
--
ID | NAME | PRODUCT | CONTENT TYPE | URL
---|-------------------------------------------------------|---------------------------------|--------------|--------------------------------------------------------------------
27 | Red Hat Enterprise Linux 5 Server RPMs x86_64 5Server | Red Hat Enterprise Linux Server | yum | https://cdn.redhat.com/content/dist/rhel/server/5/5Server/x86_64/os
1 | Red Hat Enterprise Linux 7 Server RPMs x86_64 7Server | Red Hat Enterprise Linux Server | yum | https://cdn.redhat.com/content/dist/rhel/server/7/7Server/x86_64/os
---|-------------------------------------------------------|---------------------------------|--------------|--------------------------------------------------------------------
- time hammer
u admin -p changeme erratum list|----------------|-------------|---------------------------------------------------------------------------------
....
....
------------------------------------
real 7m58.835s
user 7m32.086s
sys 0m0.266s
Actual results:
See above that the erratum list command took almost 8 minutes to finish
Expected results:
Pagination should work
Additional info:
Updated by The Foreman Bot over 8 years ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/hammer-cli/pull/200 added
Updated by The Foreman Bot over 8 years ago
- Pull request https://github.com/theforeman/hammer-cli-foreman/pull/228 added
Updated by Martin Bacovsky over 8 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset hammer-cli-foreman|a7f86beb465737a4e8c4908a0fb9abfe824d0139.
Updated by Dominic Cleal over 8 years ago
- Related to Bug #15257: Default per_page configuration enables pagination, including in CSV mode added
Updated by Andrew Kofink over 8 years ago
- Related to Bug #15542: SAT6.1 Pagination control not supported for listing content hosts in host collection with Satellite 6 command-line added
Actions