Skip to content

fix: update workspace TTL on template TTL change #15761

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

Merged
merged 7 commits into from
Dec 6, 2024

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
30 checks passed
@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