Project

General

Profile

Bug #10534

hammer pagination output should be disabled by default or respect terminal size

Added by Tomáš Strachota almost 8 years ago. Updated almost 8 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Tomáš Strachota
Category:
Foreman commands (obsolete)
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:
1190857
Pull request:
https://github.com/theforeman/hammer-cli-foreman/pull/179
Team Backlog:
Fixed in Releases:
Found in Releases:
In Kanboard:
Red Hat JIRA:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1190857
Most if not all Linux/Unix commands do not have pagination built in by default and users rely on pipes to more/less to control the page size of output.

By default hammer ships with a very small 25 line pagination setting which is bothersome as a user who wants to view more input.

Most users outside of an actual 80x25 line terminal will have way more screen real-estate beyond the 25 we ship with now.

There is also no argument available to disable paging so users are forced to know to edit a config file to adjust this value.

Ideally the pagination would be disabled entirely by default and could optionally be turned on.

Currently we have to advocate users putting in arbitrarily large values to 'disable' pagination, eg:

:per_page: 9999

Related issues

Related to Hammer CLI - Bug #5073: [RFE] Ability to disable paginationClosed2014-04-04
Related to Hammer CLI - Feature #3898: Pagination should be turned off by default in --csv modeDuplicate2013-12-17

Associated revisions

Revision 135c18b5 (diff)
Added by Tomas Strachota almost 8 years ago

Fixes #10534 - turn off pagination by default

Turns off pagination for all list commands by default.
It can still be invoked by passing --per-page explicitly.

Revision 4689f5bf
Added by mbacovsky almost 8 years ago

Merge pull request #179 from tstrachota/list_all

Fixes #10534 - turn off pagination by default

History

#1 Updated by Tomáš Strachota almost 8 years ago

  • Category set to Foreman commands (obsolete)
  • Status changed from New to Assigned

#2 Updated by Dominic Cleal almost 8 years ago

Isn't this a dupe of #5073 or #3898?

#3 Updated by Tomáš Strachota almost 8 years ago

  • Related to Bug #5073: [RFE] Ability to disable pagination added

#4 Updated by Tomáš Strachota almost 8 years ago

  • Related to Feature #3898: Pagination should be turned off by default in --csv mode added

#5 Updated by Tomáš Strachota almost 8 years ago

Not directly dupes but they are highly related. Links added.

#6 Updated by The Foreman Bot almost 8 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/hammer-cli-foreman/pull/179 added
  • Pull request deleted ()

#7 Updated by Anonymous almost 8 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

Also available in: Atom PDF