)]}'
{"id":"openstack%2Fnova~1002157","triplet_id":"openstack%2Fnova~master~I71a99aeb359673fe2a8f4ea8a3977a4ac1bfd27a","project":"openstack/nova","branch":"master","hashtags":[],"change_id":"I71a99aeb359673fe2a8f4ea8a3977a4ac1bfd27a","subject":"Switch Cyborg client from keystoneauth1 to openstacksdk","status":"ABANDONED","created":"2026-08-24 16:06:09.000000000","updated":"2026-08-24 17:03:41.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"d2fc1ca92d4e55d7a6dc9663972d06bd8866ad80","_number":1002157,"virtual_id_number":1002157,"owner":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"actions":{},"labels":{"Verified":{"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"CC":[{"_account_id":14384,"name":"Quobyte CI","email":"openstack-ci-external@quobyte.com","username":"quobyteci","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2026-08-24 17:03:41.000000000","updated_by":{"_account_id":14384,"name":"Quobyte CI","email":"openstack-ci-external@quobyte.com","username":"quobyteci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":14384,"name":"Quobyte CI","email":"openstack-ci-external@quobyte.com","username":"quobyteci","tags":["SERVICE_USER"]},"state":"CC"}],"messages":[{"id":"e72b918910048fc5b67d70932817a7754a5b0848","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2026-08-24 16:06:09.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"5dd11193d1a05092b44113b0b013b98e186b3300","tag":"autogenerated:gerrit:abandon","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2026-08-24 16:09:40.000000000","message":"Abandoned\n\nAI-assisted changed the Change-Id and I didn\u0027t notice it 🤬https://review.opendev.org/c/openstack/nova/+/1002007","accounts_in_message":[],"_revision_number":1},{"id":"d2fc1ca92d4e55d7a6dc9663972d06bd8866ad80","author":{"_account_id":14384,"name":"Quobyte CI","email":"openstack-ci-external@quobyte.com","username":"quobyteci","tags":["SERVICE_USER"]},"date":"2026-08-24 17:03:41.000000000","message":"Patch Set 1:\n\nBuild Successful \n\n* quobyteci:  nova_quobyteci : SUCCESS, logs at: https://oscilogs.quobyte.com/logs/I71a99aeb359673fe2a8f4ea8a3977a4ac1bfd27a .","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"ef392e93af6353088fd4f9532307ac022c68f9be","revisions":{"ef392e93af6353088fd4f9532307ac022c68f9be":{"kind":"REWORK","_number":1,"created":"2026-08-24 16:06:09.000000000","uploader":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"ref":"refs/changes/57/1002157/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/57/1002157/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/57/1002157/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/57/1002157/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/57/1002157/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/57/1002157/1"}}},"commit":{"parents":[{"commit":"ec47550235877e82f529f6a8c99f32dd9ea9c1a8","subject":"Merge \"libvirt: Check only x86_64 arch for SEV capability detection\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/ec47550235877e82f529f6a8c99f32dd9ea9c1a8"}]}],"author":{"name":"melanie witt","email":"melwittt@gmail.com","date":"2026-08-20 19:46:32.000000000","tz":-420},"committer":{"name":"melanie witt","email":"melwittt@gmail.com","date":"2026-08-24 16:04:56.000000000","tz":-420},"subject":"Switch Cyborg client from keystoneauth1 to openstacksdk","message":"Switch Cyborg client from keystoneauth1 to openstacksdk\n\nThe Cyborg (accelerator) client was using a raw keystoneauth1\nadapter for HTTP calls, with no microversion negotiation. Switch\nto openstacksdk via get_sdk_adapter(), matching the pattern used\nby the Ironic and Manila clients.\n\nThe SDK adapter is created per request context using\nget_sdk_adapter(\u0027accelerator\u0027, admin\u003dFalse, context\u003dcontext),\nwhich builds a connection authenticated with the user\u0027s token.\nCyborg API 2.0 requires the user\u0027s token to associate ARQs with\nthe correct project; admin credentials cannot be used until Nova\nadds support for API 2.1 which accepts an explicit project_id.\n\nBecause each request context carries a different user token, the\nadapter is cached as a per-instance lazy property on _CyborgClient\nrather than in a module-level global. This also eliminates the\nneed for reset_globals() in test teardown.\n\nThe [cyborg] config group registers with include_auth\u003dFalse since\nauthentication comes from the user\u0027s request context, not from\nservice credentials in nova.conf.\n\n_call_cyborg catches bare Exception rather than\nsdk_exc.SDKException because the old code caught\nks_exc.ClientException (the root of the entire KSA exception\nhierarchy) and SDK/KSA exceptions share no common ancestor.\n_call_cyborg is a deliberate error-conversion wrapper that never\nre-raises, so the broad catch is intentional.\n\nSeveral call sites previously created the Cyborg client\nunconditionally before checking whether accelerators were\ninvolved. These are tightened to defer client creation behind\naccelerator-presence guards:\n\n- delete_arqs_if_needed: early return when no device profile\n  and no ARQ UUIDs\n- _create_and_bind_arq_for_instance: conditional on device\n  profile or requested networks\n- _poll_shelved_instances: moved inside the device profile check\n\nAssisted-by: Claude claude-opus-4-6 claude-code-2.1.241\nChange-Id: I71a99aeb359673fe2a8f4ea8a3977a4ac1bfd27a\nSigned-off-by: melanie witt \u003cmelwittt@gmail.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/ef392e93af6353088fd4f9532307ac022c68f9be"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/ef392e93af6353088fd4f9532307ac022c68f9be"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[{"name":"Verified","description":"Verified in gate by CI","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Verified\u003dMAX AND -label:Verified\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Verified\u003dMAX","label:Verified\u003dMIN"],"atom_explanations":{"label:Verified\u003dMAX":"","label:Verified\u003dMIN":""}}},{"name":"Code-Review","description":"Code reviewed by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Code-Review\u003dMAX AND -label:Code-Review\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Code-Review\u003dMAX","label:Code-Review\u003dMIN"],"atom_explanations":{"label:Code-Review\u003dMAX":"","label:Code-Review\u003dMIN":""}}},{"name":"Review-Priority","description":"Review Priority","status":"NOT_APPLICABLE","is_legacy":false,"applicability_expression_result":{"fulfilled":false,"status":"FAIL"},"submittability_expression_result":{"expression":"is:true","fulfilled":true,"status":"NOT_EVALUATED","passing_atoms":[],"failing_atoms":[],"atom_explanations":{}}},{"name":"Workflow","description":"Approved for gate by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Workflow\u003dMAX AND -label:Workflow\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Workflow\u003dMAX","label:Workflow\u003dMIN"],"atom_explanations":{"label:Workflow\u003dMAX":"","label:Workflow\u003dMIN":""}}}]}
