)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"febf8c83f0d110f872a0d5a9c701635badc28336","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"70f6c77b_5175c9bb","updated":"2022-08-24 16:34:27.000000000","message":"Thank you, Lajos. Let me put -1 because I found one method which was detected in our job but is not covered by this change.\n\nUnfortunately we are facing a different issue with our CI and l2gw testing is degraded/disabled, so there might be a few additional items we\u0027ll see when the job is fixed, but at my first look the change looks good to me (except for the one I commented).","commit_id":"e0300260d6460a4dacae40718bf0cdc2f89dc733"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"5350c3f0cedb6485b924a97bd7388e8ff7c3df6b","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f9a71741_7cbaf535","in_reply_to":"70f6c77b_5175c9bb","updated":"2022-08-25 12:17:20.000000000","message":"Thanks, I check it, and push the additional things","commit_id":"e0300260d6460a4dacae40718bf0cdc2f89dc733"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"5a31b04edf50c5b721241b6ce604b8079da2ce19","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"bb1e008f_0fb8dc12","updated":"2022-09-08 09:33:08.000000000","message":"recheck\nLet\u0027s see with fresh oslo.db master","commit_id":"73048dc31afb4fd5db3b2554eb0fcf75c2aacc78"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"3b14fabf73335b16f54c5ce6cfddd196f800e399","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"0f2a33da_7191f4a1","updated":"2022-09-26 12:49:24.000000000","message":"recheck\nnew version from https://review.opendev.org/c/x/networking-l2gw-tempest-plugin/+/859278","commit_id":"87c1f948e5dc0d20125202a7b3a7378b788cf1e4"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"c6d0c3c9151dfabf02905c0c64e36808db702015","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"9b030619_f36fab2d","updated":"2022-09-27 09:15:21.000000000","message":"Previous run with tempest job passed, but I had to change which patch is depending on the other, I submit this one now, as both tempest and my local testing was green, please reach me out if you find any issues.","commit_id":"fbebe30031a3055c46fa45f85e85a37b4712a370"}],"networking_l2gw/db/l2gateway/l2gateway_db.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"febf8c83f0d110f872a0d5a9c701635badc28336","unresolved":true,"context_lines":[{"line_number":322,"context_line":"                return self._make_l2gw_connections_dict(gw_db)"},{"line_number":323,"context_line":"            return self._make_l2gw_connections_dict(conn_db[0])"},{"line_number":324,"context_line":""},{"line_number":325,"context_line":"    def get_l2_gateway_connections_count(self, context, filters\u003dNone):"},{"line_number":326,"context_line":"        return len(self._get_collection(context, models.L2GatewayConnection,"},{"line_number":327,"context_line":"                                        self._make_l2gw_connections_dict,"},{"line_number":328,"context_line":"                                        filters\u003dfilters))"}],"source_content_type":"text/x-python","patch_set":2,"id":"acf67667_c2306df7","line":325,"range":{"start_line":325,"start_character":8,"end_line":325,"end_character":40},"updated":"2022-08-24 16:34:27.000000000","message":"This is called by quota class, and needs CONTEXT_READER.\n\n```\n  File \"/usr/lib/python3.9/site-packages/neutron/db/quota/driver.py\", line 100, in get_detailed_project_quotas\n    used \u003d resource.count(context, plugin, project_id)\n  File \"/usr/lib/python3.9/site-packages/neutron/quota/resource.py\", line 160, in count\n    return self._count_func(context, self.plural_name, project_id)\n  File \"/usr/lib/python3.9/site-packages/neutron/quota/resource.py\", line 47, in _count_resource\n    return obj_count_getter(\n  File \"/usr/lib/python3.9/site-packages/networking_l2gw/db/l2gateway/l2gateway_db.py\", line 312, in get_l2_gateway_connections_count\n    return len(self._get_collection(context, models.L2GatewayConnection,\n  File \"/usr/lib/python3.9/site-packages/networking_l2gw/db/l2gateway/db_query.py\", line 92, in _get_collection\n    items \u003d [dict_func(c, fields) for c in query]\n  File \"/usr/lib64/python3.9/site-packages/sqlalchemy/orm/query.py\", line 2896, in __iter__\n    return self._iter().__iter__()\n  File \"/usr/lib64/python3.9/site-packages/sqlalchemy/orm/query.py\", line 2903, in _iter\n    result \u003d self.session.execute(\n  File \"/usr/lib64/python3.9/site-packages/sqlalchemy/orm/session.py\", line 1693, in execute\n    result \u003d fn(orm_exec_state)\n```","commit_id":"e0300260d6460a4dacae40718bf0cdc2f89dc733"}]}