- Notifications
You must be signed in to change notification settings - Fork1k
chore: cherry-pick remaining PRs into2.22
#17851
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
Merged
Uh oh!
There was an error while loading.Please reload this page.
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
…17757)Used in combination withcoder/terraform-provider-coder#396This is required by both#17475 and#17571Operators may need to conditionalize their templates to perform certainoperations once a prebuilt workspace has been claimed. This value will**only** be set once a claim takes place and a subsequent `terraformapply` occurs. Any `terraform apply` runs thereafter will beindistinguishable from a normal run on a workspace.---------Signed-off-by: Danny Kopping <dannykopping@gmail.com>
Partially addressescoder/internal#593
This pull request allows coder workspace agents to be reinitialized whena prebuilt workspace is claimed by a user. This facilitates the transferof ownership between the anonymous prebuilds system user and the newowner of the workspace.Only a single agent per prebuilt workspace is supported for now, butplumbing has already been done to facilitate the seamless transition tomulti-agent support.---------Signed-off-by: Danny Kopping <dannykopping@gmail.com>Co-authored-by: Danny Kopping <dannykopping@gmail.com>Signed-off-by: Danny Kopping <dannykopping@gmail.com>
…17571)Closescoder/internal#369We can't know whether a replacement (i.e. drift of terraform stateleading to a resource needing to be deleted/recreated) will take placeapriori; we can only detect it at `plan` time, because the providerdecides whether a resource must be replaced and it cannot be inferredthrough static analysis of the template.**This is likely to be the most common gotcha with using prebuilds,since it requires a slight template modification to use prebuildseffectively**, so let's head this off before it's an issue forcustomers.Drift details will now be logged in the workspace build logs:Plus a notification will be sent to template admins when this situationarises:A new metric - `coderd_prebuilt_workspaces_resource_replacements_total`- will also increment each time a workspace encounters replacements.We only track _that_ a resource replacement occurred, not how many. Justone is enough to ruin a prebuild, but we can't know apriori whichreplacement would cause this.For example, say we have 2 replacements: a `docker_container` and a`null_resource`; we don't know which one mightcause an issue (or indeed if either would), so we just track thereplacement.---------Signed-off-by: Danny Kopping <dannykopping@gmail.com>
We should only be tracking resource replacements during a prebuildclaim.Signed-off-by: Danny Kopping <dannykopping@gmail.com>
Also add some clarification about the lack of database constraints forsoft template deletion.---------Signed-off-by: Danny Kopping <dannykopping@gmail.com>Co-authored-by: Danny Kopping <dannykopping@gmail.com>
🚀 Deploying PR 17851 ... |
2.22
2.22
Signed-off-by: Danny Kopping <dannykopping@gmail.com>
stirby approved these changesMay 15, 2025
ffccfb9
intorelease/2.22 31 of 33 checks passed
Uh oh!
There was an error while loading.Please reload this page.
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.