feat(coderd): add model config coexistence#27344
Closed
mafredri wants to merge 1 commit into
Closed
Conversation
Keep global model IDs authoritative while seeding organization-owned copies for the later scoped API and runtime layers.
Member
Author
|
$Closing this stack because its merge-safe compatibility direction was incorrect. CODAGT-709 will be rebuilt as review-only dependent diffs whose combined head is merged atomically.\n\n> 🤖 This review was automatically generated with Coder Agents. |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Why
Organization-owned model APIs need real organization rows before CODAGT-714 can implement management and sharing. This migration introduces those rows without changing existing runtime behavior or invalidating global model IDs.
Global rows remain authoritative during coexistence. Each active organization receives fresh-ID inherited copies with lineage and an Everyone-group
readACL. Existing chats and settings continue to reference global IDs until later stack layers activate organization runtime behavior and perform the final remap.Depends on #27340, which stacks on #27338 and #27337.
Verification
make genmake fmtmake lintmake buildThe full test suite reached 28,019 tests and failed nine unrelated CLI dotfiles tests because
my-agent-shis absent from the environment. Changed database, migration, coderd, SDK, chatd, and enterprise packages passed.