)]}'
{"cyborg/objects/ext_arq.py":[{"author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"e52cddf224c87adc2d7fa2c4abe5799ab4675dca","unresolved":false,"context_lines":[{"line_number":266,"context_line":"        # if (self.arq.state \u003d\u003d constants.ARQ_DELETING"},{"line_number":267,"context_line":"        #         or self.arq.state \u003d\u003d ARQ_UNBOUND):"},{"line_number":268,"context_line":""},{"line_number":269,"context_line":"    def _deallocate_attach_handle(self, context, ah_id, hostname):"},{"line_number":270,"context_line":"        try:"},{"line_number":271,"context_line":"            attach_handle \u003d AttachHandle.get_by_id(context, ah_id)"},{"line_number":272,"context_line":"            attach_handle.deallocate(context)"}],"source_content_type":"text/x-python","patch_set":1,"id":"a4724b63_d82ddbb0","line":269,"updated":"2026-07-13 14:34:09.000000000","message":"The removal of the MDEV deallocation block from _deallocate_attach_handle left the hostname parameter unused. The method body now only calls AttachHandle.get_by_id and attach_handle.deallocate, neither of which needs hostname.\n\n**Severity**: SUGGESTION | **Confidence**: 0.9\n\n**Benefit**: Leaving an unused parameter is misleading to future maintainers who may assume hostname is still needed for deallocation logic. It also causes the caller to unnecessarily retain `hostname \u003d arq.hostname` before nulling it.\n\n**Recommendation**:\nRemove the hostname parameter from _deallocate_attach_handle and update the caller in unbind() to stop passing it. If hostname is needed again in the future it can be re-added at that time.","commit_id":"bc518f7fd1aba6c9223c3880d1c3cf2db8add710"}],"cyborg/tests/unit/accelerator/drivers/gpu/nvidia/test_driver.py":[{"author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"250204d00b16e8c7dc3d385ebdc75d2916621b5f","unresolved":false,"context_lines":[{"line_number":27,"context_line":"from cyborg.common import constants"},{"line_number":28,"context_line":"from cyborg.common import exception"},{"line_number":29,"context_line":"from cyborg.tests import base"},{"line_number":30,"context_line":"from cyborg.tests.unit.accelerator import fake_sysfs"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"CONF \u003d cyborg.conf.CONF"}],"source_content_type":"text/x-python","patch_set":11,"id":"5ad260f2_2c072276","line":30,"updated":"2026-07-30 12:13:26.000000000","message":"Both test files import \u0027from cyborg.tests.unit.accelerator import fake_sysfs\u0027 and call fake_sysfs.create_mdev_type(...) to set up temporary sysfs directories, but the module cyborg/tests/unit/accelerator/fake_sysfs.py does not exist anywhere in the repository. This causes ModuleNotFoundError at t...\n\n**Severity**: CRITICAL | **Confidence**: 1.0\n\n**Risk**: Every test in test_driver.py and test_utils.py will fail to collect with ModuleNotFoundError, meaning CI will report failures. The new test coverage for the MdevBusManager-based discovery refactor cannot be validated.\n\n**Priority**: Immediate\n**Why This Matters**: Every test in test_driver.py and test_utils.py will fail to collect with ModuleNotFoundError, meaning CI will report failures. The new test coverage for the MdevBusManager-based discovery refactor cannot be validated.\n\n**Recommendation**:\nAdd the missing module cyborg/tests/unit/accelerator/fake_sysfs.py with a create_mdev_type function that creates the expected sysfs directory structure: \u003csysfs\u003e/\u003cpci_addr\u003e/mdev_supported_types/\u003ctype_name\u003e/ with files \u0027available_instances\u0027, \u0027device_api\u0027, \u0027name\u0027, \u0027description\u0027, and a \u0027devices\u0027 subdirectory populated with the specified number of created instances. This mirrors what MdevBusManager._read_mdev_type expects to read.","commit_id":"5edbfb3d872e2b981a81c04180f5a64ccb08150a"}]}