Bug 723308

Summary: verbose environment information should list names not ids
Product: Red Hat Satellite Reporter: Bryan Kearney <bkearney>
Component: katello-agentAssignee: Ivan Necas <inecas>
Status: CLOSED CURRENTRELEASE QA Contact: Garik Khachikyan <gkhachik>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 6.0.1CC: gkhachik, inecas, mkoci
Target Milestone: Unspecified   
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: katello-0.1.71-1~77 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-08-22 17:50:30 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On:    
Bug Blocks: 747354    

Description Bryan Kearney 2011-07-19 17:59:19 UTC
Running this command:

environment info --org ACME_Corporation --name dev -v

gets you output which shows ids but not names:

Id:             	3                        
Name:           	dev                      
Description:
    Yup, and environment
Org Id:         	1                        
Prior Id:       	1   

It would be good for the Org Id and Prior Id to be Names, instead of Ids. So.. would prefer to see:

Id:             	3                        
Name:           	dev                      
Description:
    Yup, and environment
Org:         	        ACME_Corporation                
Prior Environment:      Locker

Comment 1 Ivan Necas 2011-08-23 15:12:05 UTC
What about grep mode? Do we keep there the ids instead of names. I would expect to use names there as well, since we always refer to organizations and environments with their names AFAIK.

Comment 2 Bryan Kearney 2011-08-23 20:42:38 UTC
I agree. I would default to names. If we are showing details about an item, it is fine to show Id and Name. But if referencing Org and Prior, use the name.

Comment 3 Ivan Necas 2011-08-24 07:38:17 UTC
Fixed in commit 20f2aa44

Comment 4 Garik Khachikyan 2011-09-23 13:38:41 UTC
# REOPEN

Seems not fixed: environment info --org org_1 --name dev -v

Returns:
--------------------
Id:          3
Name:        dev
Description:
    Development env.
Org Id:      2
Prior Id:    Locker


Checked against: katello-cli-0.1.10-1.git.278.85760dc.el6.noarch

Comment 5 Ivan Necas 2011-10-07 12:34:26 UTC
Fixed it in list action, but not in info. Fix for that part is in commit 

7baa2c3cdabd2eddbae60e7f476c0cd1126c087d

Comment 6 Garik Khachikyan 2011-11-30 11:08:59 UTC
# VERIFIED

now both environment {list,info} are showing the names of env and prior values.

Results look like:
--- list
1    Locker   None          ACME_Corporation   None               
2    Dev      None          ACME_Corporation   Locker             
3    Test     None          ACME_Corporation   Locker             
---

--- info
Id:                2
Name:              Dev
Description:
    None
Org:               ACME_Corporation
Prior Environment: Locker
---

thanks for the fix. 
Checked against: katello-0.1.117-1.el6.x86_64

Comment 9 Mike McCune 2013-08-16 18:21:56 UTC
getting rid of 6.0.0 version since that doesn't exist