Project

General

Profile

Actions

Bug #25438

closed

hammer --help does not work with --output json, csv and yaml

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

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

Description

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

Description of problem:

# for output in base table silent csv yaml json; do
  echo ">>> $output" 
  hammer -v --output $output hostgroup --help 2>&1 | head -n 2
done

>>> base
Usage:
    hammer hostgroup [OPTIONS] SUBCOMMAND [ARG] ...
>>> table
Usage:
    hammer hostgroup [OPTIONS] SUBCOMMAND [ARG] ...
>>> silent
>>> csv
Error: undefined method `[]' for nil:NilClass
>>> yaml
Error: undefined method `[]' for nil:NilClass
>>> json
Error: undefined method `[]' for nil:NilClass

# hammer --output yaml --help
Traceback (most recent call last):
    12: from /usr/bin/hammer:23:in `<main>'
    11: from /usr/bin/hammer:23:in `load'
    10: from /opt/theforeman/tfm/root/usr/share/gems/gems/hammer_cli-0.15.1/bin/hammer:147:in `<top (required)>'
     9: from /opt/theforeman/tfm/root/usr/share/gems/gems/clamp-1.1.2/lib/clamp/command.rb:132:in `run'
     8: from /opt/theforeman/tfm/root/usr/share/gems/gems/hammer_cli-0.15.1/lib/hammer_cli/abstract.rb:28:in `run'
     7: from /opt/theforeman/tfm/root/usr/share/gems/gems/hammer_cli-0.15.1/lib/hammer_cli/abstract.rb:32:in `rescue in run'
     6: from /opt/theforeman/tfm/root/usr/share/gems/gems/hammer_cli-0.15.1/lib/hammer_cli/abstract.rb:188:in `handle_exception'
     5: from /opt/theforeman/tfm/root/usr/share/gems/gems/hammer_cli-0.15.1/lib/hammer_cli/exception_handler.rb:31:in `handle_exception'
     4: from /opt/theforeman/tfm/root/usr/share/gems/gems/hammer_cli-0.15.1/lib/hammer_cli/exception_handler.rb:78:in `handle_help_wanted'
     3: from /opt/theforeman/tfm/root/usr/share/gems/gems/hammer_cli-0.15.1/lib/hammer_cli/exception_handler.rb:53:in `print_message'
     2: from /opt/theforeman/tfm/root/usr/share/gems/gems/hammer_cli-0.15.1/lib/hammer_cli/output/output.rb:19:in `print_message'
     1: from /opt/theforeman/tfm/root/usr/share/gems/gems/hammer_cli-0.15.1/lib/hammer_cli/output/adapter/yaml.rb:15:in `print_message'
/opt/theforeman/tfm/root/usr/share/gems/gems/hammer_cli-0.15.1/lib/hammer_cli/output/adapter/tree_structure.rb:22:in `prepare_message': undefined method `[]' for nil:NilClass (NoMethodError)

Version-Release number of selected component (if applicable):
Sat 6.5 snap 3

How reproducible:
always

Actions #1

Updated by Tomáš Strachota over 5 years ago

  • Category changed from Other commands to Hammer core
  • Assignee set to Oleh Fedorenko
  • Fixed in Releases hammer-cli-0.15.0 added
Actions #2

Updated by Tomáš Strachota over 5 years ago

  • Description updated (diff)
Actions #3

Updated by The Foreman Bot over 5 years ago

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

Updated by Oleh Fedorenko over 5 years ago

  • Status changed from Ready For Testing to Closed
Actions #5

Updated by Patrick Creech about 5 years ago

  • Fixed in Releases deleted (hammer-cli-0.15.0)
  • Found in Releases hammer-cli-0.15.0 added

I'm guessing this was supposed to be 'found in' not 'fixed in', as this commit doesn't exist in 0.15.0

Actions

Also available in: Atom PDF