)]}'
{"id":"openstack%2Fceilometer~940758","triplet_id":"openstack%2Fceilometer~master~I97d922c808a316e0018a8ee97490461299c44e3b","project":"openstack/ceilometer","branch":"master","topic":"volume-type-name-notification","attention_set":{"9816":{"account":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"last_update":"2025-02-05 20:14:44.000000000","reason":"Vote got outdated and was removed: Code-Review-1"},"36393":{"account":{"_account_id":36393,"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","username":"Callum027","status":"Catalyst Cloud"},"last_update":"2025-09-09 07:28:19.000000000","reason":"\u003cGERRIT_ACCOUNT_32968\u003e replied on the change","reason_account":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"}}},"removed_from_attention_set":{"32968":{"account":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"last_update":"2025-09-09 07:28:19.000000000","reason":"\u003cGERRIT_ACCOUNT_32968\u003e replied on the change","reason_account":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"}}},"hashtags":[],"change_id":"I97d922c808a316e0018a8ee97490461299c44e3b","subject":"Use volume type name in volume.size notifications (if available)","status":"NEW","created":"2025-02-05 02:26:30.000000000","updated":"2025-09-09 07:28:19.000000000","submit_type":"MERGE_IF_NECESSARY","mergeable":true,"submittable":false,"total_comment_count":10,"unresolved_comment_count":2,"has_review_started":true,"meta_rev_id":"ed7868aeb37408c011359d98e288bedb78cfe899","_number":940758,"virtual_id_number":940758,"owner":{"_account_id":36393,"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","username":"Callum027","status":"Catalyst Cloud"},"actions":{},"labels":{"Verified":{"rejected":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},{"tag":"autogenerated:zuul:gate","value":-2,"date":"2025-03-17 16:50:36.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"recommended":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"all":[{"value":1,"date":"2025-03-17 16:51:59.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},{"value":2,"date":"2025-02-13 09:05:25.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":1,"default_value":0,"optional":true},"Workflow":{"all":[{"value":0,"date":"2025-09-09 07:28:19.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2025-02-05 03:40:27.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2025-02-05 08:39:44.000000000","updated_by":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"reviewer":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"state":"REVIEWER"},{"updated":"2025-02-05 09:02:27.000000000","updated_by":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"reviewer":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"state":"REVIEWER"},{"updated":"2025-02-13 09:05:25.000000000","updated_by":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"reviewer":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"state":"REVIEWER"}],"messages":[{"id":"ad355d2791710d6a1305f3a2f2ddacbd3d1cbce9","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":36393,"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","username":"Callum027","status":"Catalyst Cloud"},"date":"2025-02-05 02:26:30.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"cdbeafb07dbf40e2a9154beefd735b66dfb6edab","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-02-05 03:40:27.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/8696f16f45d7411ebc19211cde558ff8\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/7bbf04c5a36e4bada093df029f85a428 : SUCCESS in 4m 38s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/b3f326b3be074c4aab08aab515dfe358 : SUCCESS in 5m 17s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/1a0c72e5e6b34aebb8db05a68060f5bc : SUCCESS in 2m 59s\n- openstack-tox-py312 https://zuul.opendev.org/t/openstack/build/775155180ae4490e853d61c1104be80e : SUCCESS in 3m 59s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/da4c741ffbdd4fd185bc716685c37b83 : SUCCESS in 6m 11s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/fd57ef0b99204954addf19c48e7116d2 : SUCCESS in 4m 43s\n- grenade-ceilometer https://zuul.opendev.org/t/openstack/build/a8e33c63e74e49cea3a1c4ca384e733f : FAILURE in 21m 47s (non-voting)\n- telemetry-dsvm-integration-centos-9s https://zuul.opendev.org/t/openstack/build/4c0ffb2c84254158baefca21af6d17ec : SUCCESS in 55m 35s (non-voting)\n- telemetry-dsvm-integration-centos-9s-fips https://zuul.opendev.org/t/openstack/build/f442c66a65b74e9fab969b82daaf42a7 : SUCCESS in 1h 00m 30s (non-voting)\n- telemetry-dsvm-integration https://zuul.opendev.org/t/openstack/build/d9d4b80f8b374c178cae522fdf95857e : SUCCESS in 43m 47s","accounts_in_message":[],"_revision_number":1},{"id":"0e640cb7e2abd999ce9faeb689ab3053690cccd7","author":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"date":"2025-02-05 08:39:44.000000000","message":"Patch Set 1: Code-Review+1\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"f5c9e0a102be94eda746d309d19c0dcb4c3618a1","author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"date":"2025-02-05 09:02:27.000000000","message":"Patch Set 1: Code-Review-1\n\n(2 comments)","accounts_in_message":[],"_revision_number":1},{"id":"6651294c661ff19baa464a035439018698b83076","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":36393,"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","username":"Callum027","status":"Catalyst Cloud"},"date":"2025-02-05 20:14:44.000000000","message":"Uploaded patch set 2.\n\nOutdated Votes:\n* Code-Review+1, Code-Review-1 (copy condition: \"changekind:TRIVIAL_REBASE OR is:MIN\")\n* Verified+1\n","accounts_in_message":[],"_revision_number":2},{"id":"c9e6864359084c9f80f5aeaf1bf13dbf43a68cf0","author":{"_account_id":36393,"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","username":"Callum027","status":"Catalyst Cloud"},"date":"2025-02-05 20:16:41.000000000","message":"Patch Set 2:\n\n(2 comments)","accounts_in_message":[],"_revision_number":2},{"id":"48c1920cc7e87a485bf5753537c2daf88287b9b6","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-02-05 21:25:28.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/9362431479d64a4191ce614b80166dcf\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/4e440b0a1ec8420182c466dd5c82a098 : SUCCESS in 5m 15s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/32709efd1199459fa6c004b6a47a3408 : SUCCESS in 3m 15s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/ebb3ccd8375a4617a55dab1d85bd3be1 : SUCCESS in 3m 38s\n- openstack-tox-py312 https://zuul.opendev.org/t/openstack/build/b16fed3977804fa1a609a94a62fe2c49 : SUCCESS in 4m 00s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/6e2d1c173e6a4a97af8878a397e001cc : SUCCESS in 5m 51s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/90b533b0de304b6abb4dedd103393f52 : SUCCESS in 4m 49s\n- grenade-ceilometer https://zuul.opendev.org/t/openstack/build/0e88a9732d0d4860be3f2ce0f7ce4597 : FAILURE in 16m 07s (non-voting)\n- telemetry-dsvm-integration-centos-9s https://zuul.opendev.org/t/openstack/build/3185951808e84ce299e7bb1e7bba7ef6 : SUCCESS in 1h 03m 06s (non-voting)\n- telemetry-dsvm-integration-centos-9s-fips https://zuul.opendev.org/t/openstack/build/74ec9622a9c640729b62b4f84433a020 : SUCCESS in 1h 04m 46s (non-voting)\n- telemetry-dsvm-integration https://zuul.opendev.org/t/openstack/build/f743fff3439442c1a913d57e0ceaa2ce : SUCCESS in 48m 01s","accounts_in_message":[],"_revision_number":2},{"id":"4338aac6488e8c7f2fdb130d8cc466f2d54aa957","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":36393,"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","username":"Callum027","status":"Catalyst Cloud"},"date":"2025-02-06 11:31:20.000000000","message":"Uploaded patch set 3.\n\nOutdated Votes:\n* Verified+1\n","accounts_in_message":[],"_revision_number":3},{"id":"369eaa8e4fbd26998c4a1ed30713a2ec5eb45227","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-02-06 12:26:08.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/72a6b3ff0474496fbe1381a762eba61d\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/69d79dbb79ae4251b4d39bf45a4a8712 : SUCCESS in 4m 58s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/ba23f93c2d8449fa83c0a5e3632692cd : SUCCESS in 4m 47s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/2d7fd6e3d7dc40c096d880a28bc125a2 : SUCCESS in 7m 58s\n- openstack-tox-py312 https://zuul.opendev.org/t/openstack/build/62e1b35066f545fe885769dd3a2feeaa : SUCCESS in 5m 34s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/55557dd445954e67bcca940e143c4b55 : SUCCESS in 3m 56s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/bc1ccacc31294833af918820cee8eec1 : SUCCESS in 5m 22s\n- grenade-ceilometer https://zuul.opendev.org/t/openstack/build/bfe45c38eec140b8a4b5089b101d3ed9 : FAILURE in 10m 51s (non-voting)\n- telemetry-dsvm-integration-centos-9s https://zuul.opendev.org/t/openstack/build/c15291b3617d4a6ea68d7a08585dcbde : SUCCESS in 54m 05s (non-voting)\n- telemetry-dsvm-integration-centos-9s-fips https://zuul.opendev.org/t/openstack/build/1395539a5f64410d9b0346f4d5d63666 : SUCCESS in 39m 03s (non-voting)\n- telemetry-dsvm-integration https://zuul.opendev.org/t/openstack/build/959f5ce5a425411888521a256286bd8a : SUCCESS in 43m 16s","accounts_in_message":[],"_revision_number":3},{"id":"a6cfc2f177d41922073a1160ae532b81708a460d","author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"date":"2025-02-13 09:05:25.000000000","message":"Patch Set 3: Code-Review+2","accounts_in_message":[],"_revision_number":3},{"id":"b455bbd7fb20cb932597d90336afe0f3d92b9ceb","author":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"date":"2025-03-17 16:50:25.000000000","message":"Patch Set 3: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":3},{"id":"4d8388f5913c716776184fd5737da9b045acf82f","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-03-17 16:50:36.000000000","message":"Patch Set 3: Verified-2\n\nBuild failed (gate pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\nhttps://zuul.opendev.org/t/openstack/buildset/9a6a456eb96842e7b2c9bc1c0d07ef49\n\n\nWarning:\n  Change 940674 in project openstack/cinder does not share a change queue with 940758 in project openstack/ceilometer","accounts_in_message":[],"_revision_number":3},{"id":"8ce4f699cf74a64372b464e78d972155e1f30b2b","author":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"date":"2025-03-17 16:51:59.000000000","message":"Patch Set 3: -Workflow Code-Review+1\n\n(1 comment)","accounts_in_message":[],"_revision_number":3},{"id":"943af760f77fcddeac28b6394c8e3f44dfbd2893","author":{"_account_id":36393,"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","username":"Callum027","status":"Catalyst Cloud"},"date":"2025-03-17 18:33:35.000000000","message":"Patch Set 3:\n\n(1 comment)","accounts_in_message":[],"_revision_number":3},{"id":"2c0b9674dcb8209368b282bde7a400974473265e","author":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"date":"2025-09-09 07:24:51.000000000","message":"Patch Set 3:\n\n(1 comment)","accounts_in_message":[],"_revision_number":3},{"id":"d1103766ac99c48ced1ec56c67f2d1f65b7d61ab","author":{"_account_id":36393,"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","username":"Callum027","status":"Catalyst Cloud"},"date":"2025-09-09 07:27:25.000000000","message":"Patch Set 3:\n\n(1 comment)","accounts_in_message":[],"_revision_number":3},{"id":"ed7868aeb37408c011359d98e288bedb78cfe899","author":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"date":"2025-09-09 07:28:19.000000000","message":"Patch Set 3:\n\n(1 comment)","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"ab04317a4e91d9eb2e0ad516904f6f4a337a15d5","revisions":{"f39156be9e0ceeac42cf34350bf43067c3a9cb29":{"kind":"REWORK","_number":1,"created":"2025-02-05 02:26:30.000000000","uploader":{"_account_id":36393,"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","username":"Callum027","status":"Catalyst Cloud"},"ref":"refs/changes/58/940758/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/58/940758/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/58/940758/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/58/940758/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/58/940758/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/58/940758/1"}}},"commit":{"parents":[{"commit":"9bb2891586987162e56229f6290da5e742fb13a1","subject":"Merge \"Fix outdated supported database backends\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/9bb2891586987162e56229f6290da5e742fb13a1"}]}],"author":{"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","date":"2025-02-05 02:26:14.000000000","tz":780},"committer":{"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","date":"2025-02-05 02:26:14.000000000","tz":780},"subject":"Use volume type name in volume.size notifications (if available)","message":"Use volume type name in volume.size notifications (if available)\n\nThere is a mismatch between the values returned for the\n\u0027volume_type\u0027 attribute between the Cinder volume notifications\nand what is returned when polling the Cinder API.\n\nThe Cinder APIs return the name of the volume type as \u0027volume_type\u0027,\nwhile Cinder notifications return the ID of the volume type instead.\nFor more information see this bug [1].\n\nIn Ceilometer this results in an inconsistency where for the\nvolume.size meter, polled samples set \u0027volume_type\u0027 to the name,\nwhile notification samples set it to the ID. We only want to store\none of those with that name, and make the other available under\nanother name.\n\nThere is a proposed Cinder change [2] to add a \u0027volume_type_name\u0027\nattribute to Cinder notifications.\n\nThis allows us to change the notification meter definition to\nsupply the volume type name to the \u0027volume_type\u0027 Gnocchi attribute.\nThe volume type ID is now available under the \u0027volume_type_id\u0027\nmetadata attribute, to match the metadata schema of the volume.size\npollster.\n\nA \u0027volume_type_id\u0027 attribute has been added to the \u0027volume\u0027\nGnocchi resource type to expose the ID via Gnocchi as well.\n\nBackwards compatibility with older Cinder versions is provided\nby using the notification \u0027volume_type\u0027 value to supply the\n\u0027volume_type\u0027 Gnocchi value if \u0027volume_type_name\u0027 is unavailable.\nIn this case, Ceilometer will replicate the current behaviour.\n\nFor this change to work, the relevant Cinder change [2]\nmust be approved and merged.\n\n[1]: https://bugs.launchpad.net/cinder/+bug/2097328\n[2]: https://review.opendev.org/c/openstack/cinder/+/940674\n\nChange-Id: I97d922c808a316e0018a8ee97490461299c44e3b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/f39156be9e0ceeac42cf34350bf43067c3a9cb29"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/f39156be9e0ceeac42cf34350bf43067c3a9cb29"}]},"branch":"refs/heads/master"},"5c4dea64f3b7b55df8d84a76eca0a6ac811a0d12":{"kind":"REWORK","_number":2,"created":"2025-02-05 20:14:44.000000000","uploader":{"_account_id":36393,"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","username":"Callum027","status":"Catalyst Cloud"},"ref":"refs/changes/58/940758/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/58/940758/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/58/940758/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/58/940758/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/58/940758/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/58/940758/2"}}},"commit":{"parents":[{"commit":"38c2144e279341852affd41705962d64c6c5ea30","subject":"Merge \"Replace oslo_utils.encodeutils.exception_to_unicode\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/38c2144e279341852affd41705962d64c6c5ea30"}]}],"author":{"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","date":"2025-02-05 02:26:14.000000000","tz":780},"committer":{"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","date":"2025-02-05 20:14:19.000000000","tz":780},"subject":"Use volume type name in volume.size notifications (if available)","message":"Use volume type name in volume.size notifications (if available)\n\nThere is a mismatch between the values returned for the\nvolume_type attribute between the Cinder volume notifications\nand what is returned when polling the Cinder API.\n\nThe Cinder APIs return the name of the volume type as volume_type,\nwhile Cinder notifications return the ID of the volume type instead.\nFor more information see this bug [1].\n\nIn Ceilometer this results in an inconsistency where for the\nvolume.size meter, polled samples set volume_type to the name,\nwhile notification samples set it to the ID. We only want to store\none of those with that name, and make the other available under\nanother name.\n\nThere is a proposed Cinder change [2] to add a\nvolume_type_name attribute to Cinder notifications.\n\nThis allows us to change the notification meter definition to\nsupply the volume type name to the volume_type Gnocchi attribute.\nThe volume type ID is now available under the volume_type_id\nmetadata attribute, to match the metadata schema of the volume.size\npollster.\n\nBackwards compatibility with older Cinder versions is provided\nby using the notification volume_type value to supply the\nvolume_type Gnocchi value if volume_type_name is unavailable.\nIn this case, Ceilometer will replicate the current behaviour.\n\nFor this change to work, the relevant Cinder change [2]\nmust be approved and merged.\n\n[1]: https://bugs.launchpad.net/cinder/+bug/2097328\n[2]: https://review.opendev.org/c/openstack/cinder/+/940674\n\nDepends-On: I71e7fca4a6d303462fa6d0ed69245b79cb28d646\nChange-Id: I97d922c808a316e0018a8ee97490461299c44e3b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/5c4dea64f3b7b55df8d84a76eca0a6ac811a0d12"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/5c4dea64f3b7b55df8d84a76eca0a6ac811a0d12"}]},"branch":"refs/heads/master"},"ab04317a4e91d9eb2e0ad516904f6f4a337a15d5":{"kind":"REWORK","_number":3,"created":"2025-02-06 11:31:20.000000000","uploader":{"_account_id":36393,"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","username":"Callum027","status":"Catalyst Cloud"},"ref":"refs/changes/58/940758/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/58/940758/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/58/940758/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/58/940758/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/58/940758/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/58/940758/3"}}},"commit":{"parents":[{"commit":"ce3ab93cb88ccc751d2efbb5681e8f4c4704539f","subject":"Add the volume_type_id attribute for volumes","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/ce3ab93cb88ccc751d2efbb5681e8f4c4704539f"}]}],"author":{"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","date":"2025-02-05 02:26:14.000000000","tz":780},"committer":{"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","date":"2025-02-06 11:29:58.000000000","tz":780},"subject":"Use volume type name in volume.size notifications (if available)","message":"Use volume type name in volume.size notifications (if available)\n\nThere is a mismatch between the values returned for the\nvolume_type attribute between the Cinder volume notifications\nand what is returned when polling the Cinder API.\n\nThe Cinder APIs return the name of the volume type as volume_type,\nwhile Cinder notifications return the ID of the volume type instead.\nFor more information see this bug [1].\n\nIn Ceilometer this results in an inconsistency where for the\nvolume.size meter, polled samples set volume_type to the name,\nwhile notification samples set it to the ID. We only want to store\none of those with that name, and make the other available under\nanother name.\n\nThere is a proposed Cinder change [2] to add a\nvolume_type_name attribute to Cinder notifications.\n\nThis allows us to change the notification meter definition to\nsupply the volume type name to the volume_type Gnocchi attribute.\nThe volume type ID is now available under the volume_type_id\nmetadata attribute, to match the metadata schema of the volume.size\npollster.\n\nBackwards compatibility with older Cinder versions is provided\nby using the notification volume_type value to supply the\nvolume_type Gnocchi value if volume_type_name is unavailable.\nIn this case, Ceilometer will replicate the current behaviour.\n\nFor this change to work, the relevant Cinder change [2]\nmust be approved and merged.\n\n[1]: https://bugs.launchpad.net/cinder/+bug/2097328\n[2]: https://review.opendev.org/c/openstack/cinder/+/940674\n\nDepends-On: I71e7fca4a6d303462fa6d0ed69245b79cb28d646\nChange-Id: I97d922c808a316e0018a8ee97490461299c44e3b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/ab04317a4e91d9eb2e0ad516904f6f4a337a15d5"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/ab04317a4e91d9eb2e0ad516904f6f4a337a15d5"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"rule_name":"gerrit~DefaultSubmitRule","status":"OK","labels":[{"label":"Verified","status":"MAY","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"MAY","applied_by":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"}},{"label":"Workflow","status":"MAY"}]}],"submit_requirements":[{"name":"Verified","description":"Verified in gate by CI","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Verified\u003dMAX AND -label:Verified\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":["label:Verified\u003dMIN"],"failing_atoms":["label:Verified\u003dMAX"],"atom_explanations":{"label:Verified\u003dMAX":"","label:Verified\u003dMIN":""}}},{"name":"Code-Review","description":"Code reviewed by core reviewer","status":"SATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Code-Review\u003dMAX AND -label:Code-Review\u003dMIN","fulfilled":true,"status":"PASS","passing_atoms":["label:Code-Review\u003dMAX"],"failing_atoms":["label:Code-Review\u003dMIN"],"atom_explanations":{"label:Code-Review\u003dMAX":"","label:Code-Review\u003dMIN":""}}},{"name":"Workflow","description":"Approved for gate by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Workflow\u003dMAX AND -label:Workflow\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Workflow\u003dMAX","label:Workflow\u003dMIN"],"atom_explanations":{"label:Workflow\u003dMAX":"","label:Workflow\u003dMIN":""}}}]}