Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
DanielleMaywood merged 7 commits intomainfromdm-workspace-template-ttl
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 onnew 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.

@DanielleMaywoodDanielleMaywood marked this pull request as ready for reviewDecember 5, 2024 14:58
Copy link
Member

@mafredrimafredri left a comment
edited
Loading

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.

@DanielleMaywoodDanielleMaywood merged commit40624bf intomainDec 6, 2024
30 checks passed
@DanielleMaywoodDanielleMaywood deleted the dm-workspace-template-ttl branchDecember 6, 2024 11:01
aslilac pushed a commit that referenced this pull requestDec 11, 2024
Relates to#15390Currently when a user creates a workspace, their workspace's TTL isdetermined 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 onenterprise when a template does not allow user's to configure theirworkspace's TTL, updating the template's default TTL will also updateany workspace's TTL to match this value.
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@mafredrimafredrimafredri approved these changes

@EmyrkEmyrkAwaiting requested review from Emyrk

Assignees

@DanielleMaywoodDanielleMaywood

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@DanielleMaywood@mafredri

[8]ページ先頭

©2009-2025 Movatter.jp