Bug 1420673

Summary: hammer list sub-commands fail with 'comparison of Symbol with 80 failed'
Product: Red Hat Satellite Reporter: Lukas Pramuk <lpramuk>
Component: HammerAssignee: satellite6-bugs <satellite6-bugs>
Status: CLOSED ERRATA QA Contact: Lukas Pramuk <lpramuk>
Severity: high Docs Contact:
Priority: high    
Version: 6.2.8CC: bbuckingham, bkearney, dhlavacd, ehelms, jcallaha, mhulan, tstrachota, zhunting
Target Milestone: UnspecifiedKeywords: Regression, Triaged
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: tfm-rubygem-unicode-display_width-1.0.5-2 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-03-06 08:37:56 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Lukas Pramuk 2017-02-09 09:35:15 UTC
Description of problem:
hammer lifecycle-environment commands fail with 'comparison of Symbol with 80 failed'

Version-Release number of selected component (if applicable):
@Sat6.2.8
tfm-rubygem-hammer_cli-0.5.1.13-2.el7sat.noarch

How reproducible:
100%

Steps to Reproduce:
1a. # hammer lifecycle-environment paths --organization-id 1
comparison of Symbol with 80 failed

1b. # hammer lifecycle-environment list --organization-id 1
comparison of Symbol with 80 failed

 
ArgumentError (comparison of Symbol with 80 failed):
    /opt/theforeman/tfm/root/usr/share/gems/gems/hammer_cli-0.5.1.13/lib/hammer_cli/output/adapter/table.rb:96:in `>='
    /opt/theforeman/tfm/root/usr/share/gems/gems/hammer_cli-0.5.1.13/lib/hammer_cli/output/adapter/table.rb:96:in `block in calculate_column_width'
    /opt/theforeman/tfm/root/usr/share/gems/gems/hammer_cli-0.5.1.13/lib/hammer_cli/output/adapter/table.rb:94:in `each'
    /opt/theforeman/tfm/root/usr/share/gems/gems/hammer_cli-0.5.1.13/lib/hammer_cli/output/adapter/table.rb:94:in `calculate_column_width'
    /opt/theforeman/tfm/root/usr/share/gems/gems/hammer_cli-0.5.1.13/lib/hammer_cli/output/adapter/table.rb:84:in `block in calculate_widths'
    /opt/theforeman/tfm/root/usr/share/gems/gems/hammer_cli-0.5.1.13/lib/hammer_cli/output/adapter/table.rb:84:in `map'
    /opt/theforeman/tfm/root/usr/share/gems/gems/hammer_cli-0.5.1.13/lib/hammer_cli/output/adapter/table.rb:84:in `calculate_widths'
    /opt/theforeman/tfm/root/usr/share/gems/gems/hammer_cli-0.5.1.13/lib/hammer_cli/output/adapter/table.rb:28:in `print_collection'


Actual results:
hammer fails

Expected results:
works as expected

Comment 3 Peter Ondrejka 2017-02-09 09:47:36 UTC
I'm getting the same error with pretty much every list command in hammer, e.g:

hammer organization list
hammer host list
hammer product list --organization-id 1
hammer admin logging --list
...

Comment 4 Lukas Pramuk 2017-02-09 09:49:30 UTC
Only output using table adapter is affected:

# hammer --output table lifecycle-environment list --organization-id 1
comparison of Symbol with 80 failed


While other adapters are good:
 
# hammer --output base lifecycle-environment list --organization-id 1
ID:    1
Name:  Library
Prior:

# hammer --output silent lifecycle-environment list --organization-id 1

# hammer --output csv lifecycle-environment list --organization-id 1
ID,Name,Prior
1,Library,

# hammer --output yaml lifecycle-environment list --organization-id 1
---
- ID: 1
  Name: Library
  Prior: 

# hammer --output json -p lifecycle-environment list --organization-id 1
[
  {
    "ID": 1,
    "Name": "Library",
    "Prior": null
  }
]

Comment 10 Lukas Pramuk 2017-02-21 09:16:18 UTC
VERIFIED.

@satellite-6.2.8-1.0.el7sat.noarch (snap3)
tfm-rubygem-hammer_cli-0.5.1.13-2.el7sat.noarch
tfm-rubygem-unicode-display_width-1.0.5-2.el7sat.noarch

>>> hammer {organization,product,host...} list subcommands work again

Comment 12 errata-xmlrpc 2017-03-06 08:37:56 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2017:0447