Reviewed: https://review.openstack.org/493280 Committed: https://git.openstack.org/cgit/openstack/neutron-lib/commit/?id=fe389eea25286a067f92a55ea9ad7d73160bbb25 Submitter: Zuul Branch: master
commit fe389eea25286a067f92a55ea9ad7d73160bbb25 Author: Reedip <reedip.baner...@gmail.com> Date: Sat Aug 12 06:25:32 2017 +0000 Add ipv6 canonical address converter to API This patch takes care of converting of IPv6 address for gateway-ip, cidr and allocation-pools to canonical format. Closes-Bug:1531103 Change-Id: Ibef0bd9a2f7b7dd8ddccc183f17aa31e426c7fd0 ** Changed in: neutron Status: In Progress => Fix Released -- You received this bug notification because you are a member of Yahoo! Engineering Team, which is subscribed to neutron. https://bugs.launchpad.net/bugs/1531103 Title: can update the gateway of ipv6 subnet with ipv6 address which has a leading "0" via cli Status in neutron: Fix Released Bug description: [Summary] can update the gateway of ipv6 subnet with ipv6 address which has a leading "0" via cli [Topo] devstack all-in-one node [Description and expect result] if update the gateway of ipv6 subnet with ipv6 address which has a leading "0", should reject leading '0's to avoid ambiguity. [Reproduceable or not] reproduceable [Recreate Steps] 1) create 1 ipv6 subnet: root@45-59:/opt/stack/devstack# neutron subnet-create net11 2::1/64 --ip-version 6 --name sub11 Created a new subnet: +-------------------+----------------------------------------------------+ | Field | Value | +-------------------+----------------------------------------------------+ | allocation_pools | {"start": "2::2", "end": "2::ffff:ffff:ffff:ffff"} | | cidr | 2::/64 | | dns_nameservers | | | enable_dhcp | True | | gateway_ip | 2::1 | | host_routes | | | id | e2a09fa2-52ce-4f68-9619-b635529213d0 | | ip_version | 6 | | ipv6_address_mode | | | ipv6_ra_mode | | | name | sub11 | | network_id | af909be2-5413-40a0-984f-cd58dc7289c9 | | subnetpool_id | | | tenant_id | bc125faaf00041f492eaf10f72a37c2d | +-------------------+----------------------------------------------------+ root@45-59:/opt/stack/devstack# 2)update the gateway of subnet with needless "0" in the ip address: root@45-59:/opt/stack/devstack# neutron subnet-update --gateway 2::0001 sub11 Updated subnet: sub11 root@45-59:/opt/stack/devstack# root@45-59:/opt/stack/devstack# neutron subnet-show sub11 +-------------------+----------------------------------------------------+ | Field | Value | +-------------------+----------------------------------------------------+ | allocation_pools | {"start": "2::2", "end": "2::ffff:ffff:ffff:ffff"} | | cidr | 2::/64 | | dns_nameservers | | | enable_dhcp | True | | gateway_ip | 2::0001 >>>ISSUE | | host_routes | | | id | e2a09fa2-52ce-4f68-9619-b635529213d0 | | ip_version | 6 | | ipv6_address_mode | | | ipv6_ra_mode | | | name | sub11 | | network_id | af909be2-5413-40a0-984f-cd58dc7289c9 | | subnetpool_id | | | tenant_id | bc125faaf00041f492eaf10f72a37c2d | +-------------------+----------------------------------------------------+ 3) if update the gateway of ipv6 subnet with ipv6 address which has a leading "0" via dashboard, no this issue 4) For reference, below is the occurence of updating gateway of ipv4 subnet: root@45-59:/opt/stack/devstack# neutron subnet-update --gateway 1.0.0.001 sub1 Invalid input for gateway_ip. Reason: '1.0.0.001' is not an accepted IP address, '1.0.0.1' is recommended. root@45-59:/opt/stack/devstack# [Configration] reproduceable bug, no need [logs] reproduceable bug, no need [Root cause anlyze or debug inf] reproduceable bug [Attachment] None To manage notifications about this bug go to: https://bugs.launchpad.net/neutron/+bug/1531103/+subscriptions -- Mailing list: https://launchpad.net/~yahoo-eng-team Post to : yahoo-eng-team@lists.launchpad.net Unsubscribe : https://launchpad.net/~yahoo-eng-team More help : https://help.launchpad.net/ListHelp