Conf options of network agent plugin cloud be ignored

Bug #1498048 reported by Mehdi Abaakouk
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Low
Mehdi Abaakouk

Bug Description

Hi,

Configuration options of network agent plugin could be ignored.

This is because the neutron client is not loaded at runtime but before the ceilometer service initialize oslo.conf object:

https://github.com/openstack/ceilometer/blob/master/ceilometer/network/services/base.py#L35

By chance, the bug is hidden because this file is loaded by stevedore after we run prepare_service().

Cheers,

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/225888

Changed in ceilometer:
assignee: nobody → Mehdi Abaakouk (sileht)
status: New → In Progress
Changed in ceilometer:
assignee: Mehdi Abaakouk (sileht) → Rohit Jaiswal (rohit-jaiswal-3)
gordon chung (chungg)
Changed in ceilometer:
assignee: Rohit Jaiswal (rohit-jaiswal-3) → Mehdi Abaakouk (sileht)
gordon chung (chungg)
Changed in ceilometer:
milestone: none → liberty-rc1
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (master)

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

commit d60a368e66be636318c3fa95f79dbc745c5401fd
Author: Mehdi Abaakouk <email address hidden>
Date: Mon Sep 21 16:40:12 2015 +0200

    Don't create neutron client at loadtime

    This change creates neutron client at runtime,
    to ensure oslo.config object is fully loaded.

    This also remove some unused code.

    Closes-bug: #1498048

    Change-Id: Ic3028daf74d5472e6d6a2a82fddc272a5e6beb31

Changed in ceilometer:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in ceilometer:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: liberty-rc1 → 5.0.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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