pollster/api sample notifications should have dictionary payload
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ceilometer |
Fix Released
|
Undecided
|
gordon chung |
Bug Description
the rough format of api/pollsters notificatoins is as follows:
NOTIFICATION = {
u'_
u'_
'event_type': u'sample.create',
'timestamp': u'2015-06-1909: 19: 35.786893',
u'_
u'_
'payload': [{
},
{
}],
u'_
u'_
u'_
u'_
'priority': 'info',
u'_
u'_
u'_
u'_
'publisher_id': u'ceilometer.api',
'message_id': u'939823de-
}
since payload is a list, it does not make it very extensible. for us to send anything else, we need to create an entire different event.
Changed in ceilometer: | |
milestone: | none → liberty-3 |
status: | Fix Committed → Fix Released |
Changed in ceilometer: | |
milestone: | liberty-3 → 5.0.0 |
another reason is that by default, metadata is the taken as full payload... and it's expected to be dict. we can change that behaviour but i think it should probably change here too.