openstack/cyborg
Code Issues Proposed changes

Mark conductor ARQ delete methods for removal in RPC v2

Browse Source 
ARQ deletion is now performed directly in the API layer
to enforce service token checks. The conductor RPC
endpoints are no longer called but must remain until RPC
API v2 to preserve compatibility during rolling upgrades.

Add TODO notes to remove arq_delete_by_uuid and
arq_delete_by_instance_uuid after 2027.1.

Related-Bug: #2144056
Generated-By: Cursor claude-opus-4.6
Signed-off-by: Sean Mooney <work@seanmooney.info>
Change-Id: Icaaf90bec6e9540fda492f981f80c35b846eeb73
This commit is contained in:
Sean Mooney
2026-03-13 14:48:52 +00:00
parent 359d17e680
commit 4e993d525e
2 changed files with 10 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
cyborg/conductor/manager.py
+5
View File
@@ -94,6 +94,11 @@ class ConductorManager:
obj_extarq.create(context, devprof_id)
return obj_extarq
# TODO(sean-k-mooney): Remove arq_delete_by_uuid and
# arq_delete_by_instance_uuid in RPC API v2 (after 2027.1). ARQ
# deletion is now performed directly in the API layer to enforce
# service token checks, but these endpoints must remain for RPC
# version compatibility during rolling upgrades.
def arq_delete_by_uuid(self, context, arqs):
"""Signal to conductor service to delete accelerator requests by
ARQ UUIDs.
cyborg/conductor/rpcapi.py
+5
View File
@@ -93,6 +93,11 @@ class ConductorAPI:
context, 'arq_create', obj_extarq=obj_extarq, devprof_id=devprof_id
)
# TODO(sean-k-mooney): Remove arq_delete_by_uuid and
# arq_delete_by_instance_uuid in RPC API v2 (after 2027.1). ARQ
# deletion is now performed directly in the API layer to enforce
# service token checks, but these endpoints must remain for RPC
# version compatibility during rolling upgrades.
def arq_delete_by_uuid(self, context, arqs):
"""Signal to conductor service to delete accelerator requests by
ARQ UUIDs.