)]}'
{"neutron/db/ipam_backend_mixin.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"957283d9d75448913baead8b57a6293db59ff625","unresolved":true,"context_lines":[{"line_number":311,"context_line":"                subnet_first_ip \u003d netaddr.IPAddress(subnet.first)"},{"line_number":312,"context_line":"                subnet_last_ip \u003d netaddr.IPAddress(subnet.last)"},{"line_number":313,"context_line":"        else:  # IPv6 case"},{"line_number":314,"context_line":"            subnet_first_ip \u003d netaddr.IPAddress(subnet.first)"},{"line_number":315,"context_line":"            subnet_last_ip \u003d netaddr.IPAddress(subnet.last)"},{"line_number":316,"context_line":""},{"line_number":317,"context_line":"        LOG.debug(\"Performing IP validity checks on allocation pools\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"c6d2e63c_4e2f6bed","line":314,"updated":"2023-10-26 09:19:15.000000000","message":"As I remember the gw IP should be the 1st address, so the gw IP should be left out from the pool (see the api-ref:\n\"Allocation pools with start and end IP addresses for this subnet. If allocation_pools are not specified, OpenStack Networking automatically allocates pools for covering all IP addresses in the CIDR, excluding the address reserved for the subnet gateway by default.\"","commit_id":"4ab20e850e097c5756a844c43a890d47b0195c6b"},{"author":{"_account_id":28056,"name":"Liushy","email":"liuxie_11@163.com","username":"liuxie_11"},"change_message_id":"9500bd55388c873820a1a9f674ea9b8ebdce8abe","unresolved":true,"context_lines":[{"line_number":311,"context_line":"                subnet_first_ip \u003d netaddr.IPAddress(subnet.first)"},{"line_number":312,"context_line":"                subnet_last_ip \u003d netaddr.IPAddress(subnet.last)"},{"line_number":313,"context_line":"        else:  # IPv6 case"},{"line_number":314,"context_line":"            subnet_first_ip \u003d netaddr.IPAddress(subnet.first)"},{"line_number":315,"context_line":"            subnet_last_ip \u003d netaddr.IPAddress(subnet.last)"},{"line_number":316,"context_line":""},{"line_number":317,"context_line":"        LOG.debug(\"Performing IP validity checks on allocation pools\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"a93cf0c4_a7227969","line":314,"in_reply_to":"19a2fd99_4865520d","updated":"2023-10-27 03:29:44.000000000","message":"Thanks, I know what you mean.\nHow do you explain the following case:\n\nIf we create one subnet with CIDR 2003::/64 without a gateway:\nneutron subnet-create --no-gateway --ip-version 6 net_v6 2003::/64\n\nWe could find that the allocation_pools of the the response body do not contain \u00272003::\u0027.\nWhy does IPAM delete the first IP of the CIDR?","commit_id":"4ab20e850e097c5756a844c43a890d47b0195c6b"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"fa96e44b8106021aa4a73a4a96226cbc8b8d3cf7","unresolved":true,"context_lines":[{"line_number":311,"context_line":"                subnet_first_ip \u003d netaddr.IPAddress(subnet.first)"},{"line_number":312,"context_line":"                subnet_last_ip \u003d netaddr.IPAddress(subnet.last)"},{"line_number":313,"context_line":"        else:  # IPv6 case"},{"line_number":314,"context_line":"            subnet_first_ip \u003d netaddr.IPAddress(subnet.first)"},{"line_number":315,"context_line":"            subnet_last_ip \u003d netaddr.IPAddress(subnet.last)"},{"line_number":316,"context_line":""},{"line_number":317,"context_line":"        LOG.debug(\"Performing IP validity checks on allocation pools\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"19a2fd99_4865520d","line":314,"in_reply_to":"c6d2e63c_4e2f6bed","updated":"2023-10-26 18:05:52.000000000","message":"Right, I agree with Lajos, I think we chose +1 because we use the first address as the gateway.\n\nhttps://review.opendev.org/c/openstack/neutron/+/647484","commit_id":"4ab20e850e097c5756a844c43a890d47b0195c6b"}]}