polling agent doesn't fail if hypervisor not found

Bug #1635042 reported by gordon chung
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Ceilometer
Invalid
Low
Unassigned

Bug Description

this is all over the place. this isn't failing anything though.

2016-10-19 07:49:41.134999 | 2016-10-19 07:45:00.869 5307 ERROR ceilometer.agent.manager [-] Skip loading extension for network.outgoing.packets
2016-10-19 07:49:41.155405 | 2016-10-19 07:45:00.869 5307 ERROR ceilometer.agent.manager Traceback (most recent call last):
2016-10-19 07:49:41.215836 | 2016-10-19 07:45:00.869 5307 ERROR ceilometer.agent.manager File "/home/jenkins/workspace/gate-ceilometer-tox-db-py27-mysql-ubuntu-xenial/.tox/py27-mysql/local/lib/python2.7/site-packages/stevedore/extension.py", line 163, in _load_plugins
2016-10-19 07:49:41.230924 | 2016-10-19 07:45:00.869 5307 ERROR ceilometer.agent.manager verify_requirements,
2016-10-19 07:49:41.377609 | 2016-10-19 07:45:00.869 5307 ERROR ceilometer.agent.manager File "/home/jenkins/workspace/gate-ceilometer-tox-db-py27-mysql-ubuntu-xenial/.tox/py27-mysql/local/lib/python2.7/site-packages/stevedore/extension.py", line 195, in _load_one_plugin
2016-10-19 07:49:41.438351 | 2016-10-19 07:45:00.869 5307 ERROR ceilometer.agent.manager obj = plugin(*invoke_args, **invoke_kwds)
2016-10-19 07:49:41.526092 | 2016-10-19 07:45:00.869 5307 ERROR ceilometer.agent.manager File "/home/jenkins/workspace/gate-ceilometer-tox-db-py27-mysql-ubuntu-xenial/.tox/py27-mysql/local/lib/python2.7/site-packages/ceilometer/agent/plugin_base.py", line 183, in __init__
2016-10-19 07:49:41.543770 | 2016-10-19 07:45:00.869 5307 ERROR ceilometer.agent.manager raise ExtensionLoadError(err)
2016-10-19 07:49:41.564777 | 2016-10-19 07:45:00.869 5307 ERROR ceilometer.agent.manager ExtensionLoadError: No module named libvirt

gordon chung (chungg)
summary: - libvirt errors
+ non-fatal libvirt errors in functional tests
Revision history for this message
gordon chung (chungg) wrote : Re: non-fatal libvirt errors in functional tests
Changed in ceilometer:
status: Triaged → Invalid
Revision history for this message
gordon chung (chungg) wrote :

related to fact polling service won't exit (expected) so it will time out and output a bunch of caught failures.

that said, i think if we get no module named libvirt, we should just fail rather than error

Changed in ceilometer:
status: Invalid → In Progress
status: In Progress → Triaged
importance: Undecided → Low
summary: - non-fatal libvirt errors in functional tests
+ polling agent doesn't fail if hypervisor not found
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to ceilometer (master)

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

commit 1695b4774e69e032151a47e0435b7af72826e5b6
Author: gord chung <email address hidden>
Date: Wed Oct 19 20:44:51 2016 +0000

    fix gate

    something changed and expected logs are in stdout rather than stderr
    even though it does makes sense as stdout. i have no idea why it
    was writing to stderr earlier when it shouldn't have.

    Related-Bug: #1635042
    Change-Id: I9feed1a5050a4d99cb1c0130c13e86ceeda47563

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to ceilometer (stable/newton)

Related fix proposed to branch: stable/newton
Review: https://review.openstack.org/395505

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Related fix proposed to branch: stable/newton
Review: https://review.openstack.org/395506

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on ceilometer (stable/newton)

Change abandoned by Julien Danjou (<email address hidden>) on branch: stable/newton
Review: https://review.openstack.org/395505
Reason: superseded by https://review.openstack.org/#/c/395506/

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to ceilometer (stable/newton)

Reviewed: https://review.openstack.org/395506
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=34b0aff8606a40f557df92c567278c517b0b1245
Submitter: Jenkins
Branch: stable/newton

commit 34b0aff8606a40f557df92c567278c517b0b1245
Author: Ken'ichi Ohmichi <email address hidden>
Date: Fri Oct 21 14:14:25 2016 -0700

    Fix gate problem related to _error_checker() and oslo.log change

    Since I84e3be748af10b158037866e1ee4c1375b2c3541, the internal method
    _error_checker() is changed and the ceilometer gate is broken.
    This patch fixes it.

    (cherry picked from commit e6ff08801ca34536680b43ca896f0dfdf66c761f)

    something changed and expected logs are in stdout rather than stderr
    even though it does makes sense as stdout. i have no idea why it
    was writing to stderr earlier when it shouldn't have.

    Related-Bug: #1635042
    (cherry picked from commit 1695b4774e69e032151a47e0435b7af72826e5b6)

    Change-Id: I81fe2e60a6fb65e95f477a68267979b8542fa6dc

tags: added: in-stable-newton
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to ceilometer (stable/mitaka)

Related fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/410980

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to ceilometer (stable/mitaka)

Reviewed: https://review.openstack.org/410980
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=25a5500c7dcf4d6d7b32d97f99def4f6844f2c37
Submitter: Jenkins
Branch: stable/mitaka

commit 25a5500c7dcf4d6d7b32d97f99def4f6844f2c37
Author: gord chung <email address hidden>
Date: Wed Dec 14 22:26:50 2016 +0000

    fix mitaka gate grab bag

    Since I84e3be748af10b158037866e1ee4c1375b2c3541, the internal method
    _error_checker() is changed and the ceilometer gate is broken.
    This patch fixes it.

    (cherry picked from commit e6ff08801ca34536680b43ca896f0dfdf66c761f)

    something changed and expected logs are in stdout rather than stderr
    even though it does makes sense as stdout. i have no idea why it
    was writing to stderr earlier when it shouldn't have.

    Related-Bug: #1635042
    (cherry picked from commit 1695b4774e69e032151a47e0435b7af72826e5b6)

    The API changed and the Ceilometer driver does not work anymore with it.

    Related-Bug: #1613419
    (cherry picked from commit 2b6fb87180dbedcd0a60323b587ecd8eec535d7b)

    Change-Id: Ifaa1e02f5bc30725124d581c9127713260f755dc

tags: added: in-stable-mitaka
Revision history for this message
gordon chung (chungg) wrote :

marking invalid because i don't see this ever anymore and i don't really know what i was writing about originally.

Changed in ceilometer:
status: Triaged → Invalid
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.