cad70e20cf
The credentials_factory module loads a few bit configuration at module import time when the DEFAULT_PARAMS variable is defined. This is not a good idea as it forces configuration to be loaded even during test discovery. Besides DEFAULT_PARAMS was out of date as it did not include http_timeout. Replacing the DEFAULT_PARAMS variable with calls to config.service_client_config(). Not loading CONF at test discovery time uncovered the fact that the account generator unit tests were depending on oslo log configuration parameters to be there. Mocking the log setup step fixed the issue. Change-Id: I6e0eb85b3749baedb6035f59ed1c66850f6c95fb
14 lines
540 B
YAML
14 lines
540 B
YAML
---
|
|
upgrade:
|
|
- |
|
|
Replace any call in your code to credentials_factory.DEFAULT_PARAMS with
|
|
a call to config.service_client_config().
|
|
fixes:
|
|
- |
|
|
The credentials_factory module used to load configuration at import time
|
|
which caused configuration being loaded at test discovery time.
|
|
This was fixed by removing the DEFAULT_PARAMS variable. This variable
|
|
was redundant (and outdated), the same dictionary (but up to date) can
|
|
be obtained via invoking config.service_client_config() with no service
|
|
parameter.
|