Skip to content

Commit f127c60

Browse files
fregataaclaude
andauthored
fix(BA-5772): guard allowed_vfolder_hosts against missing resource_policy key (#11185)
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent b379479 commit f127c60

2 files changed

Lines changed: 5 additions & 4 deletions

File tree

changes/11185.fix.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
Fix `KeyError: 'allowed_vfolder_hosts'` when the keypair resource policy is missing during vfolder ownership changes.

src/ai/backend/manager/models/vfolder/row.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -795,8 +795,8 @@ async def get_allowed_vfolder_hosts_by_group(
795795
result_hosts = allowed_hosts | values
796796
allowed_hosts = result_hosts
797797
# Keypair Resource Policy's allowed_vfolder_hosts
798-
final_result: VFolderHostPermissionMap = (
799-
allowed_hosts | resource_policy["allowed_vfolder_hosts"]
798+
final_result: VFolderHostPermissionMap = allowed_hosts | resource_policy.get(
799+
"allowed_vfolder_hosts", VFolderHostPermissionMap()
800800
)
801801
return final_result
802802

@@ -854,8 +854,8 @@ async def get_allowed_vfolder_hosts_by_user(
854854
result_hosts = allowed_hosts | row.allowed_vfolder_hosts
855855
allowed_hosts = result_hosts
856856
# Keypair Resource Policy's allowed_vfolder_hosts
857-
final_result: VFolderHostPermissionMap = (
858-
allowed_hosts | resource_policy["allowed_vfolder_hosts"]
857+
final_result: VFolderHostPermissionMap = allowed_hosts | resource_policy.get(
858+
"allowed_vfolder_hosts", VFolderHostPermissionMap()
859859
)
860860
return final_result
861861

0 commit comments

Comments
 (0)