Comment 2 for bug 1759345

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

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

commit a1b49c796e44df35318fc35327b132aae22dce48
Author: Ihar Hrachyshka <email address hidden>
Date: Tue Mar 27 11:32:37 2018 -0700

    Don't define polling_interval option in two places

    The option is currently defined in two places - as a generic agent
    option and as a sriov agent specific one. This is a problem now that
    oslo.config tracks location where options are defined
    (I6dec1e09dcab203c6287f9c56c866f220a42f850) because when unit test
    runner imports all modules, it registers the option from those two
    separate locations, and since locations are different, it's considered
    the options are duplicate, which results in the following error:

    DuplicateOptError: duplicate option: polling_interval

    To fix the issue, I removed the definition of the option from sriov
    module and made sure the common agent options are registered whenever
    the sriov module is imported.

    Change-Id: I485532692f4b3a5804885e0325a8352fe6b1aa65
    Closes-Bug: #1759345