UDP publisher doesn't work

Bug #1259171 reported by Nadya Privalova
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
High
Ilya Tyaptin

Bug Description

UDP and RPC publishers have different message's structures but collector, dispatcher and storages are designed for messages from rpc publisher and we have troubles with collecting messages with udp publisher.

Differences:
1. Udp messages have not message_signature, which is verified in database dispather. So all messages are discarded.

2013-12-09 12:57:27.524 3310 WARNING ceilometer.dispatcher.database [-] message signature invalid, discarding message: {'counter_name': 'image', 'user_id': None, 'name': 'image', 'resource_id': '44581e2b-4d44-43b5-8f75-67bdd9979aed', 'timestamp': '2013-12-09T12:57:27Z', 'id': '74cb65e8-60d1-11e3-9f77-fa163eebc8c9', 'volume': 1, 'source': 'openstack', 'counter_unit': 'image', 'counter_volume': 1, 'project_id': 'c0f3f25c35b745959df48403e46ffe83', 'type': 'gauge', 'resource_metadata': {'status': 'active', 'name': 'cirros-0.3.1-x86_64-uec-kernel', 'deleted': False, 'container_format': 'aki', 'created_at': '2013-12-06T13:32:30', 'disk_format': 'aki', 'updated_at': '2013-12-06T13:32:30', 'properties': {}, 'protected': False, 'checksum': 'c352f4e7121c6eae958bc1570324f17e', 'min_disk': 0, 'is_public': True, 'deleted_at': None, 'min_ram': 0, 'size': 4955792}, 'unit': 'image', 'counter_type': 'gauge'}

2. Each message is supposed to be modified before dispatcher starts to work:
   2.1 add 'counter_' to specified key names,
   2.2 id is changed to message_id
   In case of UDP 2.1 take place in collector (vs publisher/rpc.py file for RPC) and 2.2 is not the truth for UDP messages at all so this results with error from storage

description: updated
Julien Danjou (jdanjou)
Changed in ceilometer:
importance: Undecided → High
status: New → Triaged
milestone: none → icehouse-2
assignee: nobody → Julien Danjou (jdanjou)
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/62656

Changed in ceilometer:
assignee: Julien Danjou (jdanjou) → Ilya Tyaptin (ityaptin)
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (master)

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

commit cdab1401dc9cc6fdf7d079c1d139ad915a7d47da
Author: Ilya Tyaptin <email address hidden>
Date: Tue Dec 17 17:55:39 2013 +0400

    Fix work of udp publisher

     * Signature is appended also to udp messages
     * All methods for convert from sample to message are moved to publisher/utils.py
     * In configs metering_secret is moved from group publisher_rpc to publisher
     * Changed tests for udp publisher

    Fixes: bug #1259171

    Change-Id: Ic7727c69d4be6d17ff00f21e323341efd57122a7

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: icehouse-2 → 2014.1
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.