Skip to content

Commit 1ad86c1

Browse files
jopemachineclaude
andcommitted
feat(BA-6628): add allow-list rank update across the v2 surface
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent d517493 commit 1ad86c1

2 files changed

Lines changed: 18 additions & 1 deletion

File tree

changes/12517.feature.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
Expose the app config allow-list merge `rank` on the v2 REST/GraphQL APIs and CLI — optional create input (scope-type default when omitted), node field, and order field
1+
Expose the app config allow-list merge `rank` on the v2 REST/GraphQL APIs and CLI — optional create input (scope-type default when omitted), node field, order field, and a new admin update operation (`PATCH /v2/app-config-allow-list/{id}`, `adminUpdateAppConfigAllowList`, `./bai admin app-config-allow-list update --rank`) for re-ordering the merge

src/ai/backend/common/dto/manager/v2/app_config_allow_list/request.py

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,23 @@ class CreateAppConfigAllowListInput(BaseRequestModel):
4646
)
4747

4848

49+
class UpdateAppConfigAllowListInput(BaseRequestModel):
50+
"""Input for updating an app config allow-list entry.
51+
52+
Only ``rank`` is updatable — the identity pair (``config_name``, ``scope_type``)
53+
is immutable (purge and recreate to change it).
54+
"""
55+
56+
id: UUID = Field(description="App config allow-list entry id to update.")
57+
rank: int | None = Field(
58+
default=None,
59+
description=(
60+
"New merge rank applied to fragments under this entry (low to high; "
61+
"higher wins). Omit to leave unchanged."
62+
),
63+
)
64+
65+
4966
class PurgeAppConfigAllowListInput(BaseRequestModel):
5067
"""Input for purging an app config allow-list entry."""
5168

0 commit comments

Comments
 (0)