Skip to content
This repository was archived by the owner on Aug 30, 2024. It is now read-only.

Add environment status to "envs ls" #108

Merged
merged 1 commit into from
Aug 31, 2020
Merged

Add environment status to "envs ls" #108

merged 1 commit into from
Aug 31, 2020

Conversation

cmoog
Copy link
Contributor

@cmoog cmoog commented Aug 31, 2020 •
edited
Loading

Closes #106

Example usage

$ coder envs ls                                                                         
Name                ImageTag    CPUCores    MemoryGB    DiskGB    GPUs    Updating    Status    
dev                 latest      4           4           40        0       false       ON        
test-datascience    latest      4           4           10        0       false       OFF       
entdev              latest      2           4           11        0       false       OFF       
test-goland         latest      2           2           30        0       false       OFF

@cmoog cmoog changed the title Add environment status to "envs ls table" Add environment status to "envs ls" Aug 31, 2020
@cmoog cmoog requested a review from creack August 31, 2020 20:58
creack
creack approved these changes Aug 31, 2020
View reviewed changes
Copy link
Contributor

@creack creack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tiny nit, but LGTM

coder-sdk/env.go
DiskGB int `json:"disk_gb"`
GPUs int `json:"gpus"`
Updating bool `json:"updating"`
ID string `json:"id" tab:"-"`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: It would be neat to align the tags.

@cmoog cmoog merged commit e130f8b into master Aug 31, 2020
@cmoog cmoog deleted the envs-status branch August 31, 2020 21:11
cmoog added a commit that referenced this pull request Aug 31, 2020
@cmoog
fixup! Merge pull request #108 from cdr/envs-status
9e9cf93
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@creack creack creack approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add environment status in list
2 participants
@cmoog @creack