)]}'
{".pre-commit-config.yaml":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"92e9112d905d475355d450aff69cb8ab41ad832d","unresolved":false,"context_lines":[{"line_number":64,"context_line":"      - id: doc8"},{"line_number":65,"context_line":"        exclude: |"},{"line_number":66,"context_line":"          (?x)("},{"line_number":67,"context_line":"            doc/source/admin/config-qos-min-pps.rst"},{"line_number":68,"context_line":"            | doc/source/admin/deploy-provider-verifynetworkoperation.txt"},{"line_number":69,"context_line":"            | doc/source/admin/deploy-selfservice-verifynetworkoperation.txt"},{"line_number":70,"context_line":"            | doc/source/admin/shared/deploy-ha-vrrp-initialnetworks.txt"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"ed3c3f8c_f950eaef","line":67,"updated":"2025-07-01 10:56:20.000000000","message":"Why all these documents are excluded?\n\nUpdate: nevermind, this is what is now defined in the tox file.","commit_id":"be77ed3a813709eebb2659b5473b890b5978c743"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"8c02851a581f75741adab7b4ccca74930fd69b1f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"44ec6472_8adc4e95","updated":"2025-05-26 10:58:58.000000000","message":"recheck another unrelated failure","commit_id":"ec12c30606a6d62607023d9d0e54718cbbbf4139"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"12b8387ea31522a32752ba1ec57cc4c415d77cfc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f67f1154_df06747e","updated":"2025-05-23 16:09:22.000000000","message":"recheck timeout","commit_id":"ec12c30606a6d62607023d9d0e54718cbbbf4139"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"76f9900dfe8f392e4607b2f7170e21cfec8dcaf3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"36387a52_2ac2cb27","updated":"2025-05-26 15:08:34.000000000","message":"recheck timeout","commit_id":"2b392b1243b0195da2e1683ad36ed1056743318b"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"e269815e4c9b2e19147e1b10bbe3ab6f0a5e7d1e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"4888df9b_e425a68d","updated":"2025-07-05 17:39:06.000000000","message":"recheck","commit_id":"be77ed3a813709eebb2659b5473b890b5978c743"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"b9cc0ba3658d256e44d9015f2a3c182ac1838c59","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"3f60a492_bf58804d","updated":"2025-07-08 18:21:48.000000000","message":"recheck cover job failure","commit_id":"be77ed3a813709eebb2659b5473b890b5978c743"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"dc9f5a17d22ab1169fccd0795b254d8225b07e4d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"e71f3be3_f6c27f88","updated":"2025-07-08 21:38:22.000000000","message":"recheck functional failure","commit_id":"be77ed3a813709eebb2659b5473b890b5978c743"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"c8ce93420fa2a98218d3d9021863b031fefe6ee0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"3b0a12b7_36f0f02d","updated":"2025-07-07 19:59:37.000000000","message":"recheck neutron-functional","commit_id":"be77ed3a813709eebb2659b5473b890b5978c743"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"1cb2ed6b4723797fcd60a0208008137add6029ab","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"ed061cc6_5c83c1bf","updated":"2025-07-04 08:49:57.000000000","message":"recheck neutron-tempest-plugin-ovn","commit_id":"be77ed3a813709eebb2659b5473b890b5978c743"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"071c476125faca9ec49dffed3af2257f7b73277c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"f92b5fe4_8072045f","updated":"2025-07-08 15:26:50.000000000","message":"recheck test_fip_connection_for_address_scope","commit_id":"be77ed3a813709eebb2659b5473b890b5978c743"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"f2ae2f4beed675ced1787ab97ecfa4f69d7ed158","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"3f38d246_91e83341","updated":"2025-07-07 08:39:26.000000000","message":"recheck timeout","commit_id":"be77ed3a813709eebb2659b5473b890b5978c743"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"c158e5bbc0ec87cb374ba30e332d72f59e38046c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"7e5c0209_291270cc","updated":"2025-07-07 21:47:50.000000000","message":"recheck unrelated py310 job failure","commit_id":"be77ed3a813709eebb2659b5473b890b5978c743"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"34ee62363bbf8810178f5b0335ddbc03d0bd5d01","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"e0922ca8_ce8a5f78","updated":"2025-07-02 07:29:09.000000000","message":"thanks","commit_id":"be77ed3a813709eebb2659b5473b890b5978c743"}],"pyproject.toml":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"cb51b1a42fdcd2947bb17132bc68db720db60a21","unresolved":true,"context_lines":[{"line_number":36,"context_line":"    \"UP024\",  # Replace aliased errors with `OSError`"},{"line_number":37,"context_line":"    \"UP030\",  # Use implicit references for positional format fields"},{"line_number":38,"context_line":"    \"UP038\",  # Use `X | Y` in `isinstance` call instead of `(X, Y)`"},{"line_number":39,"context_line":"    \"UP039\",  # Unnecessary parentheses after class definition"},{"line_number":40,"context_line":"]"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"[tool.ruff.lint.per-file-ignores]"}],"source_content_type":"text/x-toml","patch_set":1,"id":"7bce7f84_055fbb4d","line":39,"updated":"2025-05-21 18:03:19.000000000","message":"I thought we fixed all of these at some point, but we can just do follow-ons to work through them.","commit_id":"77a98cf30fd60bc247c234594c213117bbba7121"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"bec808d077c8fc7c4bf4ab410f656ba815e3af9a","unresolved":false,"context_lines":[{"line_number":36,"context_line":"    \"UP024\",  # Replace aliased errors with `OSError`"},{"line_number":37,"context_line":"    \"UP030\",  # Use implicit references for positional format fields"},{"line_number":38,"context_line":"    \"UP038\",  # Use `X | Y` in `isinstance` call instead of `(X, Y)`"},{"line_number":39,"context_line":"    \"UP039\",  # Unnecessary parentheses after class definition"},{"line_number":40,"context_line":"]"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"[tool.ruff.lint.per-file-ignores]"}],"source_content_type":"text/x-toml","patch_set":1,"id":"c4382474_fce57a63","line":39,"in_reply_to":"7bce7f84_055fbb4d","updated":"2025-07-01 16:24:02.000000000","message":"Yes, I have a patch at the end of the series to do this.","commit_id":"77a98cf30fd60bc247c234594c213117bbba7121"}],"tox.ini":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"a85fd8f5e86306112521bc538eb153b8626dabcb","unresolved":true,"context_lines":[{"line_number":196,"context_line":""},{"line_number":197,"context_line":"[flake8]"},{"line_number":198,"context_line":"# We only enable the hacking (H) and neutron (N) checks"},{"line_number":199,"context_line":"select \u003d H,N"},{"line_number":200,"context_line":"ignore \u003d H405,H701,H702,H703,N530"},{"line_number":201,"context_line":"show-source \u003d true"},{"line_number":202,"context_line":"exclude \u003d ./.*,build,dist,doc"}],"source_content_type":"text/x-properties","patch_set":4,"id":"726a651f_9fa97f9a","line":199,"updated":"2026-03-20 01:07:32.000000000","message":"so this is a bug.\n\nauto pep8 read this config so this disabled almost all the formating\n\nthis is correct if adding ruff format but this shoudl have been in the abandoned patch https://review.opendev.org/c/openstack/neutron/+/950640/2\n\nthat is the correct tiem to add the selectr to run only the hackign rules.","commit_id":"be77ed3a813709eebb2659b5473b890b5978c743"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"85b01ffa3b29191423a542602b70a47ffed6f382","unresolved":false,"context_lines":[{"line_number":196,"context_line":""},{"line_number":197,"context_line":"[flake8]"},{"line_number":198,"context_line":"# We only enable the hacking (H) and neutron (N) checks"},{"line_number":199,"context_line":"select \u003d H,N"},{"line_number":200,"context_line":"ignore \u003d H405,H701,H702,H703,N530"},{"line_number":201,"context_line":"show-source \u003d true"},{"line_number":202,"context_line":"exclude \u003d ./.*,build,dist,doc"}],"source_content_type":"text/x-properties","patch_set":4,"id":"b01f13ff_05d5cb8e","line":199,"in_reply_to":"726a651f_9fa97f9a","updated":"2026-03-20 01:56:59.000000000","message":"Sean helped me debug this today, it was elusive why some of the formatting was not always generating a warning - thanks for finding it.\n\nI\u0027ll push a patch maybe next week to remove the select\u003d line and let autopep8 do it\u0027s magic - it\u0027s about 300 files to fix but mostly seem minor and don\u0027t affect running code.","commit_id":"be77ed3a813709eebb2659b5473b890b5978c743"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"36ae38949c9e11b5baa74a0965561689cc16202d","unresolved":false,"context_lines":[{"line_number":196,"context_line":""},{"line_number":197,"context_line":"[flake8]"},{"line_number":198,"context_line":"# We only enable the hacking (H) and neutron (N) checks"},{"line_number":199,"context_line":"select \u003d H,N"},{"line_number":200,"context_line":"ignore \u003d H405,H701,H702,H703,N530"},{"line_number":201,"context_line":"show-source \u003d true"},{"line_number":202,"context_line":"exclude \u003d ./.*,build,dist,doc"}],"source_content_type":"text/x-properties","patch_set":4,"id":"1a3f05e1_f19aacb8","line":199,"in_reply_to":"a33d1255_9950d328","updated":"2026-03-20 10:02:06.000000000","message":"I also wonder if it\u0027s worth considering just switching over to ruff now, like keystone etc. have? 🤔 Potential PTG topic?","commit_id":"be77ed3a813709eebb2659b5473b890b5978c743"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"ec0f59cb6dbe519b800958d0013344a7c159564f","unresolved":false,"context_lines":[{"line_number":196,"context_line":""},{"line_number":197,"context_line":"[flake8]"},{"line_number":198,"context_line":"# We only enable the hacking (H) and neutron (N) checks"},{"line_number":199,"context_line":"select \u003d H,N"},{"line_number":200,"context_line":"ignore \u003d H405,H701,H702,H703,N530"},{"line_number":201,"context_line":"show-source \u003d true"},{"line_number":202,"context_line":"exclude \u003d ./.*,build,dist,doc"}],"source_content_type":"text/x-properties","patch_set":4,"id":"a33d1255_9950d328","line":199,"in_reply_to":"b01f13ff_05d5cb8e","updated":"2026-03-20 09:56:13.000000000","message":"Apologies, I wasn\u0027t aware that autopep8 read this config. TIL. However:\n\n\u003e this is correct if adding ruff format but this shoudl have been in the abandoned patch https://review.opendev.org/c/openstack/neutron/+/950640/2\n\nWe did enable `ruff check` here. `ruff check` should handle all the existing non-hacking rules (E5, E7, E9, W, ...). If not, we haven\u0027t enable as many as we should have.","commit_id":"be77ed3a813709eebb2659b5473b890b5978c743"}]}
