Skip to content
This repository has been archived by the owner on May 6, 2022. It is now read-only.

Remove uuid from svcat get output #1804

Closed
carolynvs opened this issue Mar 6, 2018 · 3 comments
Closed

Remove uuid from svcat get output #1804

carolynvs opened this issue Mar 6, 2018 · 3 comments
Assignees
Labels
Milestone

Comments

@carolynvs
Copy link
Contributor

When using svcat, the uuid loses its importance because svcat understands how to work with the external names. I propose that we not display uuid in the svcat get command output. When someone wants to know the uuid, fore example so that they can use it in a kubectl command, they can look it up with svcat describe.

In it's place, I'd like to make the broker more visible (see #1749). I'm showing the final output that I'd like even though these are two separate issues (this one is just for not displaying uuid)_. 😀

$ svcat get classes
            NAME                      BROKER                         DESCRIPTION
+--------------------------+---------------------------+--------------------------------------+
  azure-rediscache            osba                        Azure Redis Cache
                                                          (Experimental)
  redis                       Minibroker                  In-cluster installation of Redis

$ svcat get plans
        NAME                  CLASS                      BROKER                   DESCRIPTION
+------------------+--------------------------+----------------------+----------------------------------------+
  basic              azure-rediscache           osba                    Basic Tier, 250MB Cache
  redis-5-7          redis                      Minibroker              In-cluster installation of Redis 5.7
@n3wscott
Copy link
Contributor

I think there is still some value in being able to query based on UUID, I hope that aspect was not removed. Maybe the uuid column can be optionally shown?

@carolynvs
Copy link
Contributor Author

carolynvs commented Mar 13, 2018

Nothing has been implemented yet, this is just a suggestion. 😀 It wasn't my intention of removing the --uuid flag supporting uuid queries. I just want remove it from the default columns displayed in the get output table.

@carolynvs carolynvs self-assigned this Apr 24, 2018
@carolynvs
Copy link
Contributor Author

Closed by #1977

@carolynvs carolynvs added this to the 1.0.0 milestone May 21, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants