)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"ba835271db3eac7585666cef6f83d513bd523d8f","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Horizon is moving to using the openstacksdk"},{"line_number":10,"context_line":"instead of neutronclient; the exceptions"},{"line_number":11,"context_line":"raised are now different, and so handle them"},{"line_number":12,"context_line":"appropriately"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Assisted-By: Claude Code"},{"line_number":15,"context_line":"Depends-On: If159d4d7cd2d43898801f4ec7165679350d053fc"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"d9edd2c2_5f5c2d6f","line":12,"range":{"start_line":11,"start_character":25,"end_line":12,"end_character":13},"updated":"2026-04-27 09:09:41.000000000","message":"+1","commit_id":"2fed2b2fe72f2207cf24039f00c121bfa286bffc"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"c66524a7c71954ce4a7ee6840c8262761a4a5771","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"16b11496_3f5aed73","updated":"2025-10-22 08:21:28.000000000","message":"Thanks for taking care, I have just one nit for OpenStackCloudException","commit_id":"20464a44e16487d5b058ccb050fb2f37485f8911"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"11139a76a152080ff9296cb4149be21dc9d9bba7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1954a2ae_3a5dce89","updated":"2025-11-11 13:31:46.000000000","message":"Thanks for working on this change, Goutham. Could you please rebase and address the comment from Lajos","commit_id":"20464a44e16487d5b058ccb050fb2f37485f8911"},{"author":{"_account_id":35133,"name":"Jan Jasek","email":"jjasek@redhat.com","username":"janjasek"},"change_message_id":"680131ad6efeaaef96bc2ee15ad4806e46d1b43c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"08a0a4c0_313e09f3","updated":"2026-05-04 08:35:28.000000000","message":"960678: SDK: Remove remaining neutronclient calls | https://review.opendev.org/c/openstack/horizon/+/960678\nWas merged on horizon side, there should not be now any blocker for this manila-ui patch.","commit_id":"a77ec044764a7d362c9fc189fcc3b57ae7fe1e6d"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"960a0ce1c7e1c923ab74d0f7ffd5e701a90b1334","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"bfc0cb82_98b17f08","updated":"2026-04-29 20:16:50.000000000","message":"LGTM, thank you!","commit_id":"a77ec044764a7d362c9fc189fcc3b57ae7fe1e6d"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"61a75f6d0fec0516a7dd8e5a503870d0247c2d76","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"47f07f5f_cc3a3719","updated":"2026-04-29 06:18:01.000000000","message":"Thanks for the review; addressed the comments.","commit_id":"a77ec044764a7d362c9fc189fcc3b57ae7fe1e6d"},{"author":{"_account_id":37598,"name":"Ivan Anfimov","display_name":"Ivan Anfimov","email":"lazekteam@gmail.com","username":"anfimovir"},"change_message_id":"56610d17f3709f6c29a795bf1818a58c003c9e73","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"59bd7f15_a6916ddd","updated":"2026-05-04 08:36:04.000000000","message":"recheck (depends now merged)","commit_id":"a77ec044764a7d362c9fc189fcc3b57ae7fe1e6d"}],"manila_ui/dashboards/project/share_networks/forms.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"c66524a7c71954ce4a7ee6840c8262761a4a5771","unresolved":true,"context_lines":[{"line_number":74,"context_line":"                        self.fields[subnet_field_name].choices \u003d [(\u0027 \u0027, \u0027 \u0027)]"},{"line_number":75,"context_line":"            except ("},{"line_number":76,"context_line":"                sdk_exceptions.NotFoundException,"},{"line_number":77,"context_line":"                sdk_exceptions.OpenStackCloudException,"},{"line_number":78,"context_line":"                Exception"},{"line_number":79,"context_line":"                ):"},{"line_number":80,"context_line":"                # If we can\u0027t get networks, there\u0027s nothing to populate"}],"source_content_type":"text/x-python","patch_set":3,"id":"c502acc2_790fd252","line":77,"updated":"2025-10-22 08:21:28.000000000","message":"nit: as I see this (OpenStackCloudException) was kept for backward compatibility only and wrappe/symlink for SDKException","commit_id":"20464a44e16487d5b058ccb050fb2f37485f8911"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"5dc194cf5493bf9678a48f9cdb80960127024a50","unresolved":false,"context_lines":[{"line_number":74,"context_line":"                        self.fields[subnet_field_name].choices \u003d [(\u0027 \u0027, \u0027 \u0027)]"},{"line_number":75,"context_line":"            except ("},{"line_number":76,"context_line":"                sdk_exceptions.NotFoundException,"},{"line_number":77,"context_line":"                sdk_exceptions.OpenStackCloudException,"},{"line_number":78,"context_line":"                Exception"},{"line_number":79,"context_line":"                ):"},{"line_number":80,"context_line":"                # If we can\u0027t get networks, there\u0027s nothing to populate"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a02cab7_1d29fd17","line":77,"in_reply_to":"c502acc2_790fd252","updated":"2026-04-26 06:17:32.000000000","message":"ack, replaced this with SDKException. Thank you!","commit_id":"20464a44e16487d5b058ccb050fb2f37485f8911"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"ac7edfdc62e0235bcbf0ae3f5c0000337b9451ae","unresolved":true,"context_lines":[{"line_number":42,"context_line":"                            [(utils.transform_dashed_name(choice.id),"},{"line_number":43,"context_line":"                              choice.name_or_id) for choice in net_choices],"},{"line_number":44,"context_line":"                    label\u003d_(\"Neutron Network\"), widget\u003dforms.Select("},{"line_number":45,"context_line":"                        attrs\u003d{\u0027class\u0027: \u0027switchable\u0027, \u0027data-slug\u0027: \u0027net\u0027}))"},{"line_number":46,"context_line":"                for net in net_choices:"},{"line_number":47,"context_line":"                    # For each network create switched choice field with"},{"line_number":48,"context_line":"                    # the its subnet choices"}],"source_content_type":"text/x-python","patch_set":6,"id":"8d8bcbf7_9adbcca6","line":45,"updated":"2026-04-29 03:53:23.000000000","message":"I guess I would have thought for the except to be around here instead of L74","commit_id":"2fed2b2fe72f2207cf24039f00c121bfa286bffc"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"4f8bc4a6674bd114d8d30a55bb6e915dc70f25e3","unresolved":false,"context_lines":[{"line_number":42,"context_line":"                            [(utils.transform_dashed_name(choice.id),"},{"line_number":43,"context_line":"                              choice.name_or_id) for choice in net_choices],"},{"line_number":44,"context_line":"                    label\u003d_(\"Neutron Network\"), widget\u003dforms.Select("},{"line_number":45,"context_line":"                        attrs\u003d{\u0027class\u0027: \u0027switchable\u0027, \u0027data-slug\u0027: \u0027net\u0027}))"},{"line_number":46,"context_line":"                for net in net_choices:"},{"line_number":47,"context_line":"                    # For each network create switched choice field with"},{"line_number":48,"context_line":"                    # the its subnet choices"}],"source_content_type":"text/x-python","patch_set":6,"id":"9d84b7d2_4e822d33","line":45,"in_reply_to":"8c7967b4_0c003c1a","updated":"2026-04-29 16:12:22.000000000","message":"Yes, I guess what you did is better.","commit_id":"2fed2b2fe72f2207cf24039f00c121bfa286bffc"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"61a75f6d0fec0516a7dd8e5a503870d0247c2d76","unresolved":false,"context_lines":[{"line_number":42,"context_line":"                            [(utils.transform_dashed_name(choice.id),"},{"line_number":43,"context_line":"                              choice.name_or_id) for choice in net_choices],"},{"line_number":44,"context_line":"                    label\u003d_(\"Neutron Network\"), widget\u003dforms.Select("},{"line_number":45,"context_line":"                        attrs\u003d{\u0027class\u0027: \u0027switchable\u0027, \u0027data-slug\u0027: \u0027net\u0027}))"},{"line_number":46,"context_line":"                for net in net_choices:"},{"line_number":47,"context_line":"                    # For each network create switched choice field with"},{"line_number":48,"context_line":"                    # the its subnet choices"}],"source_content_type":"text/x-python","patch_set":6,"id":"8c7967b4_0c003c1a","line":45,"in_reply_to":"8d8bcbf7_9adbcca6","updated":"2026-04-29 06:18:01.000000000","message":"you mean right around the neutron.network_list call.. I can do that","commit_id":"2fed2b2fe72f2207cf24039f00c121bfa286bffc"}]}
