)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"3fb90b5192773cf0eb5fe77e55e6744186507704","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2fd1654b_11f25a15","updated":"2026-06-24 01:46:46.000000000","message":"recheck","commit_id":"aeb5edf2ace71a9769b4c1443274aae10e58b10a"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"25bb2a3989dd1054c43f826cb4bc05be88a06627","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b3c60f86_97417920","updated":"2026-08-11 12:51:48.000000000","message":"recheck","commit_id":"6e3fd6c4d325090d7de8090ea1a8b97823b14c7a"}],"nova/accelerator/cyborg.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"5e32cd792f62111cc976391550c45e6468594d2d","unresolved":true,"context_lines":[{"line_number":11,"context_line":"#    WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the"},{"line_number":12,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":13,"context_line":"#    under the License."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"\"\"\""},{"line_number":16,"context_line":"   Note on object relationships:"},{"line_number":17,"context_line":"   1 device profile (DP) has D \u003e\u003d 1 request groups (just as a flavor"}],"source_content_type":"text/x-python","patch_set":1,"id":"0cc29525_55cc811c","line":14,"updated":"2026-07-07 10:28:30.000000000","message":"as an aside since this module was create the openstacksdk has gained supprot for both placement and cybrog\nim hoping to port both form using the raw keystoneauth client to using the sdk and starting the move to using the nova user and service creditals form our config after this feature is complete and the cybrog srbac functionaltiy is merged.\n\nthat might now land this cycle but i do not wnat to consider that a depency for this functionality as i need to do some pre work to do that properly in terms fo ci testing ectra so would rather not rush that.","commit_id":"aeb5edf2ace71a9769b4c1443274aae10e58b10a"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"bd86ea3914ed28e08ed372314130d78064b3cdc1","unresolved":false,"context_lines":[{"line_number":11,"context_line":"#    WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the"},{"line_number":12,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":13,"context_line":"#    under the License."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"\"\"\""},{"line_number":16,"context_line":"   Note on object relationships:"},{"line_number":17,"context_line":"   1 device profile (DP) has D \u003e\u003d 1 request groups (just as a flavor"}],"source_content_type":"text/x-python","patch_set":1,"id":"bc404a06_96ba47da","line":14,"in_reply_to":"0cc29525_55cc811c","updated":"2026-08-11 17:33:01.000000000","message":"Acknowledged","commit_id":"aeb5edf2ace71a9769b4c1443274aae10e58b10a"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"866fc9675fed164cc48626dfb8cfaf4066b3acf6","unresolved":true,"context_lines":[{"line_number":140,"context_line":"        return _cached_cyborg_max_microversion"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"    try:"},{"line_number":143,"context_line":"        endpoint_data \u003d client.get_endpoint_data()"},{"line_number":144,"context_line":"        max_ver \u003d endpoint_data.max_microversion"},{"line_number":145,"context_line":"        LOG.info(\u0027Cyborg API max microversion: %s\u0027, max_ver)"},{"line_number":146,"context_line":"        if max_ver is not None:"},{"line_number":147,"context_line":"            _cached_cyborg_max_microversion \u003d max_ver"}],"source_content_type":"text/x-python","patch_set":2,"id":"959e7aca_5661b7b4","line":144,"range":{"start_line":143,"start_character":7,"end_line":144,"end_character":48},"updated":"2026-08-11 17:27:24.000000000","message":"this is not correct. it end ups falling back to 2.0 semantics.","commit_id":"6e3fd6c4d325090d7de8090ea1a8b97823b14c7a"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"cf795ef319b7c1e863b3dd42d662dd092d167ef9","unresolved":false,"context_lines":[{"line_number":140,"context_line":"        return _cached_cyborg_max_microversion"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"    try:"},{"line_number":143,"context_line":"        endpoint_data \u003d client.get_endpoint_data()"},{"line_number":144,"context_line":"        max_ver \u003d endpoint_data.max_microversion"},{"line_number":145,"context_line":"        LOG.info(\u0027Cyborg API max microversion: %s\u0027, max_ver)"},{"line_number":146,"context_line":"        if max_ver is not None:"},{"line_number":147,"context_line":"            _cached_cyborg_max_microversion \u003d max_ver"}],"source_content_type":"text/x-python","patch_set":2,"id":"d3ca6d66_f4f6b215","line":144,"range":{"start_line":143,"start_character":7,"end_line":144,"end_character":48},"in_reply_to":"959e7aca_5661b7b4","updated":"2026-08-12 12:40:35.000000000","message":"fixed.\n\ni will work on moving cyborg and placement to useing the sdk expclitvly next cycel but for now ill resolve this","commit_id":"6e3fd6c4d325090d7de8090ea1a8b97823b14c7a"}],"nova/tests/functional/libvirt/test_accel_servers.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"e3ee9b0a12098657943f9150f964438503719a69","unresolved":false,"context_lines":[{"line_number":21,"context_line":"from lxml import etree"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"import nova.virt.libvirt.guest"},{"line_number":24,"context_line":"from nova.tests import fixtures as nova_fixtures"},{"line_number":25,"context_line":"from nova.tests.fixtures import libvirt as fakelibvirt"},{"line_number":26,"context_line":"from nova.tests.fixtures.cyborg import get_arqs"},{"line_number":27,"context_line":"from nova.tests.functional.libvirt import base"}],"source_content_type":"text/x-python","patch_set":1,"id":"49a9c300_94adb856","line":24,"updated":"2026-06-24 04:44:48.000000000","message":"pep8: H306: imports not in alphabetical order (nova.virt.libvirt.guest, nova.tests.fixtures)","commit_id":"aeb5edf2ace71a9769b4c1443274aae10e58b10a"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"5e32cd792f62111cc976391550c45e6468594d2d","unresolved":true,"context_lines":[{"line_number":21,"context_line":"from lxml import etree"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"import nova.virt.libvirt.guest"},{"line_number":24,"context_line":"from nova.tests import fixtures as nova_fixtures"},{"line_number":25,"context_line":"from nova.tests.fixtures import libvirt as fakelibvirt"},{"line_number":26,"context_line":"from nova.tests.fixtures.cyborg import get_arqs"},{"line_number":27,"context_line":"from nova.tests.functional.libvirt import base"}],"source_content_type":"text/x-python","patch_set":1,"id":"665d2903_4337e0fb","line":24,"in_reply_to":"49a9c300_94adb856","updated":"2026-07-07 10:28:30.000000000","message":"sigh...","commit_id":"aeb5edf2ace71a9769b4c1443274aae10e58b10a"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"bd86ea3914ed28e08ed372314130d78064b3cdc1","unresolved":false,"context_lines":[{"line_number":21,"context_line":"from lxml import etree"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"import nova.virt.libvirt.guest"},{"line_number":24,"context_line":"from nova.tests import fixtures as nova_fixtures"},{"line_number":25,"context_line":"from nova.tests.fixtures import libvirt as fakelibvirt"},{"line_number":26,"context_line":"from nova.tests.fixtures.cyborg import get_arqs"},{"line_number":27,"context_line":"from nova.tests.functional.libvirt import base"}],"source_content_type":"text/x-python","patch_set":1,"id":"1394ed29_d0f80544","line":24,"in_reply_to":"665d2903_4337e0fb","updated":"2026-08-11 17:33:01.000000000","message":"Done","commit_id":"aeb5edf2ace71a9769b4c1443274aae10e58b10a"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"e3ee9b0a12098657943f9150f964438503719a69","unresolved":false,"context_lines":[{"line_number":23,"context_line":"import nova.virt.libvirt.guest"},{"line_number":24,"context_line":"from nova.tests import fixtures as nova_fixtures"},{"line_number":25,"context_line":"from nova.tests.fixtures import libvirt as fakelibvirt"},{"line_number":26,"context_line":"from nova.tests.fixtures.cyborg import get_arqs"},{"line_number":27,"context_line":"from nova.tests.functional.libvirt import base"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"a1ef382f_4b713df0","line":26,"updated":"2026-06-24 04:44:48.000000000","message":"pep8: H306: imports not in alphabetical order (nova.tests.fixtures.libvirt, nova.tests.fixtures.cyborg.get_arqs)","commit_id":"aeb5edf2ace71a9769b4c1443274aae10e58b10a"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"bd86ea3914ed28e08ed372314130d78064b3cdc1","unresolved":false,"context_lines":[{"line_number":23,"context_line":"import nova.virt.libvirt.guest"},{"line_number":24,"context_line":"from nova.tests import fixtures as nova_fixtures"},{"line_number":25,"context_line":"from nova.tests.fixtures import libvirt as fakelibvirt"},{"line_number":26,"context_line":"from nova.tests.fixtures.cyborg import get_arqs"},{"line_number":27,"context_line":"from nova.tests.functional.libvirt import base"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"01368086_3ed52454","line":26,"in_reply_to":"717c8e62_fcf2bc18","updated":"2026-08-11 17:33:01.000000000","message":"Done","commit_id":"aeb5edf2ace71a9769b4c1443274aae10e58b10a"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"5e32cd792f62111cc976391550c45e6468594d2d","unresolved":true,"context_lines":[{"line_number":23,"context_line":"import nova.virt.libvirt.guest"},{"line_number":24,"context_line":"from nova.tests import fixtures as nova_fixtures"},{"line_number":25,"context_line":"from nova.tests.fixtures import libvirt as fakelibvirt"},{"line_number":26,"context_line":"from nova.tests.fixtures.cyborg import get_arqs"},{"line_number":27,"context_line":"from nova.tests.functional.libvirt import base"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"717c8e62_fcf2bc18","line":26,"in_reply_to":"a1ef382f_4b713df0","updated":"2026-07-07 10:28:30.000000000","message":"this is why i hate hackigns order... but this is vaild","commit_id":"aeb5edf2ace71a9769b4c1443274aae10e58b10a"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"5e32cd792f62111cc976391550c45e6468594d2d","unresolved":true,"context_lines":[{"line_number":54,"context_line":"        # fake driver but is ignored by the real libvirt driver."},{"line_number":55,"context_line":"        for arq in bound_arq_list:"},{"line_number":56,"context_line":"            arq[\u0027attach_handle_type\u0027] \u003d \u0027PCI\u0027"},{"line_number":57,"context_line":"        nova_fixtures.CyborgFixture.arq_list \u003d arq_list"},{"line_number":58,"context_line":"        nova_fixtures.CyborgFixture.bound_arq_list \u003d bound_arq_list"},{"line_number":59,"context_line":"        nova_fixtures.CyborgFixture.arq_uuids \u003d ["},{"line_number":60,"context_line":"            arq[\u0027uuid\u0027] for arq in arq_list]"},{"line_number":61,"context_line":"        self.cyborg.call_create_arq_count \u003d 0"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    def _start_compute_with_device_rp(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"571f80cd_a9d9981b","line":60,"range":{"start_line":57,"start_character":6,"end_line":60,"end_character":44},"updated":"2026-07-07 10:28:30.000000000","message":"asigned to fixture is not the right approch,\n\nwe awnat ot alwasy mange this though fixutres to ensure not cross test sharing.","commit_id":"aeb5edf2ace71a9769b4c1443274aae10e58b10a"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"bd86ea3914ed28e08ed372314130d78064b3cdc1","unresolved":false,"context_lines":[{"line_number":54,"context_line":"        # fake driver but is ignored by the real libvirt driver."},{"line_number":55,"context_line":"        for arq in bound_arq_list:"},{"line_number":56,"context_line":"            arq[\u0027attach_handle_type\u0027] \u003d \u0027PCI\u0027"},{"line_number":57,"context_line":"        nova_fixtures.CyborgFixture.arq_list \u003d arq_list"},{"line_number":58,"context_line":"        nova_fixtures.CyborgFixture.bound_arq_list \u003d bound_arq_list"},{"line_number":59,"context_line":"        nova_fixtures.CyborgFixture.arq_uuids \u003d ["},{"line_number":60,"context_line":"            arq[\u0027uuid\u0027] for arq in arq_list]"},{"line_number":61,"context_line":"        self.cyborg.call_create_arq_count \u003d 0"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    def _start_compute_with_device_rp(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"bfda84b9_9ddbd4e9","line":60,"range":{"start_line":57,"start_character":6,"end_line":60,"end_character":44},"in_reply_to":"571f80cd_a9d9981b","updated":"2026-08-11 17:33:01.000000000","message":"Done","commit_id":"aeb5edf2ace71a9769b4c1443274aae10e58b10a"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"5e32cd792f62111cc976391550c45e6468594d2d","unresolved":true,"context_lines":[{"line_number":41,"context_line":"        super().setUp()"},{"line_number":42,"context_line":"        self.cyborg \u003d self.useFixture(nova_fixtures.CyborgFixture())"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    def _reinit_cyborg_arqs(self, managed_mode\u003dNone):"},{"line_number":45,"context_line":"        \"\"\"Re-initialize CyborgFixture ARQ lists with managed_mode."},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"        Also sets attach_handle_type to \u0027PCI\u0027 (the libvirt driver"},{"line_number":48,"context_line":"        only accepts \u0027PCI\u0027, not the default \u0027TEST_PCI\u0027)."},{"line_number":49,"context_line":"        \"\"\""},{"line_number":50,"context_line":"        arq_list, bound_arq_list \u003d get_arqs("},{"line_number":51,"context_line":"            self.cyborg.dp_name, managed_mode\u003dmanaged_mode)"},{"line_number":52,"context_line":"        # The libvirt driver filters by attach_handle_type \u003d\u003d \u0027PCI\u0027;"},{"line_number":53,"context_line":"        # the default fixture uses \u0027TEST_PCI\u0027 which works with the"},{"line_number":54,"context_line":"        # fake driver but is ignored by the real libvirt driver."},{"line_number":55,"context_line":"        for arq in bound_arq_list:"},{"line_number":56,"context_line":"            arq[\u0027attach_handle_type\u0027] \u003d \u0027PCI\u0027"},{"line_number":57,"context_line":"        nova_fixtures.CyborgFixture.arq_list \u003d arq_list"},{"line_number":58,"context_line":"        nova_fixtures.CyborgFixture.bound_arq_list \u003d bound_arq_list"},{"line_number":59,"context_line":"        nova_fixtures.CyborgFixture.arq_uuids \u003d ["},{"line_number":60,"context_line":"            arq[\u0027uuid\u0027] for arq in arq_list]"},{"line_number":61,"context_line":"        self.cyborg.call_create_arq_count \u003d 0"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    def _start_compute_with_device_rp(self):"},{"line_number":64,"context_line":"        \"\"\"Start a libvirt compute and create a Cyborg device RP.\"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"25d643e1_70eb3bae","line":61,"range":{"start_line":44,"start_character":2,"end_line":61,"end_character":45},"updated":"2026-07-07 10:28:30.000000000","message":"hum i can proably do this cleaner.\n\nim concerne this will leak between tests since im not resetting this","commit_id":"aeb5edf2ace71a9769b4c1443274aae10e58b10a"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"bd86ea3914ed28e08ed372314130d78064b3cdc1","unresolved":false,"context_lines":[{"line_number":41,"context_line":"        super().setUp()"},{"line_number":42,"context_line":"        self.cyborg \u003d self.useFixture(nova_fixtures.CyborgFixture())"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    def _reinit_cyborg_arqs(self, managed_mode\u003dNone):"},{"line_number":45,"context_line":"        \"\"\"Re-initialize CyborgFixture ARQ lists with managed_mode."},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"        Also sets attach_handle_type to \u0027PCI\u0027 (the libvirt driver"},{"line_number":48,"context_line":"        only accepts \u0027PCI\u0027, not the default \u0027TEST_PCI\u0027)."},{"line_number":49,"context_line":"        \"\"\""},{"line_number":50,"context_line":"        arq_list, bound_arq_list \u003d get_arqs("},{"line_number":51,"context_line":"            self.cyborg.dp_name, managed_mode\u003dmanaged_mode)"},{"line_number":52,"context_line":"        # The libvirt driver filters by attach_handle_type \u003d\u003d \u0027PCI\u0027;"},{"line_number":53,"context_line":"        # the default fixture uses \u0027TEST_PCI\u0027 which works with the"},{"line_number":54,"context_line":"        # fake driver but is ignored by the real libvirt driver."},{"line_number":55,"context_line":"        for arq in bound_arq_list:"},{"line_number":56,"context_line":"            arq[\u0027attach_handle_type\u0027] \u003d \u0027PCI\u0027"},{"line_number":57,"context_line":"        nova_fixtures.CyborgFixture.arq_list \u003d arq_list"},{"line_number":58,"context_line":"        nova_fixtures.CyborgFixture.bound_arq_list \u003d bound_arq_list"},{"line_number":59,"context_line":"        nova_fixtures.CyborgFixture.arq_uuids \u003d ["},{"line_number":60,"context_line":"            arq[\u0027uuid\u0027] for arq in arq_list]"},{"line_number":61,"context_line":"        self.cyborg.call_create_arq_count \u003d 0"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    def _start_compute_with_device_rp(self):"},{"line_number":64,"context_line":"        \"\"\"Start a libvirt compute and create a Cyborg device RP.\"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"53d7e7e5_b6e6e4a2","line":61,"range":{"start_line":44,"start_character":2,"end_line":61,"end_character":45},"in_reply_to":"25d643e1_70eb3bae","updated":"2026-08-11 17:33:01.000000000","message":"Done","commit_id":"aeb5edf2ace71a9769b4c1443274aae10e58b10a"}],"nova/tests/functional/test_servers.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"5e32cd792f62111cc976391550c45e6468594d2d","unresolved":true,"context_lines":[{"line_number":6972,"context_line":"        self.assertNotIn(\u0027pci_slot\u0027, binding_profile)"},{"line_number":6973,"context_line":""},{"line_number":6974,"context_line":""},{"line_number":6975,"context_line":""},{"line_number":6976,"context_line":""},{"line_number":6977,"context_line":"class PortBindingShelvedServerTest(integrated_helpers._IntegratedTestBase):"},{"line_number":6978,"context_line":"    \"\"\"Tests for servers with ports.\"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"a0c41924_f8556789","line":6975,"updated":"2026-07-07 10:28:30.000000000","message":"This shoudl not be neded. we expect 2 blank lines not 4","commit_id":"aeb5edf2ace71a9769b4c1443274aae10e58b10a"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"bd86ea3914ed28e08ed372314130d78064b3cdc1","unresolved":false,"context_lines":[{"line_number":6972,"context_line":"        self.assertNotIn(\u0027pci_slot\u0027, binding_profile)"},{"line_number":6973,"context_line":""},{"line_number":6974,"context_line":""},{"line_number":6975,"context_line":""},{"line_number":6976,"context_line":""},{"line_number":6977,"context_line":"class PortBindingShelvedServerTest(integrated_helpers._IntegratedTestBase):"},{"line_number":6978,"context_line":"    \"\"\"Tests for servers with ports.\"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"e24fd446_eb735202","line":6975,"in_reply_to":"a0c41924_f8556789","updated":"2026-08-11 17:33:01.000000000","message":"Done","commit_id":"aeb5edf2ace71a9769b4c1443274aae10e58b10a"}],"nova/tests/unit/accelerator/test_cyborg.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"5e32cd792f62111cc976391550c45e6468594d2d","unresolved":true,"context_lines":[{"line_number":552,"context_line":"    @mock.patch(\u0027requests.get\u0027)"},{"line_number":553,"context_line":"    def test_get_cyborg_max_microversion(self, mock_requests_get):"},{"line_number":554,"context_line":"        \"\"\"Test that version discovery fetches and caches max_version.\"\"\""},{"line_number":555,"context_line":"        discovery_body \u003d {"},{"line_number":556,"context_line":"            \u0027default_version\u0027: {"},{"line_number":557,"context_line":"                \u0027id\u0027: \u0027v2\u0027, \u0027status\u0027: \u0027CURRENT\u0027,"},{"line_number":558,"context_line":"                \u0027max_version\u0027: \u00272.4\u0027, \u0027min_version\u0027: \u00272.0\u0027,"},{"line_number":559,"context_line":"            }"},{"line_number":560,"context_line":"        }"},{"line_number":561,"context_line":"        mock_resp \u003d mock.Mock()"},{"line_number":562,"context_line":"        mock_resp.status_code \u003d 200"},{"line_number":563,"context_line":"        mock_resp.json.return_value \u003d discovery_body"},{"line_number":564,"context_line":"        mock_requests_get.return_value \u003d mock_resp"},{"line_number":565,"context_line":""},{"line_number":566,"context_line":"        result \u003d cyborg._get_cyborg_max_microversion("},{"line_number":567,"context_line":"            \u0027http://example.com/accelerator\u0027, \u0027fake-token\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"61b78f4b_c061bf18","line":564,"range":{"start_line":555,"start_character":1,"end_line":564,"end_character":50},"updated":"2026-07-07 10:28:30.000000000","message":"this is not following the existing pattehr we shoudl use our fake responce objects\n\n```\n        query \u003d {\"instance\": instance_uuid}\n        content \u003d jsonutils.dumps({\u0027arqs\u0027: bound_arqs})\n        resp \u003d fake_requests.FakeResponse(200, content)\n```","commit_id":"aeb5edf2ace71a9769b4c1443274aae10e58b10a"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"bd86ea3914ed28e08ed372314130d78064b3cdc1","unresolved":false,"context_lines":[{"line_number":552,"context_line":"    @mock.patch(\u0027requests.get\u0027)"},{"line_number":553,"context_line":"    def test_get_cyborg_max_microversion(self, mock_requests_get):"},{"line_number":554,"context_line":"        \"\"\"Test that version discovery fetches and caches max_version.\"\"\""},{"line_number":555,"context_line":"        discovery_body \u003d {"},{"line_number":556,"context_line":"            \u0027default_version\u0027: {"},{"line_number":557,"context_line":"                \u0027id\u0027: \u0027v2\u0027, \u0027status\u0027: \u0027CURRENT\u0027,"},{"line_number":558,"context_line":"                \u0027max_version\u0027: \u00272.4\u0027, \u0027min_version\u0027: \u00272.0\u0027,"},{"line_number":559,"context_line":"            }"},{"line_number":560,"context_line":"        }"},{"line_number":561,"context_line":"        mock_resp \u003d mock.Mock()"},{"line_number":562,"context_line":"        mock_resp.status_code \u003d 200"},{"line_number":563,"context_line":"        mock_resp.json.return_value \u003d discovery_body"},{"line_number":564,"context_line":"        mock_requests_get.return_value \u003d mock_resp"},{"line_number":565,"context_line":""},{"line_number":566,"context_line":"        result \u003d cyborg._get_cyborg_max_microversion("},{"line_number":567,"context_line":"            \u0027http://example.com/accelerator\u0027, \u0027fake-token\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"8555215d_a7f03936","line":564,"range":{"start_line":555,"start_character":1,"end_line":564,"end_character":50},"in_reply_to":"61b78f4b_c061bf18","updated":"2026-08-11 17:33:01.000000000","message":"Done","commit_id":"aeb5edf2ace71a9769b4c1443274aae10e58b10a"}],"nova/tests/unit/virt/libvirt/test_driver.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"5e32cd792f62111cc976391550c45e6468594d2d","unresolved":true,"context_lines":[{"line_number":23443,"context_line":""},{"line_number":23444,"context_line":"    def test_guest_add_accel_pci_devices_managed_false(self):"},{"line_number":23445,"context_line":"        \"\"\"Cyborg ARQ with managed\u003dfalse produces managed\u003d\u0027no\u0027 XML.\"\"\""},{"line_number":23446,"context_line":"        self.flags(virt_type\u003d\u0027kvm\u0027, group\u003d\u0027libvirt\u0027)"},{"line_number":23447,"context_line":"        drvr \u003d libvirt_driver.LibvirtDriver(fake.FakeVirtAPI(), False)"},{"line_number":23448,"context_line":"        guest \u003d vconfig.LibvirtConfigGuest()"},{"line_number":23449,"context_line":"        accel_info \u003d [{"}],"source_content_type":"text/x-python","patch_set":1,"id":"a15b09fb_c84c15e0","line":23446,"range":{"start_line":23446,"start_character":6,"end_line":23446,"end_character":52},"updated":"2026-07-07 10:28:30.000000000","message":"this shoudl not actully be reuired\n\ni belive sriov work wiht virt type qemu so this is really just addign noise","commit_id":"aeb5edf2ace71a9769b4c1443274aae10e58b10a"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"cf795ef319b7c1e863b3dd42d662dd092d167ef9","unresolved":false,"context_lines":[{"line_number":23443,"context_line":""},{"line_number":23444,"context_line":"    def test_guest_add_accel_pci_devices_managed_false(self):"},{"line_number":23445,"context_line":"        \"\"\"Cyborg ARQ with managed\u003dfalse produces managed\u003d\u0027no\u0027 XML.\"\"\""},{"line_number":23446,"context_line":"        self.flags(virt_type\u003d\u0027kvm\u0027, group\u003d\u0027libvirt\u0027)"},{"line_number":23447,"context_line":"        drvr \u003d libvirt_driver.LibvirtDriver(fake.FakeVirtAPI(), False)"},{"line_number":23448,"context_line":"        guest \u003d vconfig.LibvirtConfigGuest()"},{"line_number":23449,"context_line":"        accel_info \u003d [{"}],"source_content_type":"text/x-python","patch_set":1,"id":"d012fed1_1ab05188","line":23446,"range":{"start_line":23446,"start_character":6,"end_line":23446,"end_character":52},"in_reply_to":"a15b09fb_c84c15e0","updated":"2026-08-12 12:40:35.000000000","message":"Done","commit_id":"aeb5edf2ace71a9769b4c1443274aae10e58b10a"}]}