Skip to content

Commit db85e4f

Browse files
committed
fix(ceilometer): Add volume_type_id to metric meta
Cinder volume notifications emit only payload.volume_type as an id, but the API pollster in Ceilometer gets back a name value from Cinder. Cinder should emit the volume_type_name in its notification payloads but sadly it does not at this time. This change should simply re-route payload volume_type to a new field we will add in gnocchi for volume_type_id, preventing flip-flopping of volume_type on the gnocchi volume_type resources.
1 parent 8ff9102 commit db85e4f

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

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

+3-2
Original file line numberDiff line numberDiff line change
@@ -231,7 +231,7 @@ conf:
231231
size:
232232
type: int
233233
fields: payload.size
234-
type:
234+
volume_type_id:
235235
fields: payload.volume_type
236236
replication_status:
237237
fields: payload.replication_status
@@ -1057,6 +1057,7 @@ conf:
10571057
attributes:
10581058
display_name: resource_metadata.(display_name|name)
10591059
volume_type: resource_metadata.volume_type
1060+
volume_type_id: resource_metadata.volume_type_id
10601061
image_id: resource_metadata.image_id
10611062
instance_id: resource_metadata.instance_id
10621063
event_create:
@@ -1295,7 +1296,7 @@ conf:
12951296
resource_id: $.payload.volume_id
12961297
metadata:
12971298
display_name: $.payload.display_name
1298-
volume_type: $.payload.volume_type
1299+
volume_type_id: $.payload.volume_type
12991300
image_id: $.payload.glance_metadata[?key=image_id].value
13001301
instance_id: $.payload.volume_attachment[0].instance_uuid
13011302

0 commit comments

Comments
 (0)