Instances won't obtain additional configuration options from DHCP when using stateless DHCPv6 subnets

Bug #1397022 reported by Ihar Hrachyshka
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Medium
Ihar Hrachyshka
Juno
Fix Released
Undecided
Unassigned

Bug Description

When additional DHCP configuration is available from DHCP server, radvd should set Other flag in RAs. It's not done by L3 agent, so clients are left unaware about additional configuration and don't spawn dhcp clients to receive those. This results in e.g. DNS nameservers set for a subnet not propagated into instance's /etc/resolv.conf, among other things.

Changed in neutron:
assignee: nobody → Ihar Hrachyshka (ihar-hrachyshka)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

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

Changed in neutron:
status: New → In Progress
tags: added: ipv6 juno-backport-potential
Kyle Mestery (mestery)
Changed in neutron:
milestone: none → kilo-2
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/137656
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=4f3a9135acb561172f7af45bc82739d6dc49f23c
Submitter: Jenkins
Branch: master

commit 4f3a9135acb561172f7af45bc82739d6dc49f23c
Author: Ihar Hrachyshka <email address hidden>
Date: Thu Nov 27 15:43:48 2014 +0100

    ipv6: set OtherConfig flag for DHCPv6 stateless subnets

    In case of DHCPv6 stateless subnets, we should inform DHCP clients about
    other configuration values available from DHCP server. This is done by
    setting O (other) flag in RAs, which is controlled by AdvOtherConfigFlag
    setting in radvd case.

    Since radvd configuration file becomes quite complex, migrated its
    generation to Jinja2.

    Added a basic unit test that checks that flag is set for stateless mode
    and not SLAAC. For stateful, it doesn't really matter whether other flag
    is set, so no need to expect any value of it.

    No more unit tests seem to be needed: conditional prefix generation is
    already covered in test_l3_agent, and other statements are common for
    all ipv6_ra_modes.

    Change-Id: I1ddad3e1f5efce2b6da4ec00b9294e08fe1e85dd
    Closes-Bug: #1397022

Changed in neutron:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/juno)

Fix proposed to branch: stable/juno
Review: https://review.openstack.org/143461

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/juno)

Reviewed: https://review.openstack.org/143461
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=327e32519d3314f3bc9709a5901aff375fd8c011
Submitter: Jenkins
Branch: stable/juno

commit 327e32519d3314f3bc9709a5901aff375fd8c011
Author: Ihar Hrachyshka <email address hidden>
Date: Thu Nov 27 15:43:48 2014 +0100

    ipv6: set OtherConfig flag for DHCPv6 stateless subnets

    In case of DHCPv6 stateless subnets, we should inform DHCP clients about
    other configuration values available from DHCP server. This is done by
    setting O (other) flag in RAs, which is controlled by AdvOtherConfigFlag
    setting in radvd case.

    Since radvd configuration file becomes quite complex, migrated its
    generation to Jinja2.

    Added a basic unit test that checks that flag is set for stateless mode
    and not SLAAC. For stateful, it doesn't really matter whether other flag
    is set, so no need to expect any value of it.

    No more unit tests seem to be needed: conditional prefix generation is
    already covered in test_l3_agent, and other statements are common for
    all ipv6_ra_modes.

    Conflicts:
     neutron/tests/unit/test_l3_agent.py

    Change-Id: I1ddad3e1f5efce2b6da4ec00b9294e08fe1e85dd
    Closes-Bug: #1397022
    (cherry picked from commit 4f3a9135acb561172f7af45bc82739d6dc49f23c)

tags: added: in-stable-juno
Thierry Carrez (ttx)
Changed in neutron:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: kilo-2 → 2015.1.0
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.