)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"ba5e0a0fe073cea1d7920ace80efae5995a2d35f","unresolved":true,"context_lines":[{"line_number":10,"context_line":"gnocchi and prometheus backends store those metrics in other units:"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"- For gnocchi, it\u0027s using ceilometer host metrics and"},{"line_number":13,"context_line":"  hardware.memory.used [1]."},{"line_number":14,"context_line":"- For prometheus, it is calculated from node_memory_MemAvailable_bytes"},{"line_number":15,"context_line":"  which is in bytes."},{"line_number":16,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"10f772b8_40a58825","line":13,"range":{"start_line":13,"start_character":23,"end_line":13,"end_character":27},"updated":"2025-06-24 13:18:15.000000000","message":"this links to a very old release doc. When trying to search for \u0027hardware.memory.used\u0027, I saw that it was removed together with all other SNMP based metrics. We need to review all gnocchi metrics at some point. IPMI metrics are still available, but \u0027cpu_l3_cache\u0027 is another one that we will need to remove from its Metric_Map.","commit_id":"75dc81a0826d4e1742e9d1170d41dbe28b62f537"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"ba5e0a0fe073cea1d7920ace80efae5995a2d35f","unresolved":true,"context_lines":[{"line_number":15,"context_line":"  which is in bytes."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"This patch is adapting the injected metrics to those units."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"I\u0027m also cleaning the labels for the host ram metrics to the prometheus"},{"line_number":20,"context_line":"backend which included cpu related labels."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"[1] https://docs.openstack.org/ceilometer/train/admin/telemetry-measurements.html#snmp-based-meters"},{"line_number":23,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"eebeec54_e3c9903c","line":20,"range":{"start_line":18,"start_character":0,"end_line":20,"end_character":42},"updated":"2025-06-24 13:18:15.000000000","message":"Ack, thanks for that.","commit_id":"75dc81a0826d4e1742e9d1170d41dbe28b62f537"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"67d3a44a6f671080b3185e398db9bf60903d4721","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ab694b3b_f33a4849","updated":"2025-06-20 10:59:14.000000000","message":"tested in https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/952807/","commit_id":"0affabcb1e74f7872c850c6fa4db7a416822b324"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"ba5e0a0fe073cea1d7920ace80efae5995a2d35f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ca36183d_3257e676","updated":"2025-06-24 13:18:15.000000000","message":"LGTM, this properly fix the unit of the injected data, according with both datasource documentation and implementations.\nAlso, memory tests were added in[1] which is validating this patch too.\nWe can furhter discuss about Gnocchi and its metrics.\n\n[1] https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/952807","commit_id":"75dc81a0826d4e1742e9d1170d41dbe28b62f537"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"e7cc79845c0c3b31b225267a70cfb0f6fd4c2df1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ae91c00c_d5eb0e56","updated":"2025-06-23 14:58:41.000000000","message":"Looks good!","commit_id":"75dc81a0826d4e1742e9d1170d41dbe28b62f537"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"d8f058b95c99b0221f216c76e03e13ccc9fb306d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"234fbfbf_6d786c50","updated":"2025-06-23 14:55:24.000000000","message":"This patch has been verified here:\nhttps://review.opendev.org/c/openstack/watcher-tempest-plugin/+/952807","commit_id":"75dc81a0826d4e1742e9d1170d41dbe28b62f537"}]}
