)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"54978f60d4ea03a73a46218979912987c925b2f8","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3c865485_770389dd","updated":"2023-02-22 12:25:51.000000000","message":"Hi, thanks for the patch. Could you open a new bug for this issue and\nadd some details there on how can I hit that issue and exactly what\u0027s the issue?\nSo I can verify your patch fix that issue.","commit_id":"1db1e4d19c74e1ef455693b8bdbefda3bbbad1fe"},{"author":{"_account_id":27838,"name":"Vadym Markov","email":"vmarkov@mirantis.com","username":"vmarkov"},"change_message_id":"304a9f5bd7993485956e594491adf2ff9418b6ec","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6e52efdc_47a45618","in_reply_to":"3c865485_770389dd","updated":"2023-02-22 16:44:36.000000000","message":"Could you point me to some example of bug related to Neutron backends? This issue was hit on prod env using Tungstenfabric and impossible on usual OVN env. So, reproduce relies on non-trivial deployment. Not sure, if OVS backend is affected by issue\n\nI can file ticket with reproduce assuming that TF env is available for reviewer, but it looks quite formal and useless.","commit_id":"1db1e4d19c74e1ef455693b8bdbefda3bbbad1fe"},{"author":{"_account_id":27838,"name":"Vadym Markov","email":"vmarkov@mirantis.com","username":"vmarkov"},"change_message_id":"0de188481ea56c8ad3dcba7feb36c804dd2a9673","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e48e26ff_1495e5a3","in_reply_to":"454ad836_419db910","updated":"2023-02-22 22:22:10.000000000","message":"Linked a ticket","commit_id":"1db1e4d19c74e1ef455693b8bdbefda3bbbad1fe"},{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"f2ea240d021dc78dca8ba2892303b3578b9efbe7","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"454ad836_419db910","in_reply_to":"6e52efdc_47a45618","updated":"2023-02-22 17:07:19.000000000","message":"I think you can find some neutron backend bugs here https://bugs.launchpad.net/horizon/+bugs?field.tag\u003dneutron\n\nThe only intention of my comment is that your patch not changing the default behavior let\u0027s say for OVN env. If you are facing this issue in your specific env. you can attach some screenshots in the bug summary so that we are more confident while reviewing. IMO adding more information in the commit-msg or in the bug is not useless. it is easy for reviewers to review the code and easy for someone who refers to this code in the future why this code is changed.","commit_id":"1db1e4d19c74e1ef455693b8bdbefda3bbbad1fe"},{"author":{"_account_id":37598,"name":"Ivan Anfimov","display_name":"Ivan Anfimov","email":"lazekteam@gmail.com","username":"anfimovir"},"change_message_id":"100d39cff60328593fecdb65b18a82168b9968d9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a43da499_eca6e5a0","updated":"2025-05-22 14:26:35.000000000","message":"@jjasek@redhat.com hello, please close this MR.","commit_id":"13c828e13daf7b5fa27a35c460b089077bbbb780"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"7853552fe098f38a22ddd00cd30ab181586253da","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0d16e0ab_7343d729","updated":"2023-03-09 13:07:49.000000000","message":"I would like to know how the error happesn to understand why enable_fip_topology_check \u003d\u003d False helps you.\nCould you check my comment in the launchpad bug [1]?\n\nWhen we implemented the original code, we checked the possibility of IndexError and our consensus was that IndexError would not happen unless \"ports\" (or \"router_ports\") has an empty \"fixed_ips\" (i.e., a port without IP).\n\nFrom the above reason, I vote -1 for the proposed approach at the moment.\n\n[1] https://bugs.launchpad.net/horizon/+bug/2008132/comments/1","commit_id":"13c828e13daf7b5fa27a35c460b089077bbbb780"},{"author":{"_account_id":27838,"name":"Vadym Markov","email":"vmarkov@mirantis.com","username":"vmarkov"},"change_message_id":"4a94e66777ba2d65765586d19fba7067b059a85c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"739080d0_b6feb86d","in_reply_to":"0d16e0ab_7343d729","updated":"2023-06-16 18:46:21.000000000","message":"Updated reproduce in the ticket","commit_id":"13c828e13daf7b5fa27a35c460b089077bbbb780"}],"openstack_dashboard/api/neutron.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"7853552fe098f38a22ddd00cd30ab181586253da","unresolved":true,"context_lines":[{"line_number":710,"context_line":"                                    for p in router_ports"},{"line_number":711,"context_line":"                                    if p.device_id in gw_routers)"},{"line_number":712,"context_line":"        except IndexError as e:"},{"line_number":713,"context_line":"            logging.error("},{"line_number":714,"context_line":"                \u0027Invalid port configuration. As a possible measure, ensure \u0027"},{"line_number":715,"context_line":"                \u0027that your networking backend actually requires the \u0027"},{"line_number":716,"context_line":"                \u0027fip topology check.\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"a3fe3b45_1ebe6c29","line":713,"range":{"start_line":713,"start_character":12,"end_line":713,"end_character":25},"updated":"2023-03-09 13:07:49.000000000","message":"Could you use LOG.error instead? It will include more information on the error message like which module emits the message and it is the way we usually use.","commit_id":"13c828e13daf7b5fa27a35c460b089077bbbb780"}]}