← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1531103] Re: can update the gateway of ipv6 subnet with ipv6 address which has a leading "0" via cli

 

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.banerjee@xxxxxxxxx>
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


References