Skip to content

Unify all workspace information in coder show #1646

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Tracked by #1939
ammario opened this issue May 21, 2022 · 9 comments
Closed
Tracked by #1939

Unify all workspace information in coder show #1646

ammario opened this issue May 21, 2022 · 9 comments
Assignees
Labels
api Area: HTTP API cli Area: CLI
Milestone

Comments

@ammario
Copy link
Member

ammario commented May 21, 2022

Right now, we have autostop information within coder autostop show, which is potentially confusing as a user has an expectation that all workspace data would be presented in a single view. Even if it's not confusing, it's cumbersome to run multiple commands to get the complete picture of a workspace.

See #1431 (comment) for background.

@tjcran
Copy link

tjcran commented May 30, 2022

@vapurrmaid @johnstcn what are both of your thoughts on this one? I agree that it makes sense to remedy this one.

@misskniss
Copy link

@johnstcn @vapurrmaid @ammario Can e get a solid Definition of Done for this one to avoid potential delays and ambiguity?

@johnstcn
Copy link
Member

johnstcn commented Jun 14, 2022

How about:

WORKSPACE     : johnstcn/cians-workspace
CREATED       : 2022-06-10T12:14:48 GMT+1
UPDATED       : 2022-06-10T12:14:48 GMT+1
OUTDATED      : false
TEMPLATE      : coder
STATUS        : running (since 022-06-14T08:08:07 GMT+1)
STOP_AT       : 2022-06-10T18:00:00 GMT+1
SCHEDULE_START: 9:30 AM Mon-Fri Europe/Dublin
SCHEDULE_STOP : +12h 

More fields as required, but no indentation -- it should be grep and awk -friendly.

@ammario
Copy link
Member Author

ammario commented Jun 14, 2022

@johnstcn it seems hard to combine the existing resources view with the new schedule information. What do you think about just showing schedule information in coder schedule show?

Screen Shot 2022-06-14 at 2 58 59 PM

@johnstcn
Copy link
Member

johnstcn commented Jun 14, 2022

@ammario Kyle was suggesting just dropping the fancy table altogether, but we can just put the schedule info under the schedule subcommand if that makes things tidier!

@ammario
Copy link
Member Author

ammario commented Jun 14, 2022

Hmm I think putting it under schedule show makes sense at first because showing resources and schedule usefully under coder show will be a challenge.

@ketang
Copy link
Contributor

ketang commented Jun 14, 2022

But without shouting field labels right?

@johnstcn
Copy link
Member

I suppose field labels only need to be shouty when they're a table header.

@johnstcn
Copy link
Member

Fixed by #2402

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api Area: HTTP API cli Area: CLI
Projects
None yet
Development

No branches or pull requests

5 participants