Bug #17010
closedHammer JSON output uses inconsistent capitalization
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": { } }
Updated by Tomáš Strachota about 7 years ago
- Related to Tracker #2823: Tracker for usability issues added
Updated by Tomáš Strachota about 7 years ago
- Related to Tracker #14914: Tracker for output related issues added
Updated by Tomáš Strachota about 7 years ago
- Has duplicate Bug #19996: Keys in JSON output has inconsistent capitalization added
Updated by Tomáš Strachota about 7 years ago
See also: http://projects.theforeman.org/issues/19996
It's a duplicate but it contains some information as well.
Updated by Oleh Fedorenko over 6 years ago
- Status changed from New to Assigned
- Assignee set to Oleh Fedorenko
- Triaged set to No
Updated by The Foreman Bot over 6 years ago
- Status changed from Assigned to Ready For Testing
- Pull request https://github.com/theforeman/hammer-cli/pull/281 added
Updated by Oleh Fedorenko over 6 years ago
- Status changed from Ready For Testing to Closed
Applied in changeset ab81fe49da3d174ac36837e0a9e4ceb9e4186b89.
Updated by Martin Bacovsky about 6 years ago
- Fixed in Releases hammer-cli-0.15.0 added