)]}'
{"neutron/plugins/ml2/plugin.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"4a527fbdfd4918048572c00c47b366e5199b6da4","unresolved":false,"context_lines":[{"line_number":103,"context_line":"from neutron.db.models import securitygroup as sg_models"},{"line_number":104,"context_line":"from neutron.db import models_v2"},{"line_number":105,"context_line":"from neutron.db import provisioning_blocks"},{"line_number":106,"context_line":"from neutron.db.quota import driver  # noqa"},{"line_number":107,"context_line":"from neutron.db import securitygroups_rpc_base as sg_db_rpc"},{"line_number":108,"context_line":"from neutron.db import segments_db"},{"line_number":109,"context_line":"from neutron.db import subnet_service_type_mixin"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_ee3beccd","side":"PARENT","line":106,"updated":"2019-10-11 13:43:18.000000000","message":"The one reason I\u0027m hesitant on removing this is that there has been a number of changes over the past few days removing these \"noqa\" imports and all have been wrong - the import is typically needed to load something important.  Maybe this one is different but it is not passing zuul yet either.","commit_id":"2ffaa40b43ceb19f1584afd3d8a9c18efeca7f6a"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"3f3fb4d36b1c6f2a0ff099bbbc3a25f057565976","unresolved":false,"context_lines":[{"line_number":103,"context_line":"from neutron.db.models import securitygroup as sg_models"},{"line_number":104,"context_line":"from neutron.db import models_v2"},{"line_number":105,"context_line":"from neutron.db import provisioning_blocks"},{"line_number":106,"context_line":"from neutron.db.quota import driver  # noqa"},{"line_number":107,"context_line":"from neutron.db import securitygroups_rpc_base as sg_db_rpc"},{"line_number":108,"context_line":"from neutron.db import segments_db"},{"line_number":109,"context_line":"from neutron.db import subnet_service_type_mixin"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_4410df00","side":"PARENT","line":106,"in_reply_to":"3fa7e38b_ee3beccd","updated":"2019-10-11 15:35:06.000000000","message":"I have the same feeling.\n\nApart from that, IIRC this was required to load the DB quota driver propoerly [1], but something might be changed since then. I don\u0027t see any message defined at [2] in the q-svc log of neutron-tempest-plugin-api job.\n\nWe need to check it carefully before merging it.\n\nI haven\u0027t tested it locally yet. I just stop my devstack instance :-(\n\n[1] https://github.com/openstack/neutron/blob/master/neutron/quota/__init__.py#L172\n[2] https://github.com/openstack/neutron/blob/master/neutron/quota/__init__.py#L175-L176","commit_id":"2ffaa40b43ceb19f1584afd3d8a9c18efeca7f6a"}]}