)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"07c32156b7039be3b86a4d4a1992a3d9eaa3cb84","unresolved":true,"context_lines":[{"line_number":7,"context_line":"ruff: Enable preview rules"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We could disable flake8 again now (except for hacking and neutron"},{"line_number":10,"context_line":"rules). That\u0027s left to a separate change though."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I49683689ab9894b8a63e24340e621371cbe06075"},{"line_number":13,"context_line":"Signed-off-by: Stephen Finucane \u003cstephenfin@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"d59c4c0c_03c7f2d8","line":10,"updated":"2026-03-20 14:30:24.000000000","message":"We actually can\u0027t do this while we still rely on autopep8","commit_id":"10748fe8c4c7f379e3c442f66233e60ac8958457"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c3b76c19a6abf82a6fab9ad41e9677649ac4f875","unresolved":false,"context_lines":[{"line_number":7,"context_line":"ruff: Enable preview rules"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We could disable flake8 again now (except for hacking and neutron"},{"line_number":10,"context_line":"rules). That\u0027s left to a separate change though."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I49683689ab9894b8a63e24340e621371cbe06075"},{"line_number":13,"context_line":"Signed-off-by: Stephen Finucane \u003cstephenfin@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"0d69c43d_1a5cdf13","line":10,"in_reply_to":"09b7de58_69a958a7","updated":"2026-03-24 09:12:42.000000000","message":"Done","commit_id":"10748fe8c4c7f379e3c442f66233e60ac8958457"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"320581953a3d46b3e0e4ce3425168d2201cae2b2","unresolved":true,"context_lines":[{"line_number":7,"context_line":"ruff: Enable preview rules"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We could disable flake8 again now (except for hacking and neutron"},{"line_number":10,"context_line":"rules). That\u0027s left to a separate change though."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I49683689ab9894b8a63e24340e621371cbe06075"},{"line_number":13,"context_line":"Signed-off-by: Stephen Finucane \u003cstephenfin@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"09b7de58_69a958a7","line":10,"in_reply_to":"d59c4c0c_03c7f2d8","updated":"2026-03-23 15:56:05.000000000","message":"yep unless we disabled autopep8s default config parsing and provided a seperate one just for it but i dont think that makes sense to do.","commit_id":"10748fe8c4c7f379e3c442f66233e60ac8958457"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"320581953a3d46b3e0e4ce3425168d2201cae2b2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0427ad69_1472a153","updated":"2026-03-23 15:56:05.000000000","message":"this looks reasonable to me","commit_id":"10748fe8c4c7f379e3c442f66233e60ac8958457"}],"neutron/opts.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"f9ce77dca79e2a0783c9e80009cec7d42c6ab3db","unresolved":true,"context_lines":[{"line_number":18,"context_line":"from oslo_config import cfg"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"import neutron.agent.agent_extensions_manager"},{"line_number":21,"context_line":"import neutron.agent.securitygroups_rpc"},{"line_number":22,"context_line":"import neutron.api.wsgi"},{"line_number":23,"context_line":"import neutron.conf.agent.agent_extensions_manager"},{"line_number":24,"context_line":"import neutron.conf.agent.common"}],"source_content_type":"text/x-python","patch_set":2,"id":"20b7e227_5edd1a1d","side":"PARENT","line":21,"updated":"2026-03-23 19:43:06.000000000","message":"Obviously some of these need to stay. I can\u0027t remember why it was done this way but can cleanup later","commit_id":"ba094466a26aeafa9e33bbb4b18a93bd33fc3be6"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c3b76c19a6abf82a6fab9ad41e9677649ac4f875","unresolved":false,"context_lines":[{"line_number":18,"context_line":"from oslo_config import cfg"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"import neutron.agent.agent_extensions_manager"},{"line_number":21,"context_line":"import neutron.agent.securitygroups_rpc"},{"line_number":22,"context_line":"import neutron.api.wsgi"},{"line_number":23,"context_line":"import neutron.conf.agent.agent_extensions_manager"},{"line_number":24,"context_line":"import neutron.conf.agent.common"}],"source_content_type":"text/x-python","patch_set":2,"id":"c68380e1_ee78f76a","side":"PARENT","line":21,"in_reply_to":"20b7e227_5edd1a1d","updated":"2026-03-24 09:12:42.000000000","message":"Done","commit_id":"ba094466a26aeafa9e33bbb4b18a93bd33fc3be6"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"6c35919d50718764744462d1cfa084b0d8a7cca3","unresolved":true,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"# FIXME(stephenfin): We are importing some of these modules (with the noqa"},{"line_number":21,"context_line":"# lines) for their side effects. That\u0027s a bad idea and we should explicitly"},{"line_number":22,"context_line":"# load all options."},{"line_number":23,"context_line":"import neutron.agent.agent_extensions_manager  # noqa: F401"},{"line_number":24,"context_line":"import neutron.agent.securitygroups_rpc  # noqa: F401"},{"line_number":25,"context_line":"import neutron.api.wsgi  # noqa: F401"}],"source_content_type":"text/x-python","patch_set":3,"id":"0cee8712_32eb08e8","line":22,"updated":"2026-03-24 12:15:59.000000000","message":"oh ya that sub optimal","commit_id":"f90b2081029d54233aaaff7d2564530bd23c4309"}],"pyproject.toml":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"320581953a3d46b3e0e4ce3425168d2201cae2b2","unresolved":true,"context_lines":[{"line_number":365,"context_line":""},{"line_number":366,"context_line":"[tool.ruff.lint]"},{"line_number":367,"context_line":"select \u003d [\"E4\", \"E5\", \"E7\", \"E9\", \"F\", \"G\", \"S\", \"UP\"]"},{"line_number":368,"context_line":"preview \u003d true"},{"line_number":369,"context_line":"ignore \u003d ["},{"line_number":370,"context_line":"    \"E741\",  # Ambiguous variable name: `l`"},{"line_number":371,"context_line":"    \"S104\",  # Possible binding to all interfaces"}],"source_content_type":"text/x-toml","patch_set":1,"id":"1408b1e5_2a822632","line":368,"updated":"2026-03-23 15:56:05.000000000","message":"the initall concern withthis woudl be instablity.\nbut sicne we pin the pre-commit hook to a specific release it means teh content that will be included is stable.\n\n\ndo we know if there is any timeline to bring the E rules out fo preview?\n\nthe need for this is also deminished with the next patch in the serices so that is less concerning but just wondering in general if there is a graduation path for these E rules out of preview.","commit_id":"10748fe8c4c7f379e3c442f66233e60ac8958457"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"69c4620394fade59482f5e86e83ff51af61d8f88","unresolved":true,"context_lines":[{"line_number":365,"context_line":""},{"line_number":366,"context_line":"[tool.ruff.lint]"},{"line_number":367,"context_line":"select \u003d [\"E4\", \"E5\", \"E7\", \"E9\", \"F\", \"G\", \"S\", \"UP\"]"},{"line_number":368,"context_line":"preview \u003d true"},{"line_number":369,"context_line":"ignore \u003d ["},{"line_number":370,"context_line":"    \"E741\",  # Ambiguous variable name: `l`"},{"line_number":371,"context_line":"    \"S104\",  # Possible binding to all interfaces"}],"source_content_type":"text/x-toml","patch_set":1,"id":"d3bee772_a1b9a266","line":368,"in_reply_to":"1408b1e5_2a822632","updated":"2026-03-23 17:09:41.000000000","message":"I don\u0027t know but it\u0027s probably not high-priority given they\u0027re not an issue with ruff-format","commit_id":"10748fe8c4c7f379e3c442f66233e60ac8958457"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"9c53e4c70c9d3d2e1c4dfc6a76a17a3e5447ae7c","unresolved":false,"context_lines":[{"line_number":365,"context_line":""},{"line_number":366,"context_line":"[tool.ruff.lint]"},{"line_number":367,"context_line":"select \u003d [\"E4\", \"E5\", \"E7\", \"E9\", \"F\", \"G\", \"S\", \"UP\"]"},{"line_number":368,"context_line":"preview \u003d true"},{"line_number":369,"context_line":"ignore \u003d ["},{"line_number":370,"context_line":"    \"E741\",  # Ambiguous variable name: `l`"},{"line_number":371,"context_line":"    \"S104\",  # Possible binding to all interfaces"}],"source_content_type":"text/x-toml","patch_set":1,"id":"42865caf_167104f9","line":368,"in_reply_to":"d3bee772_a1b9a266","updated":"2026-03-23 18:02:34.000000000","message":"Acknowledged","commit_id":"10748fe8c4c7f379e3c442f66233e60ac8958457"}]}