)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":38842,"name":"Silvia Wachira","display_name":"Silvia Wachira","email":"wachirasilvia8@gmail.com","username":"Silvia-Wachira"},"change_message_id":"203c3732e54b2c2066e9d6dee23f2f2451538ad2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"cdf55238_129e2b3c","updated":"2026-03-26 06:25:10.000000000","message":"Done, I have made the changes. Thank you for the review! Could you please take another look?","commit_id":"34e90444f74b1f21cfdfaef8e33329a39a441fa8"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"364f8eafe8aba3cc16dd0d2cc8fecb4e7d06b2e2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"41d53a6c_6a342d69","updated":"2026-03-25 21:38:04.000000000","message":"Looks good for the most part. Please take a look at the comments inline.","commit_id":"34e90444f74b1f21cfdfaef8e33329a39a441fa8"},{"author":{"_account_id":38842,"name":"Silvia Wachira","display_name":"Silvia Wachira","email":"wachirasilvia8@gmail.com","username":"Silvia-Wachira"},"change_message_id":"571e4098e2ed3f92b0a5ecc543c716fa412f797a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"0d999c87_15b03e52","updated":"2026-03-27 06:34:55.000000000","message":"recheck","commit_id":"725e9fccace8db88d6f3fde781d3f0d402162d42"},{"author":{"_account_id":38842,"name":"Silvia Wachira","display_name":"Silvia Wachira","email":"wachirasilvia8@gmail.com","username":"Silvia-Wachira"},"change_message_id":"a12ac014cbf6a6218d807c3beeb0b22ed31299ac","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"dc8e2417_6e91a20f","updated":"2026-03-26 11:03:35.000000000","message":"recheck","commit_id":"725e9fccace8db88d6f3fde781d3f0d402162d42"},{"author":{"_account_id":38842,"name":"Silvia Wachira","display_name":"Silvia Wachira","email":"wachirasilvia8@gmail.com","username":"Silvia-Wachira"},"change_message_id":"489b3a27904e2856205d54cb734070f4310f5cd1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"eec0e4ea_caafd79e","updated":"2026-03-26 12:10:58.000000000","message":"recheck","commit_id":"725e9fccace8db88d6f3fde781d3f0d402162d42"},{"author":{"_account_id":38842,"name":"Silvia Wachira","display_name":"Silvia Wachira","email":"wachirasilvia8@gmail.com","username":"Silvia-Wachira"},"change_message_id":"361b61ea1575a127f1c002d3ab9b0c9d42f511de","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"4baba9b0_a680fe52","updated":"2026-03-31 11:52:17.000000000","message":"I have made the necessary changes. Please review","commit_id":"0d84e92d24634a8f3154992def138c5981d66800"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"227a070db731f029bb2b1406f48e7a9df353bfb8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"d86224cc_19425914","updated":"2026-03-31 21:17:54.000000000","message":"Thanks for working on the comments, LGTM!","commit_id":"2bf604f7b8590f53c34f87c3792b1e1d4e88c5f8"},{"author":{"_account_id":38842,"name":"Silvia Wachira","display_name":"Silvia Wachira","email":"wachirasilvia8@gmail.com","username":"Silvia-Wachira"},"change_message_id":"1714a6a3fde8b37dd3c7caff0edcf917bfcc874a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"168a23e9_f12777f8","updated":"2026-04-01 00:05:20.000000000","message":"run-NetApp CI","commit_id":"2bf604f7b8590f53c34f87c3792b1e1d4e88c5f8"}],"manila/api/v2/share_network_subnets.py":[{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"bdadc4aafa5a1c7711c184ac9312088d6e80dfbd","unresolved":true,"context_lines":[{"line_number":92,"context_line":"            if shares:"},{"line_number":93,"context_line":"                msg \u003d _(\"Cannot delete share network subnet %(id)s \""},{"line_number":94,"context_line":"                        \"of share network %(network_id)s, it has\""},{"line_number":95,"context_line":"                        \" one or more shares.\") % {"},{"line_number":96,"context_line":"                    \u0027id\u0027: share_network_subnet_id,"},{"line_number":97,"context_line":"                    \u0027network_id\u0027: share_network_id}"},{"line_number":98,"context_line":"                LOG.error(msg)"}],"source_content_type":"text/x-python","patch_set":7,"id":"09de3826_1301d885","line":95,"updated":"2026-03-31 10:34:56.000000000","message":"maintain space at the end of line in msg. This will be consistent.","commit_id":"725e9fccace8db88d6f3fde781d3f0d402162d42"},{"author":{"_account_id":38842,"name":"Silvia Wachira","display_name":"Silvia Wachira","email":"wachirasilvia8@gmail.com","username":"Silvia-Wachira"},"change_message_id":"d87a8fa10e04c4aaafaf7b203a408b3ce20cd418","unresolved":true,"context_lines":[{"line_number":92,"context_line":"            if shares:"},{"line_number":93,"context_line":"                msg \u003d _(\"Cannot delete share network subnet %(id)s \""},{"line_number":94,"context_line":"                        \"of share network %(network_id)s, it has\""},{"line_number":95,"context_line":"                        \" one or more shares.\") % {"},{"line_number":96,"context_line":"                    \u0027id\u0027: share_network_subnet_id,"},{"line_number":97,"context_line":"                    \u0027network_id\u0027: share_network_id}"},{"line_number":98,"context_line":"                LOG.error(msg)"}],"source_content_type":"text/x-python","patch_set":7,"id":"6d3c198a_177bbe86","line":95,"in_reply_to":"09de3826_1301d885","updated":"2026-03-31 11:47:25.000000000","message":"Done. Moved the space to the end of the line for consistency. Also applied the same fix to the share groups message.","commit_id":"725e9fccace8db88d6f3fde781d3f0d402162d42"},{"author":{"_account_id":38842,"name":"Silvia Wachira","display_name":"Silvia Wachira","email":"wachirasilvia8@gmail.com","username":"Silvia-Wachira"},"change_message_id":"16ded606209be859c7281f0db722b54e155a8185","unresolved":false,"context_lines":[{"line_number":92,"context_line":"            if shares:"},{"line_number":93,"context_line":"                msg \u003d _(\"Cannot delete share network subnet %(id)s \""},{"line_number":94,"context_line":"                        \"of share network %(network_id)s, it has\""},{"line_number":95,"context_line":"                        \" one or more shares.\") % {"},{"line_number":96,"context_line":"                    \u0027id\u0027: share_network_subnet_id,"},{"line_number":97,"context_line":"                    \u0027network_id\u0027: share_network_id}"},{"line_number":98,"context_line":"                LOG.error(msg)"}],"source_content_type":"text/x-python","patch_set":7,"id":"952c1779_1f10268f","line":95,"in_reply_to":"6d3c198a_177bbe86","updated":"2026-03-31 11:47:52.000000000","message":"Done","commit_id":"725e9fccace8db88d6f3fde781d3f0d402162d42"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"857bbf971522343582622312b09d9e2623f4558c","unresolved":true,"context_lines":[{"line_number":114,"context_line":"        if not self._all_share_servers_are_auto_deletable("},{"line_number":115,"context_line":"                share_network_subnet):"},{"line_number":116,"context_line":"            msg \u003d _(\"The service cannot determine if there are any \""},{"line_number":117,"context_line":"                    \"non-managed shares on the share network subnet %(id)s \""},{"line_number":118,"context_line":"                    \"of share network %(network_id)s, so it cannot be \""},{"line_number":119,"context_line":"                    \"deleted. Please contact the cloud administrator to \""},{"line_number":120,"context_line":"                    \"rectify.\") % {"}],"source_content_type":"text/x-python","patch_set":10,"id":"f8a716f2_35c76bc5","line":117,"updated":"2026-03-31 20:56:18.000000000","message":"Good catch fixing the missing space before \"so\" in the old\nmessage (\"subnet %(id)s,so it cannot\" -\u003e now properly spaced).\nThis was a pre-existing bug.","commit_id":"2bf604f7b8590f53c34f87c3792b1e1d4e88c5f8"},{"author":{"_account_id":38842,"name":"Silvia Wachira","display_name":"Silvia Wachira","email":"wachirasilvia8@gmail.com","username":"Silvia-Wachira"},"change_message_id":"53dd1295ed4b51fd2adfe1605b3421acf92d527d","unresolved":false,"context_lines":[{"line_number":114,"context_line":"        if not self._all_share_servers_are_auto_deletable("},{"line_number":115,"context_line":"                share_network_subnet):"},{"line_number":116,"context_line":"            msg \u003d _(\"The service cannot determine if there are any \""},{"line_number":117,"context_line":"                    \"non-managed shares on the share network subnet %(id)s \""},{"line_number":118,"context_line":"                    \"of share network %(network_id)s, so it cannot be \""},{"line_number":119,"context_line":"                    \"deleted. Please contact the cloud administrator to \""},{"line_number":120,"context_line":"                    \"rectify.\") % {"}],"source_content_type":"text/x-python","patch_set":10,"id":"6a15edb9_d3ad8dbc","line":117,"in_reply_to":"f159fbf0_35ae7fcd","updated":"2026-04-07 08:14:13.000000000","message":"Done","commit_id":"2bf604f7b8590f53c34f87c3792b1e1d4e88c5f8"},{"author":{"_account_id":38842,"name":"Silvia Wachira","display_name":"Silvia Wachira","email":"wachirasilvia8@gmail.com","username":"Silvia-Wachira"},"change_message_id":"cd3c62b7e1e0ae9831b7581b12b58c296da0a7d8","unresolved":true,"context_lines":[{"line_number":114,"context_line":"        if not self._all_share_servers_are_auto_deletable("},{"line_number":115,"context_line":"                share_network_subnet):"},{"line_number":116,"context_line":"            msg \u003d _(\"The service cannot determine if there are any \""},{"line_number":117,"context_line":"                    \"non-managed shares on the share network subnet %(id)s \""},{"line_number":118,"context_line":"                    \"of share network %(network_id)s, so it cannot be \""},{"line_number":119,"context_line":"                    \"deleted. Please contact the cloud administrator to \""},{"line_number":120,"context_line":"                    \"rectify.\") % {"}],"source_content_type":"text/x-python","patch_set":10,"id":"f159fbf0_35ae7fcd","line":117,"in_reply_to":"f8a716f2_35c76bc5","updated":"2026-04-07 08:13:50.000000000","message":"Thanks for the note! Glad the pre-existing spacing issue got caught as part of this fix.","commit_id":"2bf604f7b8590f53c34f87c3792b1e1d4e88c5f8"}],"manila/tests/api/v2/test_share_network_subnets.py":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"364f8eafe8aba3cc16dd0d2cc8fecb4e7d06b2e2","unresolved":true,"context_lines":[{"line_number":170,"context_line":"            self.controller, \u0027_all_share_servers_are_auto_deletable\u0027,"},{"line_number":171,"context_line":"            mock.Mock(return_value\u003dFalse))"},{"line_number":172,"context_line":""},{"line_number":173,"context_line":"        self.assertRaisesRegex(exc.HTTPConflict,"},{"line_number":174,"context_line":"                               self.share_network[\u0027id\u0027],"},{"line_number":175,"context_line":"                               self.controller.delete,"},{"line_number":176,"context_line":"                               req,"}],"source_content_type":"text/x-python","patch_set":5,"id":"0cdbbc3c_08aee0b3","line":173,"range":{"start_line":173,"start_character":13,"end_line":173,"end_character":30},"updated":"2026-03-25 21:38:04.000000000","message":"not necessary","commit_id":"34e90444f74b1f21cfdfaef8e33329a39a441fa8"},{"author":{"_account_id":38842,"name":"Silvia Wachira","display_name":"Silvia Wachira","email":"wachirasilvia8@gmail.com","username":"Silvia-Wachira"},"change_message_id":"203c3732e54b2c2066e9d6dee23f2f2451538ad2","unresolved":false,"context_lines":[{"line_number":170,"context_line":"            self.controller, \u0027_all_share_servers_are_auto_deletable\u0027,"},{"line_number":171,"context_line":"            mock.Mock(return_value\u003dFalse))"},{"line_number":172,"context_line":""},{"line_number":173,"context_line":"        self.assertRaisesRegex(exc.HTTPConflict,"},{"line_number":174,"context_line":"                               self.share_network[\u0027id\u0027],"},{"line_number":175,"context_line":"                               self.controller.delete,"},{"line_number":176,"context_line":"                               req,"}],"source_content_type":"text/x-python","patch_set":5,"id":"576701e3_a11afda7","line":173,"range":{"start_line":173,"start_character":13,"end_line":173,"end_character":30},"in_reply_to":"0cdbbc3c_08aee0b3","updated":"2026-03-26 06:25:10.000000000","message":"Done","commit_id":"34e90444f74b1f21cfdfaef8e33329a39a441fa8"},{"author":{"_account_id":38842,"name":"Silvia Wachira","display_name":"Silvia Wachira","email":"wachirasilvia8@gmail.com","username":"Silvia-Wachira"},"change_message_id":"9d09f579ebd963811fe044980d554bcd4d4ca99f","unresolved":false,"context_lines":[{"line_number":170,"context_line":"            self.controller, \u0027_all_share_servers_are_auto_deletable\u0027,"},{"line_number":171,"context_line":"            mock.Mock(return_value\u003dFalse))"},{"line_number":172,"context_line":""},{"line_number":173,"context_line":"        self.assertRaisesRegex(exc.HTTPConflict,"},{"line_number":174,"context_line":"                               self.share_network[\u0027id\u0027],"},{"line_number":175,"context_line":"                               self.controller.delete,"},{"line_number":176,"context_line":"                               req,"}],"source_content_type":"text/x-python","patch_set":5,"id":"3256b8a0_c6b43b41","line":173,"range":{"start_line":173,"start_character":13,"end_line":173,"end_character":30},"in_reply_to":"576701e3_a11afda7","updated":"2026-03-31 17:34:50.000000000","message":"Thanks, I’ve removed the unnecessary parts as suggested.","commit_id":"34e90444f74b1f21cfdfaef8e33329a39a441fa8"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"364f8eafe8aba3cc16dd0d2cc8fecb4e7d06b2e2","unresolved":true,"context_lines":[{"line_number":203,"context_line":"            db_api, \u0027share_instance_get_all_by_share_server\u0027,"},{"line_number":204,"context_line":"            mock.Mock(return_value\u003d[self.share]))"},{"line_number":205,"context_line":""},{"line_number":206,"context_line":"        self.assertRaisesRegex(exc.HTTPConflict,"},{"line_number":207,"context_line":"                               self.share_network[\u0027id\u0027],"},{"line_number":208,"context_line":"                               self.controller.delete,"},{"line_number":209,"context_line":"                               req,"}],"source_content_type":"text/x-python","patch_set":5,"id":"b77ce103_36599503","line":206,"range":{"start_line":206,"start_character":13,"end_line":206,"end_character":30},"updated":"2026-03-25 21:38:04.000000000","message":"not necessary","commit_id":"34e90444f74b1f21cfdfaef8e33329a39a441fa8"},{"author":{"_account_id":38842,"name":"Silvia Wachira","display_name":"Silvia Wachira","email":"wachirasilvia8@gmail.com","username":"Silvia-Wachira"},"change_message_id":"203c3732e54b2c2066e9d6dee23f2f2451538ad2","unresolved":false,"context_lines":[{"line_number":203,"context_line":"            db_api, \u0027share_instance_get_all_by_share_server\u0027,"},{"line_number":204,"context_line":"            mock.Mock(return_value\u003d[self.share]))"},{"line_number":205,"context_line":""},{"line_number":206,"context_line":"        self.assertRaisesRegex(exc.HTTPConflict,"},{"line_number":207,"context_line":"                               self.share_network[\u0027id\u0027],"},{"line_number":208,"context_line":"                               self.controller.delete,"},{"line_number":209,"context_line":"                               req,"}],"source_content_type":"text/x-python","patch_set":5,"id":"6eee355d_858edd7b","line":206,"range":{"start_line":206,"start_character":13,"end_line":206,"end_character":30},"in_reply_to":"b77ce103_36599503","updated":"2026-03-26 06:25:10.000000000","message":"Done","commit_id":"34e90444f74b1f21cfdfaef8e33329a39a441fa8"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"857bbf971522343582622312b09d9e2623f4558c","unresolved":true,"context_lines":[{"line_number":154,"context_line":"        self.mock_policy_check.assert_called_once_with("},{"line_number":155,"context_line":"            context, self.resource_name, \u0027delete\u0027)"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"    def test_delete_subnet_with_share_servers_fail(self):"},{"line_number":158,"context_line":"        req \u003d fakes.HTTPRequest.blank(\u0027/subnets/%s\u0027 % self.subnet[\u0027id\u0027],"},{"line_number":159,"context_line":"                                      version\u003d\"2.51\")"},{"line_number":160,"context_line":"        context \u003d req.environ[\u0027manila.context\u0027]"}],"source_content_type":"text/x-python","patch_set":10,"id":"ee7b49ca_2a51c7bc","line":157,"updated":"2026-03-31 20:56:18.000000000","message":"Pre-existing issue (not introduced by this patch):\ntest_delete_subnet_with_share_servers_fail does not mock\ndb_api.share_network_get, unlike the other two delete tests.\nIt also doesn\u0027t mock share_group_get_all_by_share_server.\nBoth are called in the code path. This works because\nmock_object defaults to a no-op mock, but it would be more\nexplicit to include them. Could be addressed in a follow-up.","commit_id":"2bf604f7b8590f53c34f87c3792b1e1d4e88c5f8"},{"author":{"_account_id":38842,"name":"Silvia Wachira","display_name":"Silvia Wachira","email":"wachirasilvia8@gmail.com","username":"Silvia-Wachira"},"change_message_id":"53dd1295ed4b51fd2adfe1605b3421acf92d527d","unresolved":false,"context_lines":[{"line_number":154,"context_line":"        self.mock_policy_check.assert_called_once_with("},{"line_number":155,"context_line":"            context, self.resource_name, \u0027delete\u0027)"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"    def test_delete_subnet_with_share_servers_fail(self):"},{"line_number":158,"context_line":"        req \u003d fakes.HTTPRequest.blank(\u0027/subnets/%s\u0027 % self.subnet[\u0027id\u0027],"},{"line_number":159,"context_line":"                                      version\u003d\"2.51\")"},{"line_number":160,"context_line":"        context \u003d req.environ[\u0027manila.context\u0027]"}],"source_content_type":"text/x-python","patch_set":10,"id":"ba9cbed1_59fadca0","line":157,"in_reply_to":"1bd32c97_cef6c14c","updated":"2026-04-07 08:14:13.000000000","message":"Done","commit_id":"2bf604f7b8590f53c34f87c3792b1e1d4e88c5f8"},{"author":{"_account_id":38842,"name":"Silvia Wachira","display_name":"Silvia Wachira","email":"wachirasilvia8@gmail.com","username":"Silvia-Wachira"},"change_message_id":"cd3c62b7e1e0ae9831b7581b12b58c296da0a7d8","unresolved":true,"context_lines":[{"line_number":154,"context_line":"        self.mock_policy_check.assert_called_once_with("},{"line_number":155,"context_line":"            context, self.resource_name, \u0027delete\u0027)"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"    def test_delete_subnet_with_share_servers_fail(self):"},{"line_number":158,"context_line":"        req \u003d fakes.HTTPRequest.blank(\u0027/subnets/%s\u0027 % self.subnet[\u0027id\u0027],"},{"line_number":159,"context_line":"                                      version\u003d\"2.51\")"},{"line_number":160,"context_line":"        context \u003d req.environ[\u0027manila.context\u0027]"}],"source_content_type":"text/x-python","patch_set":10,"id":"1bd32c97_cef6c14c","line":157,"in_reply_to":"ee7b49ca_2a51c7bc","updated":"2026-04-07 08:13:50.000000000","message":"Thanks for flagging this! I\u0027ll address it in a follow-up patch by adding explicit mocks for db_api.share_network_get and share_group_get_all_by_share_server to make the test intent clearer.","commit_id":"2bf604f7b8590f53c34f87c3792b1e1d4e88c5f8"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"857bbf971522343582622312b09d9e2623f4558c","unresolved":true,"context_lines":[{"line_number":217,"context_line":"        )"},{"line_number":218,"context_line":"        self.mock_policy_check.assert_called_once_with("},{"line_number":219,"context_line":"            context, self.resource_name, \u0027delete\u0027)"},{"line_number":220,"context_line":""},{"line_number":221,"context_line":"    def test_delete_subnet_with_share_groups_fail(self):"},{"line_number":222,"context_line":"        req \u003d fakes.HTTPRequest.blank(\u0027/subnets/%s\u0027 % self.subnet[\u0027id\u0027],"},{"line_number":223,"context_line":"                                      version\u003d\"2.51\")"}],"source_content_type":"text/x-python","patch_set":10,"id":"d7bf13e9_8237898b","line":220,"updated":"2026-03-31 20:56:18.000000000","message":"Nice addition of this missing test case. Consider also updating\ntest_delete_subnet_with_shares_fail (line 190) to use\nassertRaisesRegex (like you did here) to verify the share\nnetwork ID appears in the shares conflict error message too.\nNot blocking.","commit_id":"2bf604f7b8590f53c34f87c3792b1e1d4e88c5f8"},{"author":{"_account_id":38842,"name":"Silvia Wachira","display_name":"Silvia Wachira","email":"wachirasilvia8@gmail.com","username":"Silvia-Wachira"},"change_message_id":"53dd1295ed4b51fd2adfe1605b3421acf92d527d","unresolved":false,"context_lines":[{"line_number":217,"context_line":"        )"},{"line_number":218,"context_line":"        self.mock_policy_check.assert_called_once_with("},{"line_number":219,"context_line":"            context, self.resource_name, \u0027delete\u0027)"},{"line_number":220,"context_line":""},{"line_number":221,"context_line":"    def test_delete_subnet_with_share_groups_fail(self):"},{"line_number":222,"context_line":"        req \u003d fakes.HTTPRequest.blank(\u0027/subnets/%s\u0027 % self.subnet[\u0027id\u0027],"},{"line_number":223,"context_line":"                                      version\u003d\"2.51\")"}],"source_content_type":"text/x-python","patch_set":10,"id":"13722c77_2db17a5d","line":220,"in_reply_to":"4166ea94_0bc4fe73","updated":"2026-04-07 08:14:13.000000000","message":"Done","commit_id":"2bf604f7b8590f53c34f87c3792b1e1d4e88c5f8"},{"author":{"_account_id":38842,"name":"Silvia Wachira","display_name":"Silvia Wachira","email":"wachirasilvia8@gmail.com","username":"Silvia-Wachira"},"change_message_id":"cd3c62b7e1e0ae9831b7581b12b58c296da0a7d8","unresolved":true,"context_lines":[{"line_number":217,"context_line":"        )"},{"line_number":218,"context_line":"        self.mock_policy_check.assert_called_once_with("},{"line_number":219,"context_line":"            context, self.resource_name, \u0027delete\u0027)"},{"line_number":220,"context_line":""},{"line_number":221,"context_line":"    def test_delete_subnet_with_share_groups_fail(self):"},{"line_number":222,"context_line":"        req \u003d fakes.HTTPRequest.blank(\u0027/subnets/%s\u0027 % self.subnet[\u0027id\u0027],"},{"line_number":223,"context_line":"                                      version\u003d\"2.51\")"}],"source_content_type":"text/x-python","patch_set":10,"id":"4166ea94_0bc4fe73","line":220,"in_reply_to":"d7bf13e9_8237898b","updated":"2026-04-07 08:13:50.000000000","message":"Thank you for the review. I will address this in a follow up patch.","commit_id":"2bf604f7b8590f53c34f87c3792b1e1d4e88c5f8"}]}