)]}'
{"neutron/tests/functional/agent/ovn/metadata/test_metadata_agent.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"11266dd1276ccd3a7f8f6b4085f35f3095e8d145","unresolved":true,"context_lines":[{"line_number":273,"context_line":"        n_utils.wait_until_true(check_for_metadata, timeout\u003d5, exception\u003dexc)"},{"line_number":274,"context_line":""},{"line_number":275,"context_line":"        # Delete Chassis and assert"},{"line_number":276,"context_line":"        chassis \u003d self.sb_api.lookup(\u0027Chassis\u0027, self.chassis_name)"},{"line_number":277,"context_line":"        self.del_fake_chassis(chassis.name)"},{"line_number":278,"context_line":"        self.assertRaises(idlutils.RowNotFound, self.sb_api.lookup,"},{"line_number":279,"context_line":"                          \u0027Chassis\u0027, self.chassis_name)"}],"source_content_type":"text/x-python","patch_set":4,"id":"5b18c43a_5dbc1930","line":276,"range":{"start_line":276,"start_character":0,"end_line":276,"end_character":3},"updated":"2021-09-22 14:09:58.000000000","message":"Why do we use \"Chassis\" here? (and in L279)","commit_id":"0a92e9de6ceb92ca9257ae8b847b3f35f805a16c"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"9f13c04034068b6958033a239fcba28f84a2e337","unresolved":false,"context_lines":[{"line_number":273,"context_line":"        n_utils.wait_until_true(check_for_metadata, timeout\u003d5, exception\u003dexc)"},{"line_number":274,"context_line":""},{"line_number":275,"context_line":"        # Delete Chassis and assert"},{"line_number":276,"context_line":"        chassis \u003d self.sb_api.lookup(\u0027Chassis\u0027, self.chassis_name)"},{"line_number":277,"context_line":"        self.del_fake_chassis(chassis.name)"},{"line_number":278,"context_line":"        self.assertRaises(idlutils.RowNotFound, self.sb_api.lookup,"},{"line_number":279,"context_line":"                          \u0027Chassis\u0027, self.chassis_name)"}],"source_content_type":"text/x-python","patch_set":4,"id":"10cfc725_4e942d62","line":276,"range":{"start_line":276,"start_character":0,"end_line":276,"end_character":3},"in_reply_to":"5b18c43a_5dbc1930","updated":"2021-09-22 14:12:49.000000000","message":"I did it because we are passing the result to del_fake_chassis/add_fake_chassis, which create actual Chassis objects (and Chassis_Private iff it is supported).","commit_id":"0a92e9de6ceb92ca9257ae8b847b3f35f805a16c"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"f063e64471770d23a4a35b1a21f30214f7624bed","unresolved":true,"context_lines":[{"line_number":273,"context_line":"        n_utils.wait_until_true(check_for_metadata, timeout\u003d5, exception\u003dexc)"},{"line_number":274,"context_line":""},{"line_number":275,"context_line":"        # Delete Chassis and assert"},{"line_number":276,"context_line":"        chassis \u003d self.sb_api.lookup(\u0027Chassis\u0027, self.chassis_name)"},{"line_number":277,"context_line":"        self.del_fake_chassis(chassis.name)"},{"line_number":278,"context_line":"        self.assertRaises(idlutils.RowNotFound, self.sb_api.lookup,"},{"line_number":279,"context_line":"                          \u0027Chassis\u0027, self.chassis_name)"}],"source_content_type":"text/x-python","patch_set":4,"id":"50c92324_3b90e892","line":276,"range":{"start_line":276,"start_character":0,"end_line":276,"end_character":3},"in_reply_to":"5b18c43a_5dbc1930","updated":"2021-09-22 14:13:23.000000000","message":"Ok, in both cases (with and without chassis_private), that will work. chassis creation and deletion method will affect chassis_private too","commit_id":"0a92e9de6ceb92ca9257ae8b847b3f35f805a16c"}]}
