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

Skip to content

fix: update workspace TTL on template TTL change#15761

Merged
DanielleMaywood merged 7 commits intomainfrom
dm-workspace-template-ttl
Dec 6, 2024
Merged

fix: update workspace TTL on template TTL change#15761
DanielleMaywood merged 7 commits intomainfrom
dm-workspace-template-ttl

Conversation

@DanielleMaywood
Copy link
Contributor

Relates to #15390

Currently when a user creates a workspace, their workspace's TTL is determined by the template's default TTL. If the Coder instance is AGPL, or if the template has disallowed the user from configuring autostop, then it is not possible to change the workspace's TTL after creation. Any changes to the template's default TTL only takes effect on new workspaces.

This PR modifies the behaviour slightly so that on AGPL Coder, or on enterprise when a template does not allow user's to configure their workspace's TTL, updating the template's default TTL will also update any workspace's TTL to match this value.

@DanielleMaywood DanielleMaywood force-pushed the dm-workspace-template-ttl branch from aa0b769 to 021a806 Compare December 5, 2024 10:15
@DanielleMaywood DanielleMaywood marked this pull request as ready for review December 5, 2024 14:58
Copy link
Member

@mafredri mafredri left a comment

Choose a reason for hiding this comment

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

LGTM, only minor suggestions/nits.

@DanielleMaywood DanielleMaywood merged commit 40624bf into main Dec 6, 2024
@DanielleMaywood DanielleMaywood deleted the dm-workspace-template-ttl branch December 6, 2024 11:01
aslilac pushed a commit that referenced this pull request Dec 11, 2024
Relates to #15390

Currently when a user creates a workspace, their workspace's TTL is
determined by the template's default TTL. If the Coder instance is AGPL,
or if the template has disallowed the user from configuring autostop,
then it is not possible to change the workspace's TTL after creation.
Any changes to the template's default TTL only takes effect on _new_
workspaces.

This PR modifies the behaviour slightly so that on AGPL Coder, or on
enterprise when a template does not allow user's to configure their
workspace's TTL, updating the template's default TTL will also update
any workspace's TTL to match this value.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments