Bug #17010

Hammer JSON output uses inconsistent capitalization

Added by Adam Ruzicka about 1 year ago. Updated about 1 month ago.

Status:New
Priority:Normal
Assigned To:-
Category:Other commands
Target version:Foreman - Team Ivan backlog
Difficulty: Bugzilla link:1253820
Found in release: Pull request:
Story points-
Velocity based estimate-

Description

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

Description of problem:
Hammer returns inconsistent capitalization in its json output. For example, a user trying to parse the id's in an output may have to handle 'ID', 'Id', and 'id'.

Version-Release number of selected component (if applicable):
Satellite 6.1 GA

How reproducible:
Always

Steps to Reproduce:
1. run one of the two commands detailed in Additional Info, or your own choice.
2. run at least one more command
3. Check consistency for returned keys

Actual results:
keys are inconsistently capitalized

Expected results:
keys are inconsistently capitalized. I prefer lowercase for ease

Additional info:

hammer --output=json organization info --id="31" 
{
  "Id": 31,
  "Name": "KVybjk",
  "Users": [

  ],
  "Smart proxies": [
    {
      "name": "rhsm-qe-1.rhq.lab.eng.bos.redhat.com",
      "id": 1,
      "url": "https://rhsm-qe-1.rhq.lab.eng.bos.redhat.com:9090" 
    }
...
  "Templates": [
    {
      "id": 44,
      "name": "Boot disk iPXE - generic host",
      "template_kind_id": 1,
      "template_kind_name": "Bootdisk" 
    },

hammer -v -u --output=json activation-key info --id="57" 
{
  "Name": "rI6jNBJVaR",
  "ID": 57,
  "Description": null,
  "Content Host Limit": "Unlimited",
  "Auto Attach": true,
  "Lifecycle Environment": null,
  "Content View": null,
  "Host Collections": {
  }
}

Related issues

Related to Hammer CLI - Tracker #2823: Tracker for usability issues New 07/24/2013
Related to Hammer CLI - Tracker #14914: Tracker for output related issues New 05/03/2016
Duplicated by Hammer CLI - Bug #19996: Keys in JSON output has inconsistent capitalization Duplicate 06/13/2017

History

#1 Updated by Adam Ruzicka about 1 year ago

  • Target version set to Team Ivan backlog

#2 Updated by Tomáš Strachota 2 months ago

  • Description updated (diff)

#3 Updated by Tomáš Strachota about 1 month ago

#4 Updated by Tomáš Strachota about 1 month ago

#5 Updated by Tomáš Strachota about 1 month ago

  • Duplicated by Bug #19996: Keys in JSON output has inconsistent capitalization added

#6 Updated by Tomáš Strachota about 1 month ago

See also: http://projects.theforeman.org/issues/19996
It's a duplicate but it contains some information as well.

Also available in: Atom PDF