)]}'
{"neutron/tests/unit/db/test_ipam_pluggable_backend.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"3c05044bec9ce8ba41f3e57e59233b94e5972ca0","unresolved":true,"context_lines":[{"line_number":415,"context_line":"                         subnetpool_id\u003dconstants.IPV6_PD_POOL_ID,"},{"line_number":416,"context_line":"                         ipv6_ra_mode\u003dconstants.IPV6_SLAAC,"},{"line_number":417,"context_line":"                         ipv6_address_mode\u003dconstants.IPV6_SLAAC):"},{"line_number":418,"context_line":"            self.assertEqual(3, pool_mock.get_instance.call_count)"},{"line_number":419,"context_line":"            self.assertTrue(mocks[\u0027driver\u0027].allocate_subnet.called)"},{"line_number":420,"context_line":"            request \u003d mocks[\u0027driver\u0027].allocate_subnet.call_args[0][0]"},{"line_number":421,"context_line":"            self.assertIsInstance(request, ipam_req.SpecificSubnetRequest)"}],"source_content_type":"text/x-python","patch_set":1,"id":"aeed56c3_0f8c8703","line":418,"range":{"start_line":418,"start_character":0,"end_line":418,"end_character":13},"updated":"2022-10-03 11:48:13.000000000","message":"Sorry, why this change?","commit_id":"a3e68e8f76e2792c30d1527bb20ee5961c6a8a53"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"d3809e34bfd675a016c2d850f9bbcd2bb5d9571c","unresolved":true,"context_lines":[{"line_number":415,"context_line":"                         subnetpool_id\u003dconstants.IPV6_PD_POOL_ID,"},{"line_number":416,"context_line":"                         ipv6_ra_mode\u003dconstants.IPV6_SLAAC,"},{"line_number":417,"context_line":"                         ipv6_address_mode\u003dconstants.IPV6_SLAAC):"},{"line_number":418,"context_line":"            self.assertEqual(3, pool_mock.get_instance.call_count)"},{"line_number":419,"context_line":"            self.assertTrue(mocks[\u0027driver\u0027].allocate_subnet.called)"},{"line_number":420,"context_line":"            request \u003d mocks[\u0027driver\u0027].allocate_subnet.call_args[0][0]"},{"line_number":421,"context_line":"            self.assertIsInstance(request, ipam_req.SpecificSubnetRequest)"}],"source_content_type":"text/x-python","patch_set":1,"id":"e582b685_4af9763f","line":418,"range":{"start_line":418,"start_character":0,"end_line":418,"end_character":13},"in_reply_to":"aeed56c3_0f8c8703","updated":"2022-10-04 07:09:13.000000000","message":"TBH I didn\u0027t check it deeply but when I added core plugin it started failing and I assumed (maybe wrong and I should check it better) that it\u0027s called also somewhere during the core plugin initialization.","commit_id":"a3e68e8f76e2792c30d1527bb20ee5961c6a8a53"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"6401e38b0333397191c1ba7a0a350e43de2efede","unresolved":true,"context_lines":[{"line_number":415,"context_line":"                         subnetpool_id\u003dconstants.IPV6_PD_POOL_ID,"},{"line_number":416,"context_line":"                         ipv6_ra_mode\u003dconstants.IPV6_SLAAC,"},{"line_number":417,"context_line":"                         ipv6_address_mode\u003dconstants.IPV6_SLAAC):"},{"line_number":418,"context_line":"            self.assertEqual(3, pool_mock.get_instance.call_count)"},{"line_number":419,"context_line":"            self.assertTrue(mocks[\u0027driver\u0027].allocate_subnet.called)"},{"line_number":420,"context_line":"            request \u003d mocks[\u0027driver\u0027].allocate_subnet.call_args[0][0]"},{"line_number":421,"context_line":"            self.assertIsInstance(request, ipam_req.SpecificSubnetRequest)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5d5cee06_3fffc056","line":418,"range":{"start_line":418,"start_character":0,"end_line":418,"end_character":13},"in_reply_to":"e582b685_4af9763f","updated":"2022-10-04 13:00:29.000000000","message":"I found it. When using \"TestPlugin\", that inherits from \"Ml2Plugin\", we call \"Ml2Plugin.create_subnet\". This method calls \"_after_create_subnet\" -\u003e \"_create_subnet_postcommit\" that calls \"self.ipam.get_subnet(context, result[\u0027id\u0027])\". This is the extra new call detected here.\n\nBefore this patch, we where calling \"NeutronDbPluginV2.create_subnet\" instead.","commit_id":"a3e68e8f76e2792c30d1527bb20ee5961c6a8a53"}]}