)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":36393,"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","username":"Callum027","status":"Catalyst Cloud"},"change_message_id":"a45cfa88782742840d8ab5e541df627dd8d48128","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e0d19e19_77f72180","updated":"2025-08-28 09:53:56.000000000","message":"If I\u0027m too late for feature freeze for this patch, let me know and I\u0027ll update the patch so it can go into 2026.1 instead. Thanks!","commit_id":"b502af5bc689cfd2fe1e939c4138663182d197c1"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"548d4aa2fc591b9e1230646ee39a6c19914534a9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"5aee41c3_f9504813","updated":"2025-08-28 10:20:29.000000000","message":"Thank you!","commit_id":"b502af5bc689cfd2fe1e939c4138663182d197c1"},{"author":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"change_message_id":"da3be8b7bb77374e9db0237b54acd8ac5f737e16","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d5a62446_c1e54ace","updated":"2025-08-28 09:08:46.000000000","message":"Thanks for your patch. The maximum memory available to the instance is sent via notification from Nova and it\u0027s called just \"memory\".\n\nI see value on having the metric also retrieved by the pollster, and there has been patches to do the same, for example, with the disk: https://review.opendev.org/c/openstack/ceilometer/+/943366.\n\nWhat we requested in that patch is that the metric retrieved by the pollster has the same name as the one received via notification, as they report the same information.\n\nIn this regard, I would ask you to call this metric \"memory\" instead of \"memory.available\".","commit_id":"b502af5bc689cfd2fe1e939c4138663182d197c1"},{"author":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"change_message_id":"d06e51a848269fd2fe1ddc04faf7ce79abb4e61e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"56f18fad_881ad6e7","in_reply_to":"2306b90a_1ae65851","updated":"2025-08-28 10:23:21.000000000","message":"oh yeah! I missed that one too.\n\nThank you very much Callum.","commit_id":"b502af5bc689cfd2fe1e939c4138663182d197c1"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"548d4aa2fc591b9e1230646ee39a6c19914534a9","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"2306b90a_1ae65851","in_reply_to":"5728b506_e3e98239","updated":"2025-08-28 10:20:29.000000000","message":"Are you referring to this patch https://review.opendev.org/c/openstack/ceilometer/+/957214 ?","commit_id":"b502af5bc689cfd2fe1e939c4138663182d197c1"},{"author":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"change_message_id":"8b29d079aedc19817a42cfdb864ffedf93bf354c","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"5728b506_e3e98239","in_reply_to":"bf11fd68_15c6027d","updated":"2025-08-28 10:07:14.000000000","message":"Ah, gotcha, I missed the \"memory\" patch, it was during my holidays.\n\nThank you, now I understand much better your string of patches 😊\n\nI wonder that if you are at it, are you intending to also add in the future the \"vcpus\" metric to the pollster? That way we will have both.","commit_id":"b502af5bc689cfd2fe1e939c4138663182d197c1"},{"author":{"_account_id":36393,"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","username":"Callum027","status":"Catalyst Cloud"},"change_message_id":"5dbadfef8b187a0cac548a2500a7c98c5b8bb148","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"bf11fd68_15c6027d","in_reply_to":"d5a62446_c1e54ace","updated":"2025-08-28 09:15:48.000000000","message":"Hi Juan, to give more context [I actually added a `memory` pollster in this patch](https://review.opendev.org/c/openstack/ceilometer/+/957216).\n\nThe issue I\u0027m seeing with it is I\u0027m seeing that [notification metrics and pollster metrics do not match](https://paste.openstack.org/show/bwr3zGzeYKJNzv9K5yZL/), because the pollster is currently using the **available memory as seen inside the instance** via the libvirt balloon driver, instead of the memory allocated to the instance by Nova/libvirt (I should have tested this more thoroughly, I apologise).\n\nI\u0027ve made a patch to fix the new `memory` pollster here: https://review.opendev.org/c/openstack/ceilometer/+/958733\n\nWhat I\u0027m doing in this patch is making a new metric for exposing the available memory inside the instance (because I\u0027d like to use it), but as a separate metric to maintain compatibility.","commit_id":"b502af5bc689cfd2fe1e939c4138663182d197c1"},{"author":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"change_message_id":"8b29d079aedc19817a42cfdb864ffedf93bf354c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d783eb55_2cebb9cd","in_reply_to":"e0d19e19_77f72180","updated":"2025-08-28 10:07:14.000000000","message":"I think that this feature is simple enough that can go into Flamingo. I use to apply the feature freeze to more complex features.","commit_id":"b502af5bc689cfd2fe1e939c4138663182d197c1"}]}