)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"87759913c593d98b8a99a0b39a5d6569f9116bf4","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This change adds Loki as a storage backend supporting"},{"line_number":10,"context_line":"the v2 storage API in addition of the existing InfluxDB,"},{"line_number":11,"context_line":"OpenSearch and ElasticSearch options."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I51bda8f2edd634bfe21f1a00fd55fd1fcf19d133"},{"line_number":14,"context_line":"Signed-off-by: jlarriba \u003cjlarriba@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"338e4f55_79b15f55","line":11,"range":{"start_line":11,"start_character":15,"end_line":11,"end_character":28},"updated":"2025-08-29 12:56:55.000000000","message":"Nit: Elasticsearch","commit_id":"f2f65f8e3616bb6998c8f270b4969553e24b9a20"},{"author":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"change_message_id":"dcb9f98a1185d47ea12605aeb480233eecdaf3c9","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This change adds Loki as a storage backend supporting"},{"line_number":10,"context_line":"the v2 storage API in addition of the existing InfluxDB,"},{"line_number":11,"context_line":"OpenSearch and ElasticSearch options."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I51bda8f2edd634bfe21f1a00fd55fd1fcf19d133"},{"line_number":14,"context_line":"Signed-off-by: jlarriba \u003cjlarriba@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"db6455bc_5aa2d91b","line":11,"range":{"start_line":11,"start_character":15,"end_line":11,"end_character":28},"in_reply_to":"338e4f55_79b15f55","updated":"2025-09-01 08:31:25.000000000","message":"Done","commit_id":"f2f65f8e3616bb6998c8f270b4969553e24b9a20"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"d829c6615c01da0b697957beceedab71d87929c3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"bc9f9931_df8b03fa","updated":"2025-05-26 14:32:26.000000000","message":"Hello Huan, \nDo you have a spec for this development so we can understand a little bit more about the new storage backend that is being proposed, its requirement and what operations it is compatible with?","commit_id":"8f8ff5ec9f5101e0fa83cc3454fc2d8c1cc6221c"},{"author":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"change_message_id":"713ca010cd4a514fba0fdb0a03239693c36d98e0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ba299f86_7b31c766","in_reply_to":"7c0d3768_b5942d4e","updated":"2025-05-28 14:34:31.000000000","message":"Added a new spec for this feature:  https://review.opendev.org/c/openstack/cloudkitty-specs/+/951147","commit_id":"8f8ff5ec9f5101e0fa83cc3454fc2d8c1cc6221c"},{"author":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"change_message_id":"b576ff374da015acdbc5fa4d33e083a3cd5547e2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7c0d3768_b5942d4e","in_reply_to":"bc9f9931_df8b03fa","updated":"2025-05-26 14:51:39.000000000","message":"Hi Rafael,\n\nthank you for your comment.\n\nIt is proposed mainly because the other options cannot be released as part of a complete product or because we cannot support them once deployed. Do you have an example of a spec so I could write all the rationale in the expected format? (I should\u0027ve started from there, that\u0027s true)","commit_id":"8f8ff5ec9f5101e0fa83cc3454fc2d8c1cc6221c"},{"author":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"change_message_id":"e4fbef5d16e983fca25839a1228888c0efea6ebf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"ce0be6dd_ff50f080","updated":"2025-06-10 10:34:49.000000000","message":"Can we please get some attention on this patch or a spec [1] regarding this feature?\n\n[1] https://review.opendev.org/c/openstack/cloudkitty-specs/+/951147","commit_id":"f16c2b02c2534a1d94a422193edc5f22f0ad78f2"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"153f07f705933b701620eebbc924f0c79f934c58","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"c2c45abb_d53bbaf7","updated":"2025-06-23 17:40:58.000000000","message":"We have some open issues to be addressed in the spec, before moving this one forward.","commit_id":"f16c2b02c2534a1d94a422193edc5f22f0ad78f2"},{"author":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"change_message_id":"233cd82f302624d1bc6d6956d1c067a0d426a11f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"fa6dce22_b06dda5c","updated":"2025-07-14 13:13:20.000000000","message":"Added support for the \"delete\" function by using the Loki v1/delete api. This makes reprocessing possible when using Loki as storage backend.","commit_id":"0f61020d209a6c1f947f35cc573e70dd8b60d771"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"87759913c593d98b8a99a0b39a5d6569f9116bf4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"21d10c6a_5f34f9e0","updated":"2025-08-29 12:56:55.000000000","message":"Can you please add docs to doc/source/admin/configuration/storage.rst","commit_id":"f2f65f8e3616bb6998c8f270b4969553e24b9a20"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"3048d15046fe38cbc184d66e8d1611f541b51eb9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"194886bc_4d82bccd","updated":"2025-08-18 12:07:00.000000000","message":"Is there a way to move this forward here? The spec has been approved, the functionality adds another storage backend, but should not change any behavior with other backends (verified by CI testing).","commit_id":"f2f65f8e3616bb6998c8f270b4969553e24b9a20"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"cc11553e909431b62e6461ea51a0dabdb653f2f0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"170bc83b_f5bd1f2a","updated":"2025-08-28 07:52:37.000000000","message":"Thank you for your reviews and work here so far. Since this patch has already one +2 and it does not change any existing behavior, would it make sense to proceed with this (get another +2/+W) and get it merged. We\u0027re happy to fix bugs/misbehaviors as they are identified.","commit_id":"f2f65f8e3616bb6998c8f270b4969553e24b9a20"},{"author":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"change_message_id":"dcb9f98a1185d47ea12605aeb480233eecdaf3c9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"39c384ae_941e3f05","in_reply_to":"21d10c6a_5f34f9e0","updated":"2025-09-01 08:31:25.000000000","message":"Done","commit_id":"f2f65f8e3616bb6998c8f270b4969553e24b9a20"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"f08b19552b07c16fc3762de15e1c6992e30d0e09","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"d3d75cfe_e528f7ba","updated":"2025-09-15 12:18:04.000000000","message":"My understanding is that the release branch (2025.2) has been branched, and the master branch is back open for new features? We have https://review.opendev.org/c/openstack/cloudkitty/+/960455 to prove that the feature is working.\n\nI\u0027d love to see patches landing on-top, like enabling TLS for the loki connection.","commit_id":"eb08c974226b844cae2d065d2b889b419c0d4031"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"5b5dd74de91fc5712ec5d9937e112dda119df754","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"0e5f7a62_052f29e6","updated":"2025-09-08 15:16:28.000000000","message":"Still looks good to me","commit_id":"eb08c974226b844cae2d065d2b889b419c0d4031"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"fce3324c588691b8480685b42b53fe6a4be0ba3c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"544ba403_401fb2b2","updated":"2025-09-29 14:35:18.000000000","message":"recheck TypeError: expected string or bytes-like object, got \u0027NoneType\u0027","commit_id":"eb08c974226b844cae2d065d2b889b419c0d4031"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"83cd5be35484ec5f5bb68b7869fbed3f20a37381","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"5523afcd_22fe0291","updated":"2025-09-30 07:12:35.000000000","message":"recheck oslo_config.cfg.NoSuchGroupError: no such group [oslo_policy]","commit_id":"eb08c974226b844cae2d065d2b889b419c0d4031"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"3cad674209123826406be861ce2cfabf5365f245","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"4f9c94ae_18352ec1","updated":"2025-09-29 20:30:58.000000000","message":"recheck same error in another job","commit_id":"eb08c974226b844cae2d065d2b889b419c0d4031"}],"cloudkitty/storage/v2/loki/__init__.py":[{"author":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"change_message_id":"f86e5753b038551220081141201dd08e8555c448","unresolved":true,"context_lines":[{"line_number":80,"context_line":""},{"line_number":81,"context_line":"        self._conn \u003d os_client.LokiClient("},{"line_number":82,"context_line":"            CONF.storage_loki.url,"},{"line_number":83,"context_line":"            CONF.storage_loki.tenant,"},{"line_number":84,"context_line":"            CONF.storage_loki.stream,"},{"line_number":85,"context_line":"            CONF.storage_loki.content_type,"},{"line_number":86,"context_line":"            CONF.storage_loki.buffer_size)"}],"source_content_type":"text/x-python","patch_set":5,"id":"2ae74aa1_0260d864","line":83,"updated":"2025-06-04 12:56:09.000000000","message":"Just a thought: Shouldn\u0027t we use the advantage of Loki multitenancy and instead of using pre-configured loki tenant, figure out project_id (probably) of the rating we collect or constrain the data retrieval with tenant (in case of non-admin user of course)?","commit_id":"f16c2b02c2534a1d94a422193edc5f22f0ad78f2"},{"author":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"change_message_id":"9d82742aa5d4d9c1008d8d3b4d901280cdd5495c","unresolved":false,"context_lines":[{"line_number":80,"context_line":""},{"line_number":81,"context_line":"        self._conn \u003d os_client.LokiClient("},{"line_number":82,"context_line":"            CONF.storage_loki.url,"},{"line_number":83,"context_line":"            CONF.storage_loki.tenant,"},{"line_number":84,"context_line":"            CONF.storage_loki.stream,"},{"line_number":85,"context_line":"            CONF.storage_loki.content_type,"},{"line_number":86,"context_line":"            CONF.storage_loki.buffer_size)"}],"source_content_type":"text/x-python","patch_set":5,"id":"cea10794_f6b05775","line":83,"in_reply_to":"2ae74aa1_0260d864","updated":"2025-06-05 07:23:00.000000000","message":"Definitively we want to introduce multitenancy capabilities, but I think this patch is big enough with just the basic functionality. More advanced things like multitenancy support and TLS encrypted connections that need to be supported will be incoming in subsequent patches, smaller and easier to review.","commit_id":"f16c2b02c2534a1d94a422193edc5f22f0ad78f2"},{"author":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"change_message_id":"38d57f1785dd215749e06dc35dc7b2e10b2966fb","unresolved":false,"context_lines":[{"line_number":80,"context_line":""},{"line_number":81,"context_line":"        self._conn \u003d os_client.LokiClient("},{"line_number":82,"context_line":"            CONF.storage_loki.url,"},{"line_number":83,"context_line":"            CONF.storage_loki.tenant,"},{"line_number":84,"context_line":"            CONF.storage_loki.stream,"},{"line_number":85,"context_line":"            CONF.storage_loki.content_type,"},{"line_number":86,"context_line":"            CONF.storage_loki.buffer_size)"}],"source_content_type":"text/x-python","patch_set":5,"id":"9293e360_e981f753","line":83,"in_reply_to":"cea10794_f6b05775","updated":"2025-06-05 08:32:15.000000000","message":"Fair enough","commit_id":"f16c2b02c2534a1d94a422193edc5f22f0ad78f2"}]}