Skip to content

feat: update workspace deadline when workspace ttl updated #2165

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.

Sign up for GitHub

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

Merged
merged 20 commits into from
Jun 9, 2022
Merged

feat: update workspace deadline when workspace ttl updated #2165

Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
5c443de
RED: update unit test with new desired behaviour
johnstcn Jun 7, 2022
c2127c1
cli: warn user upon ttl-triggered workspace shutdown
johnstcn Jun 7, 2022
07320a4
GREEN: do the thing
johnstcn Jun 8, 2022
7846dd8
REFACTOR: coderd: putWorkspaceTTL: wrap in tx
johnstcn Jun 8, 2022
6d7187d
update unit tests
johnstcn Jun 8, 2022
daa9a5f
feat: WorkspaceScheduleForm: show shutdown time when editing workspac…
johnstcn Jun 8, 2022
fe79c0c
make it prettier
johnstcn Jun 8, 2022
85da57b
hopefully fix unit tests
johnstcn Jun 9, 2022
8c38a74
Merge remote-tracking branch 'origin/main' into cj/1783/updating-work…
johnstcn Jun 9, 2022
a4c07df
Address PR comments
johnstcn Jun 9, 2022
582873a
Merge origin/main
greyscaled Jun 9, 2022
9a7c623
chore: WorkspaceScheduleForm: add unit tests
johnstcn Jun 9, 2022
0278a93
WorkspaceScheduleForm: refactor ttlShutdownAt
johnstcn Jun 9, 2022
a372dce
apply suggestions from code review
johnstcn Jun 9, 2022
ac1d29a
fixup! apply suggestions from code review
johnstcn Jun 9, 2022
23b474c
yarn lint; more pr comments
johnstcn Jun 9, 2022
de28477
bump ci
johnstcn Jun 9, 2022
a52d815
Merge remote-tracking branch 'origin/main' into cj/1783/updating-work…
johnstcn Jun 9, 2022
fd86474
mind your language
johnstcn Jun 9, 2022
2b9f93c
fumpt
johnstcn Jun 9, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
hopefully fix unit tests
  • Loading branch information
@johnstcn
johnstcn committed Jun 9, 2022
commit 85da57bdc4753821b11adc762e1d66f72462510c
7 changes: 4 additions & 3 deletions coderd/workspaces_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -624,7 +624,7 @@ func TestWorkspaceUpdateTTL(t *testing.T) {
name: "update ttl",
ttlMillis: ptr.Ref(12 * time.Hour.Milliseconds()),
expectedError: "",
expectedDeadline: ptr.Ref(time.Now().Add(12 * time.Hour)),
expectedDeadline: ptr.Ref(time.Now().Add(12*time.Hour + time.Minute)),
},
{
name: "below minimum ttl",
Expand All @@ -635,13 +635,13 @@ func TestWorkspaceUpdateTTL(t *testing.T) {
name: "minimum ttl",
ttlMillis: ptr.Ref(time.Minute.Milliseconds()),
expectedError: "",
expectedDeadline: ptr.Ref(time.Now().Add(time.Minute)),
expectedDeadline: ptr.Ref(time.Now().Add(2 * time.Minute)),
},
{
name: "maximum ttl",
ttlMillis: ptr.Ref((24 * 7 * time.Hour).Milliseconds()),
expectedError: "",
expectedDeadline: ptr.Ref(time.Now().Add(24 * 7 * time.Hour)),
expectedDeadline: ptr.Ref(time.Now().Add(24*7*time.Hour + time.Minute)),
},
{
name: "above maximum ttl",
Expand Down Expand Up @@ -676,6 +676,7 @@ func TestWorkspaceUpdateTTL(t *testing.T) {
cwr.AutostartSchedule = nil
cwr.TTLMillis = nil
})
_ = coderdtest.AwaitWorkspaceBuildJob(t, client, workspace.LatestBuild.ID)
)

err := client.UpdateWorkspaceTTL(ctx, workspace.ID, codersdk.UpdateWorkspaceTTLRequest{
Expand Down