oslo.config

AttributeError: 'NoneType' object has no attribute '_config_dirs'

Bug #1547612 reported by Davanum Srinivas (DIMS)
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Medium
Lianhao Lu
Murano
Fix Released
High
Unassigned
Murano mitaka-3 "m3"
OpenStack Heat
Fix Released
Undecided
Davanum Srinivas (DIMS)
oslo.config
Won't Fix
Undecided
Davanum Srinivas (DIMS)

Bug Description

http://logs.openstack.org/periodic/periodic-heat-py34-with-oslo-master/0e978ba/console.html#_2016-02-19_06_17_53_133

Looks like we did not cover the case where someone is using find_file without calling CONF() in review:
https://review.openstack.org/#/c/281248/

OpenStack Infra (hudson-openstack)
Changed in oslo.config:
assignee: nobody → Davanum Srinivas (DIMS) (dims-v)
status: New → In Progress
OpenStack Infra (hudson-openstack)
Changed in heat:
assignee: nobody → Davanum Srinivas (DIMS) (dims-v)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on oslo.config (master)

Change abandoned by Davanum Srinivas (dims) (<email address hidden>) on branch: master
Review: https://review.openstack.org/282459

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

Reviewed: https://review.openstack.org/282563
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=803755cfa63690fd3b340f1b2afe6534d24a7d5b
Submitter: Jenkins
Branch: master

commit 803755cfa63690fd3b340f1b2afe6534d24a7d5b
Author: Davanum Srinivas <email address hidden>
Date: Fri Feb 19 18:10:04 2016 -0500

    Use oslo.config fixture to avoid gate break

    With Ibd0566f11df62da031afb128c9687c5e8c7b27ae, we are changing
    the oslo.config internals a bit. The code in Heat test suite
    unnecesssarily depends on oslo.config internals. Just using
    the oslo.config fixture is good enough for these tests. This
    will avoid CI failures next week when we make a new release
    for oslo.config.

    Closes-Bug: #1547612
    Change-Id: Ibf784d08fbad099189507ff4f665788aee7a3a97

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

Fix proposed to branch: stable/liberty
Review: https://review.openstack.org/283659

Revision history for this message
Matt Riedemann (mriedem) wrote :

Also impact ceilometer:

http://logs.openstack.org/periodic/periodic-ceilometer-py34-with-oslo-master/73efcb5/console.html#_2016-02-21_06_11_43_502

Revision history for this message
Matt Riedemann (mriedem) wrote :

And murano:

http://logs.openstack.org/periodic-stable/periodic-murano-python27-liberty/df56335/console.html

gordon chung (chungg)
Changed in ceilometer:
status: New → Triaged
importance: Undecided → Medium
Matt Riedemann (mriedem)
tags: added: liberty-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to heat (stable/liberty)

Reviewed: https://review.openstack.org/283659
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=e2f916c61d02f39dabf7f39ec6ff771979959ed0
Submitter: Jenkins
Branch: stable/liberty

commit e2f916c61d02f39dabf7f39ec6ff771979959ed0
Author: Davanum Srinivas <email address hidden>
Date: Fri Feb 19 18:10:04 2016 -0500

    Use oslo.config fixture to avoid gate break

    With Ibd0566f11df62da031afb128c9687c5e8c7b27ae, we are changing
    the oslo.config internals a bit. The code in Heat test suite
    unnecesssarily depends on oslo.config internals. Just using
    the oslo.config fixture is good enough for these tests. This
    will avoid CI failures next week when we make a new release
    for oslo.config.

    Closes-Bug: #1547612
    Change-Id: Ibf784d08fbad099189507ff4f665788aee7a3a97
    (cherry picked from commit 803755cfa63690fd3b340f1b2afe6534d24a7d5b)

tags: added: in-stable-liberty
Davanum Srinivas (DIMS) (dims-v)
Changed in oslo.config:
status: In Progress → Fix Committed
Revision history for this message
Stan Lagun (slagun) wrote :

For Murano there is already a bug for this: https://bugs.launchpad.net/murano/+bug/1548578

Changed in murano:
status: New → Fix Committed
milestone: none → mitaka-3
importance: Undecided → High
Lianhao Lu (lianhao-lu)
Changed in ceilometer:
assignee: nobody → Lianhao Lu (lianhao-lu)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (master)

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

Changed in ceilometer:
status: Triaged → In Progress
Revision history for this message
Thierry Carrez (ttx) wrote : Fix included in openstack/heat 6.0.0.0b3

This issue was fixed in the openstack/heat 6.0.0.0b3 development milestone.

Kirill Zaitsev (kzaitsev)
Changed in murano:
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (master)

Reviewed: https://review.openstack.org/287636
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=13f6b72f9f96bba436265f9b22cbaac6ebefb9a5
Submitter: Jenkins
Branch: master

commit 13f6b72f9f96bba436265f9b22cbaac6ebefb9a5
Author: Lianhao Lu <email address hidden>
Date: Thu Mar 3 07:40:33 2016 +0000

    Fixed corner cases of incorrect use of oslo.config

    In the tests there are some corner cases where we call oslo.config.cfg
    fild_file before CONF, which is the incorrect use of oslo.config. This
    patch fixes those.

    Change-Id: Idf02156a4d8f8c92aa8f4aaf24ff2c3780b9b3c9
    Closes-Bug: #1547612

Changed in ceilometer:
status: In Progress → Fix Released
Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/ceilometer 6.0.0.0rc1

This issue was fixed in the openstack/ceilometer 6.0.0.0rc1 release candidate.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/heat 5.0.2

This issue was fixed in the openstack/heat 5.0.2 release.

Ken Giusti (kgiusti)
Changed in oslo.config:
status: Fix Committed → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.