)]}'
{"nova/api/validation/extra_specs/validators.py":[{"robot_id":"zuul","robot_run_id":"9db468c3993547bb8a913f536c940009","url":"https://zuul.teim.app/t/main/buildset/9db468c3993547bb8a913f536c940009","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:manual-ci","change_message_id":"6c6804fb50ca66cdd4eeb66a7f7b0fa9c22b077d","patch_set":2,"id":"1d1d6460_9d112faf","line":67,"updated":"2026-01-21 13:23:16.000000000","message":"Consider adding type hints to the newly modified functions for better code documentation\n\n**Severity**: SUGGESTION | **Confidence**: 0.7\n\n**Benefit**: Improved code readability and better IDE support with static type checking\n\n**Recommendation**:\nAdd return type annotations to functions that were modified by removing global statements","commit_id":"2b7b27132451fd4a491a812de3669b3771595c90"}],"nova/tests/unit/compute/test_compute.py":[{"robot_id":"zuul","robot_run_id":"9db468c3993547bb8a913f536c940009","url":"https://zuul.teim.app/t/main/buildset/9db468c3993547bb8a913f536c940009","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:manual-ci","change_message_id":"6c6804fb50ca66cdd4eeb66a7f7b0fa9c22b077d","patch_set":2,"id":"225371f7_d87d976e","line":420,"updated":"2026-01-21 13:23:16.000000000","message":"Mock usage in tests lacks consistent autospec\u003dTrue parameter\n\n**Severity**: WARNING | **Confidence**: 0.7\n\n**Impact**: Mock objects without autospec may not properly enforce interface contracts\n\n**Suggestion**:\nAdd autospec\u003dTrue to all @mock.patch decorators to ensure proper mock behavior","commit_id":"2b7b27132451fd4a491a812de3669b3771595c90"}],"nova/tests/unit/fake_network.py":[{"robot_id":"zuul","robot_run_id":"9db468c3993547bb8a913f536c940009","url":"https://zuul.teim.app/t/main/buildset/9db468c3993547bb8a913f536c940009","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:manual-ci","change_message_id":"6c6804fb50ca66cdd4eeb66a7f7b0fa9c22b077d","patch_set":2,"id":"ed0cf942_1c406780","line":154,"updated":"2026-01-21 13:23:16.000000000","message":"Removed \u0027global\u0027 statements without proper validation in functions that modify global variables\n\n**Severity**: WARNING | **Confidence**: 0.8\n\n**Impact**: Could lead to undefined behavior if _real_functions is not properly initialized\n\n**Suggestion**:\nAdd explicit check for _real_functions existence before accessing it","commit_id":"2b7b27132451fd4a491a812de3669b3771595c90"}],"nova/utils.py":[{"robot_id":"zuul","robot_run_id":"9db468c3993547bb8a913f536c940009","url":"https://zuul.teim.app/t/main/buildset/9db468c3993547bb8a913f536c940009","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:manual-ci","change_message_id":"6c6804fb50ca66cdd4eeb66a7f7b0fa9c22b077d","patch_set":2,"id":"f4e60892_91ebdf95","line":92,"updated":"2026-01-21 13:23:16.000000000","message":"Global executor access patterns in utils.py could benefit from better encapsulation\n\n**Severity**: WARNING | **Confidence**: 0.6\n\n**Impact**: Direct global access patterns make testing and debugging more difficult\n\n**Suggestion**:\nConsider using dependency injection or factory patterns for executor management","commit_id":"2b7b27132451fd4a491a812de3669b3771595c90"}]}