)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":36393,"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","username":"Callum027","status":"Catalyst Cloud"},"change_message_id":"3aaf38b14fab14171cc175394b296502c067be36","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d992dd69_86ef8202","updated":"2025-08-14 20:53:08.000000000","message":"I didn\u0027t think too much about whether or not these were was published in `shutoff` state, but if they aren\u0027t then they shouldn\u0027t cause any issues for instances waiting a resize confirm I think.\n\nI don\u0027t think it\u0027s a big deal that they\u0027re not published for shut off instances, you also don\u0027t get CPU time or anything like that so any metrics you\u0027d like to aggregate it with aren\u0027t being populated anyway.","commit_id":"4893e6242d339f5b66d12a3820106c4e9849a6f4"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"6b52bbd4070e492e8d7a7c910cc6eabe8afd6106","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"845d4a06_df2e1233","updated":"2025-08-13 12:25:55.000000000","message":"I have to restore my memory, but how does this behave when an instance is resized ?\nIIRC nova keeps the original domain at source until resize is confirmed so we may get two conflicting meter values from two hosts, I guess.","commit_id":"4893e6242d339f5b66d12a3820106c4e9849a6f4"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"d7eeada9c840873f72e736680a120fbf2b91c112","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"22b2dabe_6199aa26","updated":"2025-08-13 11:50:15.000000000","message":"Thank you!","commit_id":"4893e6242d339f5b66d12a3820106c4e9849a6f4"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"92a6691126de308ae677b31367600a60a90e8de2","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b4e1c81f_701a01c2","in_reply_to":"2208e2ab_e8cced27","updated":"2025-08-14 14:49:59.000000000","message":"I still don\u0027t get why only the power_state is affected, it seems we\u0027re missing something there. It would make sense to exempt the power_state reporting from only reporting while the domain is running. I need to go back and look at the patch.\n\nI\u0027d think this and also the other Calum patch add metrics to pollsters actually improve the metrics reported and don\u0027t do any harm.","commit_id":"4893e6242d339f5b66d12a3820106c4e9849a6f4"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"917a92d414f4769e26dc73327a0761b3e61b0478","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b7c6fb0e_102a46b4","in_reply_to":"845d4a06_df2e1233","updated":"2025-08-14 13:58:41.000000000","message":"In my understanding, this is not different to other pollsters being polled from computes.\nThe nova behavior is not changed due to this patch. \n\nFor the issue with resizing, or migrating, we may want to do something about pollsters pulling data from instances being paused or stopped.","commit_id":"4893e6242d339f5b66d12a3820106c4e9849a6f4"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"310dea41b4af8d9da07253da6920a487b00a112c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7371de6a_78d140f3","in_reply_to":"b4e1c81f_701a01c2","updated":"2025-08-14 16:12:28.000000000","message":"I was referring to the logic in pollster which skip generating samples with None value. but I later noticed that current inspector does not generate stats for instance in shutoff state. So I think we should be fine...\n\nhttps://github.com/openstack/ceilometer/blob/8763625d82bca2be025ce9334c0927d90d1cdc03/ceilometer/compute/virt/libvirt/inspector.py#L208","commit_id":"4893e6242d339f5b66d12a3820106c4e9849a6f4"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"bfac1bde9885d1245c58546c78ee28df38697a9c","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2208e2ab_e8cced27","in_reply_to":"b7c6fb0e_102a46b4","updated":"2025-08-14 14:38:32.000000000","message":"\u003e In my understanding, this is not different to other pollsters being polled from computes.\n\nIIUC the other pollsters only get metrics when the domain is running, so may not cause duplication.\nThe only exception we currently have is power_state pollster which we discussed yesterday.","commit_id":"4893e6242d339f5b66d12a3820106c4e9849a6f4"}],"releasenotes/notes/add-vcpus-pollster-23cfa683d07092b3.yaml":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"6b52bbd4070e492e8d7a7c910cc6eabe8afd6106","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The ``vcpus`` meter now has measures published by a compute pollster,"},{"line_number":5,"context_line":"    in addition to the existing notification meter."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"15b0554b_eeaf1fec","line":5,"range":{"start_line":3,"start_character":5,"end_line":5,"end_character":51},"updated":"2025-08-13 12:25:55.000000000","message":"It\u0027s probably worth mentioning that users have to enable \u0027vcpus\u0027 meter in their polling.yaml .","commit_id":"4893e6242d339f5b66d12a3820106c4e9849a6f4"}]}
