Skip to content

Commit 7f876b3

Browse files
jopemachineclaude
andcommitted
docs(BA-6041): deprecate MountOption.mount_destination for session creation paths
This field originally entered the per-vfolder option model in PR #2168 (service-creation wire schema only). Session creation never read it — mount_map / mount_id_map carry destinations there. Mark it deprecated and state explicitly that it applies to inference service creation only, so callers stop putting it under session mount_options where it is silently ignored. Follow-up to BA-6040 / PR #11608. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent f1a770c commit 7f876b3

1 file changed

Lines changed: 11 additions & 1 deletion

File tree

  • src/ai/backend/common/dto/manager/session

src/ai/backend/common/dto/manager/session/types.py

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -154,7 +154,17 @@ class MountOption(BaseFieldModel):
154154

155155
mount_destination: str | None = Field(
156156
default=None,
157-
description="Mount destination inside the container. Defaults to ``/home/work/{folder_name}``.",
157+
deprecated=(
158+
"Inline mount_destination is supported only by the inference service creation"
159+
" wire schema (``ServiceConfigModel.extra_mounts``) for legacy reasons. Session"
160+
" creation (``CreationConfigV*.mount_options``) does NOT read this field — use"
161+
" ``mount_map`` / ``mount_id_map`` to specify destinations for sessions."
162+
),
163+
description=(
164+
"Mount destination inside the container. Defaults to ``/home/work/{folder_name}``."
165+
" **Service creation only** — ignored on session creation paths; supply session"
166+
" destinations via ``mount_map`` / ``mount_id_map`` instead."
167+
),
158168
)
159169
type: MountTypes = MountTypes.BIND
160170
permission: MountPermission | None = Field(

0 commit comments

Comments
 (0)