Project

General

Profile

Actions

Bug #18340

closed

Hammer params for ordering are inconsistent

Added by Tomáš Strachota about 7 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Category:
Other commands
Target version:
-
Difficulty:
Triaged:
No
Team Backlog:
Marek
Fixed in Releases:
Found in Releases:
In Kanboard:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1417726

Description of problem:

Foreman part of hammer cli uses parameter --order="<FIELD_NAME> <DIRECTION>" for ordering results. Katello part uses two parameters --by="<FIELD_NAME>" and --order="<DIRECTION>"

For example compare

hammer architecture list -h

and

hammer repository list -h

Apart from the above there are additional issues:
  • in Katello commands --by and --order seem to have no effect (probably related to bz 1138262)
  • in Foreman commands there's no validation of the direction and API silently falls back to 'ASC'

Version-Release number of selected component (if applicable):
hammer-cli-foreman 0.9.0
hammer-cli-katello 0.3.0

Actual results:
There are various parameters for ordering results, some of the dysfunctional.

Expected results:
Hammer should provide unified commands for ordering fields. Probably only --order should be used.


Related issues 1 (0 open1 closed)

Blocked by Katello - Bug #18341: API sort hash is not working, but sort_by and sort_order are working but undocumentedClosedActions
Actions #1

Updated by Tomáš Strachota about 7 years ago

  • Subject changed from Hammer params for ordering are inconsistent to Hammer params for ordering are inconsistent
  • Description updated (diff)
  • Target version set to 115
Actions #2

Updated by Tomáš Strachota about 7 years ago

  • Blocked by Bug #18341: API sort hash is not working, but sort_by and sort_order are working but undocumented added
Actions #3

Updated by Oleh Fedorenko over 5 years ago

  • Status changed from New to Assigned
  • Assignee set to Oleh Fedorenko
  • Triaged set to No
Actions #4

Updated by Oleh Fedorenko over 5 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/Katello/hammer-cli-katello/pull/588 added
Actions #5

Updated by Oleh Fedorenko over 5 years ago

  • Status changed from Ready For Testing to Closed
Actions

Also available in: Atom PDF