)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"1763c2fbc3f9d97518c1fdb5abc3552cec89bae3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"5ef65137_7a0428cc","updated":"2025-07-31 05:27:08.000000000","message":"@dtantsur@protonmail.com I changed the implementation somewhat. mind re-reviewing?","commit_id":"651e57a5ac3d229023dcd870a5ddd01068d5760f"}],"ironic/tests/unit/drivers/modules/redfish/test_inspect.py":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"c9f267bc38f2950d59da40c8c51010b643aae05a","unresolved":true,"context_lines":[{"line_number":112,"context_line":""},{"line_number":113,"context_line":"        # make the summary follow the data above by making it"},{"line_number":114,"context_line":"        # a property like sushy and returning the same data"},{"line_number":115,"context_line":"        type(system_mock.ethernet_interfaces).summary \u003d mock.PropertyMock("},{"line_number":116,"context_line":"            side_effect\u003dlambda: {"},{"line_number":117,"context_line":"                obj.mac_address: obj.status.state"},{"line_number":118,"context_line":"                for obj in"}],"source_content_type":"text/x-python","patch_set":4,"id":"3699bd6f_16a72e1c","line":115,"updated":"2025-08-05 13:19:59.000000000","message":"Per https://docs.python.org/3/library/unittest.mock.html#patching-descriptors-and-proxy-objects, you should be able to use `mock.patch` or anything that actually cleans up after the test.","commit_id":"382a9f5c8096a45b135f66e4402a831d9a337083"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"46bab653521beb1d55ed24398f8a70f949857d48","unresolved":true,"context_lines":[{"line_number":112,"context_line":""},{"line_number":113,"context_line":"        # make the summary follow the data above by making it"},{"line_number":114,"context_line":"        # a property like sushy and returning the same data"},{"line_number":115,"context_line":"        type(system_mock.ethernet_interfaces).summary \u003d mock.PropertyMock("},{"line_number":116,"context_line":"            side_effect\u003dlambda: {"},{"line_number":117,"context_line":"                obj.mac_address: obj.status.state"},{"line_number":118,"context_line":"                for obj in"}],"source_content_type":"text/x-python","patch_set":4,"id":"39de9431_21a1b722","line":115,"in_reply_to":"3699bd6f_16a72e1c","updated":"2025-08-05 14:35:16.000000000","message":"Agreed. I\u0027m touching the rest of these mocks in follow ons and I\u0027ll fix it up there as we discussed on IRC.","commit_id":"382a9f5c8096a45b135f66e4402a831d9a337083"}]}
