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:
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.
Reviewed: https:/ /review. openstack. org/557003 /git.openstack. org/cgit/ openstack/ neutron/ commit/ ?id=a1b49c796e4 4df35318fc35327 b132aae22dce48
Committed: https:/
Submitter: Zuul
Branch: master
commit a1b49c796e44df3 5318fc35327b132 aae22dce48
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 cab203c6287f9c5 6c866f220a42f85 0) because when unit test
option and as a sriov agent specific one. This is a problem now that
oslo.config tracks location where options are defined
(I6dec1e09d
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:
DuplicateOp tError: 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: I485532692f4b3a 5804885e0325a83 52fe6b1aa65
Closes-Bug: #1759345