Skip to content

Fix issue 91 - #5

Closed
andyne13 wants to merge 2 commits into
mainfrom
fix-issue-91
Closed

Fix issue 91#5
andyne13 wants to merge 2 commits into
mainfrom
fix-issue-91

Conversation

@andyne13

@andyne13 andyne13 commented Jul 2, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@antncy antncy closed this Jul 7, 2025
@antncy
antncy deleted the fix-issue-91 branch July 11, 2025 12:09
Matteovanypersele pushed a commit to Matteovanypersele/openrag that referenced this pull request Jul 6, 2026
…point mutations

linagora#5 Preset rename now migrates partition references atomically: PgPresetRepository.rename
   inserts the new-named row, repoints every partition that referenced the old name
   (partitions are a plain string ref with no FK), then drops the old row in one
   transaction. PresetService.update_preset rejects renaming onto an existing name.
   Previously a rename orphaned partitions on a preset name that no longer existed.

linagora#6 ModelEndpointService.update_model_endpoint evicts the 'default' alias client cache
   when the updated endpoint is the current default, so callers that resolved 'default'
   don't keep a stale client (mirrors set_default).

linagora#7 delete_model_endpoint promotes a surviving endpoint to default when the current
   default is deleted, so the 'default' alias keeps resolving instead of disappearing
   (load_all only adds it for an is_default row) and a later factory("default") raising.

Adds unit tests for all three plus a real-Postgres check of the rename migration.
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