gnocchi event dispacher is not deriving from event dispatch

Bug #1627928 reported by yong sheng gong
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Undecided
Mehdi Abaakouk

Bug Description

gnocchi event dispacher is not working right at now:
1. it is not deriving from base event dispatcher
2. the client method is not right
3. the client API calls are not easy to show

Changed in ceilometer:
assignee: nobody → yong sheng gong (gongysh)
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/377374

Changed in ceilometer:
status: New → In Progress
description: updated
description: updated
Changed in ceilometer:
assignee: yong sheng gong (gongysh) → Mehdi Abaakouk (sileht)
Changed in ceilometer:
assignee: Mehdi Abaakouk (sileht) → yong sheng gong (gongysh)
Changed in ceilometer:
assignee: yong sheng gong (gongysh) → Mehdi Abaakouk (sileht)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (stable/newton)

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/384319

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

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

commit bc073f7592250b13450c566470576d0cbc5a0ec4
Author: gong yong sheng <gong.yongsheng@99cloud.net>
Date: Tue Sep 27 08:50:56 2016 +0000

    make gnocchi event dispatcher work

    The gnocchi event dispatcher have been broken by two major changes
    because of no tests coverage (requests->gnocchiclient and panko
    refactoring)

    To fix that:
    * the event dispatcher class derive the base event dispatcher class.
    * Use correct gnocchi client method to search resources.
    * Use correct gnocchi client method to update resources.
    * Since record_event received a raw notification now and no more the event,
      we change the parsing codetoo.
    * And obviously add a test !

    Change-Id: I94b7393d79e495c5101225b4097c7073978ca7d7
    Co-Authored-By: Mehdi Abaakouk <email address hidden>
    Closes-bug: 1627928

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

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

commit a417a8754cb297a352b73f7f748006c363bf9ee9
Author: gong yong sheng <gong.yongsheng@99cloud.net>
Date: Tue Sep 27 08:50:56 2016 +0000

    make gnocchi event dispatcher work

    The gnocchi event dispatcher have been broken by two major changes
    because of no tests coverage (requests->gnocchiclient and panko
    refactoring)

    To fix that:
    * the event dispatcher class derive the base event dispatcher class.
    * Use correct gnocchi client method to search resources.
    * Use correct gnocchi client method to update resources.
    * Since record_event received a raw notification now and no more the event,
      we change the parsing codetoo.
    * And obviously add a test !

    Change-Id: I94b7393d79e495c5101225b4097c7073978ca7d7
    Co-Authored-By: Mehdi Abaakouk <email address hidden>
    Closes-bug: 1627928

tags: added: in-stable-newton
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/ceilometer 7.0.1

This issue was fixed in the openstack/ceilometer 7.0.1 release.

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

This issue was fixed in the openstack/ceilometer 8.0.0 release.

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.