)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"21515a6872223d0c816ef2bc7581a2ae726e7eeb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a167ce5b_49edb6da","updated":"2021-11-22 09:49:21.000000000","message":"The -1 is for the lack of UTs.","commit_id":"1ee8f9f2eeaee5ec0bc4c9e9574b4a1e8c1d64a7"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"2d269b3dc4f90ecbe696777b74b1d3b317d48a8e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b6695cde_63681015","updated":"2021-11-22 08:25:42.000000000","message":"The CLI code of the neutronclient was deprecated and is going to be deleted. Only OSC should implement new commands.","commit_id":"1ee8f9f2eeaee5ec0bc4c9e9574b4a1e8c1d64a7"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"a78f46335e9de40881174688c774899a0eed2a55","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0bb16e10_76d63bb5","in_reply_to":"02352d98_0e16d72c","updated":"2021-11-30 09:03:01.000000000","message":"yes, exactly, I looked for unit tests as well but we have no unit tests for client code.","commit_id":"1ee8f9f2eeaee5ec0bc4c9e9574b4a1e8c1d64a7"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"43071db60176618ecd03bda26fa45fdaa68158e1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"dabd15b0_4ec42955","in_reply_to":"97203498_98a18bfb","updated":"2021-11-22 09:44:58.000000000","message":"This is why I didn\u0027t -1 this patch. That was just a heads-up.","commit_id":"1ee8f9f2eeaee5ec0bc4c9e9574b4a1e8c1d64a7"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"a6b4b2e1e3355a2d4ac6335192a5371989bd9f3a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ea66f748_7e407550","in_reply_to":"a167ce5b_49edb6da","updated":"2021-11-23 00:54:03.000000000","message":"Seems there is no such need now 😊\nhttps://review.opendev.org/c/openstack/python-neutronclient/+/812132\nhttps://review.opendev.org/c/openstack/python-neutronclient/+/706227\nhttps://review.opendev.org/c/openstack/python-neutronclient/+/675900","commit_id":"1ee8f9f2eeaee5ec0bc4c9e9574b4a1e8c1d64a7"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"a0c9f1eb269dd1389cf08e4145256a42c45b4e35","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"97203498_98a18bfb","in_reply_to":"b6695cde_63681015","updated":"2021-11-22 09:01:17.000000000","message":"Quote from commit message:\n\"\"\"\nNeutron fullstack test cases [2] rely on the neutron client\nto interact with the neutron-server (API).\n\"\"\"","commit_id":"1ee8f9f2eeaee5ec0bc4c9e9574b4a1e8c1d64a7"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"c9eea3fc1f2860799fef0c16e653ce45353eda51","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"02352d98_0e16d72c","in_reply_to":"ea66f748_7e407550","updated":"2021-11-23 01:04:25.000000000","message":"I found out why test cases is not needed. Because this is to add \"rest API call\" construction methods only, and all test cases are for shell commands. Another merged patch with no test cases, and you had +2ed for it:\n\nhttps://review.opendev.org/c/openstack/python-neutronclient/+/727796","commit_id":"1ee8f9f2eeaee5ec0bc4c9e9574b4a1e8c1d64a7"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"a34f786ae9976b07d2bc18320233c63e7e892b9b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"94cbc1df_eb2f0403","updated":"2021-12-06 13:57:12.000000000","message":"We should move fullstack tests to the OSC/SDK finally 😊","commit_id":"a92d8db81c7f13e46aa64be471de06054379be75"}]}