← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1866129] Re: dnsmasq_version_supported() sometimes throws an exception

 

Reviewed:  https://review.opendev.org/711353
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=cc281afeecc58e48f0d5bf3a5d6588e96e1fd33f
Submitter: Zuul
Branch:    master

commit cc281afeecc58e48f0d5bf3a5d6588e96e1fd33f
Author: Brian Haley <bhaley@xxxxxxxxxx>
Date:   Wed Mar 4 17:30:15 2020 -0500

    Register DNSMASQ_OPTS in functional sanity tests
    
    test_dnsmasq_version() is sometimes triggering a traceback
    because the new dnsmasq_enable_addr6_list config option isn't
    being found. Register the option in the test class.
    
    Change-Id: Ie441f831bcd0835ae8e1cd082005640b65b7393a
    Closes-bug: #1866129


** 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/1866129

Title:
  dnsmasq_version_supported() sometimes throws an exception

Status in neutron:
  Fix Released

Bug description:
  I saw this traceback recently in
  https://review.opendev.org/#/c/710460/ but it seems related to the
  recent change https://review.opendev.org/#/c/704436/ (DHCPv6 - Use
  addr6_list in dnsmasq)

   Traceback (most recent call last):
        File "/home/zuul/src/opendev.org/openstack/neutron/neutron/tests/base.py", line 182, in func
          return f(self, *args, **kwargs)
        File "/home/zuul/src/opendev.org/openstack/neutron/neutron/tests/functional/sanity/test_sanity.py", line 32, in test_dnsmasq_version
          checks.dnsmasq_version_supported()
        File "/home/zuul/src/opendev.org/openstack/neutron/neutron/cmd/sanity/checks.py", line 238, in dnsmasq_version_supported
          if (cfg.CONF.dnsmasq_enable_addr6_list is True and
        File "/home/zuul/src/opendev.org/openstack/neutron/.tox/dsvm-functional/lib/python3.6/site-packages/oslo_config/cfg.py", line 2209, in __getattr__
          raise NoSuchOptError(name)
      oslo_config.cfg.NoSuchOptError: no such option dnsmasq_enable_addr6_list in group [DEFAULT]

  I think the SanityTestCase class needs to register the DNSMASQ_OPTS.

  Don't know why it's not completely breaking the gate but think it's an
  easy fix.

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1866129/+subscriptions


References