Project

General

Profile

Actions
Copy link

Bug #21627

closed

Message when trying to order by attribute not supporting ordering is not helpful

Added by Swapnil Abnave over 6 years ago. Updated almost 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Swapnil Abnave
Category:
Other commands
Target version:
-
Difficulty:
Triaged:
Bugzilla link:
1510964
Pull request:
https://github.com/theforeman/hammer-cli-foreman/pull/341
Team Backlog:
Fixed in Releases:
Found in Releases:
In Kanboard:
Red Hat JIRA:

Description

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

Description of problem:

  1. hammer host list --order id
    Error: 400 Bad Request

The id has not defined ordering on it so the failure is expected but the message is not helpful. The API sends nice message which is not propagated by hammer

[ERROR 2017-11-08 08:20:18 API] 400 Bad Request
[DEBUG 2017-11-08 08:20:18 API] {
    "error" => {
        "message" => "the field 'id' in the order statement is not valid field for search",
          "class" => "ScopedSearch::QueryNotSupported" 
    }
}

Version-Release number of selected component (if applicable):
Hammer 0.11

How reproducible:
Always

Steps to Reproduce:
1. hammer host list --order id
2. see the ouptut
3.

Actual results:
Error: 400 Bad Request

Expected results:
Error: the field 'id' in the order statement is not valid field for search

Additional info:

Actions
Copy link
 #1

Updated by The Foreman Bot over 6 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/hammer-cli/pull/259 added
Actions
Copy link
 #2

Updated by The Foreman Bot over 6 years ago

  • Pull request https://github.com/theforeman/hammer-cli-foreman/pull/341 added
Actions
Copy link
 #3

Updated by Swapnil Abnave over 6 years ago

  • Pull request deleted (https://github.com/theforeman/hammer-cli/pull/259)
Actions
Copy link
 #4

Updated by Swapnil Abnave over 6 years ago

  • Target version set to 230
Actions
Copy link
 #5

Updated by Swapnil Abnave over 6 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions
Copy link
 #6

Updated by Lukáš Hellebrandt about 6 years ago

Verified with tfm-rubygem-hammer_cli-0.12.0-1.

$ hammer --version
hammer (0.12.0) * hammer_cli_foreman (0.13.pre.develop) * hammer_cli_foreman_bootdisk (0.1.3) * hammer_cli_foreman_docker (unknown version) * hammer_cli_foreman_tasks (unknown version) * hammer_cli_katello (0.11.0)

$ hammer host list --order id
[Foreman] Username: <login>
[Foreman] Password for admin:
400 Bad Request
the field 'id' in the order statement is not valid field for search

Actions
Copy link

Also available in: Atom PDF