)]}'
{"nova/tests/unit/api/openstack/compute/test_attach_interfaces.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"a2afe04dced2f06fc0408031a0d26645bfaf7cd6","unresolved":true,"context_lines":[{"line_number":139,"context_line":"        self.stub_out(\u0027nova.compute.api.API.get\u0027, fake_get_instance)"},{"line_number":140,"context_line":"        self.attachments \u003d self.controller_cls()"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"        show_port_patch \u003d mock.patch.object("},{"line_number":143,"context_line":"            self.attachments.network_api, \u0027show_port\u0027, fake_show_port)"},{"line_number":144,"context_line":"        show_port_patch.start()"},{"line_number":145,"context_line":"        self.addCleanup(show_port_patch.stop)"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"        list_ports_patch \u003d mock.patch.object("},{"line_number":148,"context_line":"            self.attachments.network_api, \u0027list_ports\u0027, fake_list_ports)"},{"line_number":149,"context_line":"        list_ports_patch.start()"},{"line_number":150,"context_line":"        self.addCleanup(list_ports_patch.stop)"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"        self.req \u003d fakes.HTTPRequest.blank(\u0027\u0027)"},{"line_number":153,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"992c30d0_2e94101b","line":150,"range":{"start_line":142,"start_character":8,"end_line":150,"end_character":46},"updated":"2025-12-12 17:15:26.000000000","message":"i know your following the existing bad patther bug if your refactorign this why not do it perperly by either using\n\nself.useFixtures(fixtures.MockPatchObject(...))\n\nor use self.stub_out()\n\nwe shoudl not be manually starting hte patcher and registering the stop","commit_id":"000f3673d52c20b6cce306af954aa7f239982828"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"b1b960c63957e2628f437df5d218b3186e7444db","unresolved":false,"context_lines":[{"line_number":139,"context_line":"        self.stub_out(\u0027nova.compute.api.API.get\u0027, fake_get_instance)"},{"line_number":140,"context_line":"        self.attachments \u003d self.controller_cls()"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"        show_port_patch \u003d mock.patch.object("},{"line_number":143,"context_line":"            self.attachments.network_api, \u0027show_port\u0027, fake_show_port)"},{"line_number":144,"context_line":"        show_port_patch.start()"},{"line_number":145,"context_line":"        self.addCleanup(show_port_patch.stop)"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"        list_ports_patch \u003d mock.patch.object("},{"line_number":148,"context_line":"            self.attachments.network_api, \u0027list_ports\u0027, fake_list_ports)"},{"line_number":149,"context_line":"        list_ports_patch.start()"},{"line_number":150,"context_line":"        self.addCleanup(list_ports_patch.stop)"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"        self.req \u003d fakes.HTTPRequest.blank(\u0027\u0027)"},{"line_number":153,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9d05df52_8c3e6af7","line":150,"range":{"start_line":142,"start_character":8,"end_line":150,"end_character":46},"in_reply_to":"992c30d0_2e94101b","updated":"2026-01-05 10:17:33.000000000","message":"Yeah, I was trying to be consistent with the rest of the file. Updated this and the other instance of this","commit_id":"000f3673d52c20b6cce306af954aa7f239982828"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"e3e004a1779fd3e51251832c00840138483bc7c6","unresolved":false,"context_lines":[{"line_number":139,"context_line":"        self.stub_out(\u0027nova.compute.api.API.get\u0027, fake_get_instance)"},{"line_number":140,"context_line":"        self.attachments \u003d self.controller_cls()"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"        show_port_patch \u003d mock.patch.object("},{"line_number":143,"context_line":"            self.attachments.network_api, \u0027show_port\u0027, fake_show_port)"},{"line_number":144,"context_line":"        show_port_patch.start()"},{"line_number":145,"context_line":"        self.addCleanup(show_port_patch.stop)"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"        list_ports_patch \u003d mock.patch.object("},{"line_number":148,"context_line":"            self.attachments.network_api, \u0027list_ports\u0027, fake_list_ports)"},{"line_number":149,"context_line":"        list_ports_patch.start()"},{"line_number":150,"context_line":"        self.addCleanup(list_ports_patch.stop)"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"        self.req \u003d fakes.HTTPRequest.blank(\u0027\u0027)"},{"line_number":153,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"67ffaba7_36a477bf","line":150,"range":{"start_line":142,"start_character":8,"end_line":150,"end_character":46},"in_reply_to":"9d05df52_8c3e6af7","updated":"2026-01-08 16:26:34.000000000","message":"thanks looks good to me","commit_id":"000f3673d52c20b6cce306af954aa7f239982828"}]}