)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30380,"name":"ZhouHeng","email":"zhouhenglc@inspur.com","username":"zhouhenglc"},"change_message_id":"7546cb64d179f4a296891add2ca70d0f9262d4ab","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a9b32974_bbbd7336","updated":"2022-03-21 00:37:36.000000000","message":"code looks good, please add the reason for replace in commit message.","commit_id":"04f3462662e765129ff529d8484576b01444ee07"},{"author":{"_account_id":30380,"name":"ZhouHeng","email":"zhouhenglc@inspur.com","username":"zhouhenglc"},"change_message_id":"99b49f37f4a3e56190f8c24e54aa841d1b1b582d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"be44925d_b0f30c1e","updated":"2022-03-21 01:54:57.000000000","message":"Title is to long, more good practice[1]\n[1] https://wiki.openstack.org/wiki/GitCommitMessages\n\nyou can update it like:\n\nreplace tearDown with addCleanup\n\nAll setUp and tearDown methods must upcall using the super() \nmethod.tearDown methods should be avoided and addCleanup calls \nshould be preferred[1].\n\n[1] https://github.com/openstack/neutron/blob/master/HACKING.rst\n\n","commit_id":"68b7d8c923ca2bb78dead5c92046fcf10bd63fc3"},{"author":{"_account_id":30380,"name":"ZhouHeng","email":"zhouhenglc@inspur.com","username":"zhouhenglc"},"change_message_id":"3ec2dc10e7c36de70dd8cf631263e495dfb322cd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"9c5d32b5_5a7a7dfd","updated":"2022-03-21 02:57:53.000000000","message":"please rebase with https://review.opendev.org/c/openstack/neutron-fwaas/+/831478","commit_id":"4910b6210fd99597d7a7c5fd3b10fe43f6d56a7b"},{"author":{"_account_id":30380,"name":"ZhouHeng","email":"zhouhenglc@inspur.com","username":"zhouhenglc"},"change_message_id":"478e6fc7b5a7c173911a6d641df8c022190c51b1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"883b4058_4ef62f7e","updated":"2022-03-21 02:48:07.000000000","message":"thansk Liangyi, neutron-fwaas project has many test cases use mock.patch(...).start() and not Cleanup(like [1]), this may affect other tests, can you submit a patch and fix it?\n\n[1] https://github.com/openstack/neutron-fwaas/blob/master/neutron_fwaas/tests/unit/services/firewall/service_drivers/agents/drivers/linux/test_iptables_fwaas_v2.py#L43","commit_id":"4910b6210fd99597d7a7c5fd3b10fe43f6d56a7b"}]}