OVS: Add support for IPv6 addresses as tunnel endpoints

[OpenStack Infra]

https://review.openstack.org/257335
Dear bug triager. This bug was created since a commit was marked with DOCIMPACT.
Your project "openstack/neutron" is set up so that we directly report the documentation bugs against it. If this needs changing, the docimpact-group option needs to be added for the project. You can ask the OpenStack infra team (#openstack-infra on freenode) for help if you need to.

commit 773394a1887bec6ab4c2ff0308f0e830e9a9089f
Author: Frode Nordahl <email address hidden>
Date: Mon Dec 14 13:51:48 2015 +0100

    OVS: Add support for IPv6 addresses as tunnel endpoints

    Remove IPv4 restriction for local_ip configuration statement.

    Check for IP version mismatch of local_ip and remote_ip before creating
    tunnel.

    Create hash of remote IPv6 address for OVS interface/port name with least
    posibility for collissions.

    Fix existing tests that fail because of the added check for IP version
    and subsequently valid IP addresses in _setup_tunnel_port.

    DocImpact

    Change-Id: I9ec137ef8c688b678a0c61f07e9a01382acbeb13
    Closes-Bug: #1525895

[Brian Haley (brian-haley)]

Not sure if I should be the owner since I did most of the later work on this change.

The change on docs.openstack.org really just need to mention either address family is possible, I didn't actually find any place where it said "IPv4 only". So matching the config.py change I made the setting could be described like this:

local_ip - Local IP address of tunnel endpoint. Can be either an IPv4 or IPv6 address.

[Rohit Agarwalla (rohitagarwalla)]

Is the following fix (diff attached) acceptable for this bug ?

[Brian Haley (brian-haley)]

Thanks Rohit for updating the doc. That change looks good to me for now, when a follow-on [1] matches we can just update it again.

[1] https://review.openstack.org/#/c/320121/14/neutron/plugins/ml2/drivers/openvswitch/agent/common/config.py

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/335147

[OpenStack Infra (hudson-openstack)]

Change abandoned by Rohit Agarwalla (<email address hidden>) on branch: master
Review: https://review.openstack.org/335147
Reason: Updates to these files are done automatically and not manually.

[Rohit Agarwalla (rohitagarwalla)]

@brian-haley do you know which docs need to updated for this actually ?

[Brian Haley (brian-haley)]

Rohit - I'm thinking just what's on docs.openstack.org? Matt might have a better answer.

[Armando Migliaccio (armando-migliaccio)]

The proposed fix (https://review.openstack.org/#/c/352027/) merged.

[Armando Migliaccio (armando-migliaccio)]

Neutron marked invalid as there's no devref to be supplied.