Project

General

Profile

Actions

Bug #6971

closed

CLI: A number of info command doesn't work if you pass them bad ids

Added by dustin tsang almost 10 years ago. Updated almost 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Hammer
Target version:
Difficulty:
Triaged:
Yes
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1085417
Try these commands:

hammer product info --id 885858585885
hammer system info --id 8588585858

You'll get weird results like:

undefined method `readable?' for nil:NilClass

or

Error: 400 Bad Request

I'd expect a message like:

Could not find product with id 885858585885

Actions #1

Updated by dustin tsang almost 10 years ago

Currently these commands seem to be having problems (excerpt from https://gist.github.com/dustint-rh/a60d05d0c62549b27dab):

hammer capsule info --id 2141
protected method `resource_name' called for #<Katello::Api::V2::CapsulesController:0x0000001d229e18>

hammer product info --id 2141
no resource loaded

hammer repository info --id 2141
wrong number of arguments (0 for 1)

Actions #2

Updated by The Foreman Bot almost 10 years ago

  • Status changed from New to Ready For Testing
  • Target version set to 54
  • Pull request https://github.com/Katello/hammer-cli-katello/pull/216 added
  • Pull request deleted ()
Actions #3

Updated by Eric Helms almost 10 years ago

  • Triaged changed from No to Yes
Actions #5

Updated by dustin tsang almost 10 years ago

  • Pull request added
  • Pull request deleted (https://github.com/Katello/hammer-cli-katello/pull/216)
Actions #6

Updated by The Foreman Bot almost 10 years ago

  • Pull request https://github.com/Katello/hammer-cli-katello/pull/216 added
  • Pull request deleted ()
Actions #7

Updated by dustin tsang almost 10 years ago

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

Updated by Eric Helms over 9 years ago

  • translation missing: en.field_release set to 13
Actions

Also available in: Atom PDF