Thanks to visit codestin.com
Credit goes to github.com

Skip to content

cli: No way to view auto start after it's set #1431

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 #1317
ammario opened this issue May 13, 2022 · 13 comments Β· Fixed by #1436
Closed
Tracked by #1317

cli: No way to view auto start after it's set #1431

ammario opened this issue May 13, 2022 · 13 comments Β· Fixed by #1436
Assignees
Labels
api Area: HTTP API
Milestone

Comments

@ammario
Copy link
Member

ammario commented May 13, 2022

There's no way in the CLI to read auto start. Ideally it shows in coder show AND coder ls as an optional column.

This is also related to #1429 since coder ls will want a compressed view of the schedule.

cc @johnstcn

@johnstcn johnstcn self-assigned this May 13, 2022
@greyscaled greyscaled changed the title No way to view auto start after it's set cli: No way to view auto start after it's set May 13, 2022
@ammario
Copy link
Member Author

ammario commented May 13, 2022

@johnstcn this bit is obnoxious... is it easy to drop?
image

@ammario ammario reopened this May 13, 2022
@johnstcn
Copy link
Member

johnstcn commented May 13, 2022

@ammario This better? #1440

image

@ammario
Copy link
Member Author

ammario commented May 13, 2022

Ideally everything about the workspace is within the existing coder show <workspace>.

In the future, we'll have quotas, autostop, and who knows what, and we don't want a show command for each.

Also we have to talk about your terminal configuration at some point πŸ˜‰

@ammario
Copy link
Member Author

ammario commented May 13, 2022

Oh yeah, I love the next though. It would be cool if in parenthesis it was like "5 days remaining".

@greyscaled
Copy link
Contributor

Somehow this didn't get auto-merged. It's already done (by #1436)

@greyscaled greyscaled added api Area: HTTP API cli Area: CLI and removed api Area: HTTP API labels May 14, 2022
@ammario ammario reopened this May 14, 2022
@ammario
Copy link
Member Author

ammario commented May 14, 2022

@vapurrmaid I reopened intentionally to work through an improvement.

@greyscaled
Copy link
Contributor

@vapurrmaid I reopened intentionally to work through an improvement.

TY for context, all good πŸ‘ŒπŸ»

@ammario
Copy link
Member Author

ammario commented May 18, 2022

@johnstcn where does this sit on your radar?

@johnstcn
Copy link
Member

@johnstcn where does this sit on your radar?

  • coder autostart|autostop show is implemented
  • coder ls shows schedules

Remaining:

  • implement workspace metadata output in coder show (currently only shows resources)

@ammario
Copy link
Member Author

ammario commented May 18, 2022

What will the new 'coder show' output look like?

@johnstcn
Copy link
Member

What will the new 'coder show' output look like?

I was thinking of having the metadata in a separate vertical-aligned table, something like the following:

β”Œβ”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”
β”‚ NAME           β”‚ gcp-cian     |
| STATUS         | stopped      |
| AUTOSTART      | 30 9 * * 1-5 |
| AUTOSTOP       | -            |
| // additional fields ...      |
β””β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”˜
β”Œβ”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”
β”‚ RESOURCE                  STATUS      ACCESS β”‚
β”œβ”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€
β”‚ google_compute_disk.root  persistent         β”‚
β””β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”˜

However, I'm open to suggestions!

@ammario
Copy link
Member Author

ammario commented May 18, 2022

We moved away from pretty tables in coder ls in favor of parsability:

WORKSPACE       TEMPLATE  STATUS   LAST BUILT  OUTDATED  AUTOSTART    AUTOSTOP
ammario/basher  coder     Running  4d23h       false     0 9 * * 1-5  -

I think this is the right call for each of our commands. Perhaps something like:

NAME ammar
AUTOSTART xxx

πŸ€·πŸ½β€β™‚οΈ

@ammario
Copy link
Member Author

ammario commented May 21, 2022

Closed in favor of #1646

@ammario ammario closed this as completed May 21, 2022
@misskniss misskniss added duplicate and removed bug api Area: HTTP API cli Area: CLI labels Jun 14, 2022
@misskniss misskniss modified the milestone: Community MVP Jun 14, 2022
@misskniss misskniss added Community MVP api Area: HTTP API and removed duplicate labels Jun 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api Area: HTTP API
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants