IPv6 control plane support does not work with the fully routed topology (/128)

Bug #1848941 reported by Radosław Piliszek
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
kolla-ansible
Fix Released
Medium
Radosław Piliszek
Train
Fix Released
Medium
Radosław Piliszek
Ussuri
Fix Released
Medium
Radosław Piliszek
Victoria
Fix Released
Medium
Radosław Piliszek

Bug Description

kolla_address filters out /128 addresses as these are the keepalived's default and there is no more reliable way to figure that from environment (no concept of secondaries). VIP address has to be filtered out as we use the first address (ansible delivers the last assigned as first).
This breaks when keepalived is changed to use a different prefix length but it is very unlikely anyone would do that in this case.
On the other hand, this also breaks deployments when the real address is /128. This may be the case with some (still rare in this use case) routed-only topologies (that omit L2 broadcast domains).

Tags: ipv6
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to kolla-ansible (master)

Fix proposed to branch: master
Review: https://review.opendev.org/729338

Changed in kolla-ansible:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to kolla-ansible (master)

Reviewed: https://review.opendev.org/729338
Committed: https://git.openstack.org/cgit/openstack/kolla-ansible/commit/?id=0674236eedd76ae06f168db0da2b249d3ba756b2
Submitter: Zuul
Branch: master

commit 0674236eedd76ae06f168db0da2b249d3ba756b2
Author: Radosław Piliszek <email address hidden>
Date: Tue May 19 19:49:14 2020 +0200

    Fix kolla_address in IPv6 fully-routed topo case

    This includes some lightweight refactoring to avoid code
    duplication.

    This patch is made to be backportable to Train.
    We now include Ansible in testing since Ussuri so the comments
    about the bool filter are wrong.

    Change-Id: Ia2e0f7f24988763bacfeafefb7977021f5949f4e
    Closes-bug: #1848941

Changed in kolla-ansible:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to kolla-ansible (stable/ussuri)

Fix proposed to branch: stable/ussuri
Review: https://review.opendev.org/744276

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to kolla-ansible (stable/train)

Fix proposed to branch: stable/train
Review: https://review.opendev.org/744277

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to kolla-ansible (stable/train)

Reviewed: https://review.opendev.org/744277
Committed: https://git.openstack.org/cgit/openstack/kolla-ansible/commit/?id=cc28890904792525a722519e0240b5e0f8bd9256
Submitter: Zuul
Branch: stable/train

commit cc28890904792525a722519e0240b5e0f8bd9256
Author: Radosław Piliszek <email address hidden>
Date: Tue May 19 19:49:14 2020 +0200

    Fix kolla_address in IPv6 fully-routed topo case

    This includes some lightweight refactoring to avoid code
    duplication.

    This patch is made to be backportable to Train.
    We now include Ansible in testing since Ussuri so the comments
    about the bool filter are wrong.

    Change-Id: Ia2e0f7f24988763bacfeafefb7977021f5949f4e
    Closes-bug: #1848941
    (cherry picked from commit 0674236eedd76ae06f168db0da2b249d3ba756b2)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to kolla-ansible (stable/ussuri)

Reviewed: https://review.opendev.org/744276
Committed: https://git.openstack.org/cgit/openstack/kolla-ansible/commit/?id=8a770bbdb4757082b81dd6beca618303b5b8a008
Submitter: Zuul
Branch: stable/ussuri

commit 8a770bbdb4757082b81dd6beca618303b5b8a008
Author: Radosław Piliszek <email address hidden>
Date: Tue May 19 19:49:14 2020 +0200

    Fix kolla_address in IPv6 fully-routed topo case

    This includes some lightweight refactoring to avoid code
    duplication.

    This patch is made to be backportable to Train.
    We now include Ansible in testing since Ussuri so the comments
    about the bool filter are wrong.

    Change-Id: Ia2e0f7f24988763bacfeafefb7977021f5949f4e
    Closes-bug: #1848941
    (cherry picked from commit 0674236eedd76ae06f168db0da2b249d3ba756b2)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/kolla-ansible 10.2.0

This issue was fixed in the openstack/kolla-ansible 10.2.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/kolla-ansible 9.3.0

This issue was fixed in the openstack/kolla-ansible 9.3.0 release.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.