Skip to content

Commit 383c492

Browse files
authored
fix(ceilometer): update json path to vm uuid (rackerlabs#435)
The cinder trait for instance_id being parsed out was trying to get the instance id from `payload.volume_attachment[0].server_id` but upon inspection of the raw messages emitted from cinder arriving via RMQ, it's clear that key is actually, `instance_uuid`, not `server_id`. Now, when cinder messages traverse ceilometer, instance uuid is populated as expected. We don't have to wait for the pollster to run. Furthermore, explicitly define event_create and event_update on the gnocchi volume resource to look out for volume create/attach events resepectively, to keep the resouce up-to-date in gnocchi in near real-time.
1 parent c26c0db commit 383c492

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

base-helm-configs/ceilometer/ceilometer-helm-overrides.yaml

+7-2
Original file line numberDiff line numberDiff line change
@@ -216,7 +216,7 @@ conf:
216216
image_id:
217217
fields: payload.glance_metadata[?key=image_id].value
218218
instance_id:
219-
fields: payload.volume_attachment[0].server_id
219+
fields: payload.volume_attachment[0].instance_uuid
220220
- event_type: ['volume.transfer.*', 'volume.exists', 'volume.retype', 'volume.create.*', 'volume.delete.*', 'volume.resize.*', 'volume.attach.*', 'volume.detach.*', 'volume.update.*', 'snapshot.transfer.accept.end']
221221
traits:
222222
<<: *cinder_traits
@@ -1029,15 +1029,20 @@ conf:
10291029
volume_type: resource_metadata.volume_type
10301030
image_id: resource_metadata.image_id
10311031
instance_id: resource_metadata.instance_id
1032+
event_create:
1033+
- volume.create.end
10321034
event_delete:
10331035
- volume.delete.end
10341036
- snapshot.delete.end
10351037
event_update:
1038+
- volume.attach.end
10361039
- volume.transfer.accept.end
10371040
- snapshot.transfer.accept.end
10381041
event_attributes:
10391042
id: resource_id
10401043
project_id: project_id
1044+
image_id: image_id
1045+
instance_id: instance_id
10411046

10421047
- resource_type: volume_provider
10431048
metrics:
@@ -1262,7 +1267,7 @@ conf:
12621267
display_name: $.payload.display_name
12631268
volume_type: $.payload.volume_type
12641269
image_id: $.payload.glance_metadata[?key=image_id].value
1265-
instance_id: $.payload.volume_attachment[0].server_id
1270+
instance_id: $.payload.volume_attachment[0].instance_uuid
12661271

12671272
- name: 'snapshot.size'
12681273
event_type:

0 commit comments

Comments
 (0)