)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"ea23301350dc1cdcb4ba8e0af0c95e55793b9687","unresolved":true,"context_lines":[{"line_number":13,"context_line":"CloudKitty and you wish to use these group by methods, you will need"},{"line_number":14,"context_line":"to reprocess the desired timeframe."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Closes: https://storyboard.openstack.org/#!/story/2009839"},{"line_number":17,"context_line":"Story: #2009839"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: Iad296f54f6701af84e168796aec9b1033a2a8a2d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"6d46a0b1_4e61ec5c","line":16,"range":{"start_line":16,"start_character":0,"end_line":16,"end_character":57},"updated":"2023-02-06 13:56:23.000000000","message":"As far as I know this doesn\u0027t do anything special. Just keep the reference to the story on the next line.","commit_id":"92e5757f8f8416195065ddc0e457d3aaf9c51303"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"b77b754ba772490abe140cbe930fc11aa674b558","unresolved":false,"context_lines":[{"line_number":13,"context_line":"CloudKitty and you wish to use these group by methods, you will need"},{"line_number":14,"context_line":"to reprocess the desired timeframe."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Closes: https://storyboard.openstack.org/#!/story/2009839"},{"line_number":17,"context_line":"Story: #2009839"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: Iad296f54f6701af84e168796aec9b1033a2a8a2d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"21c59278_9842509c","line":16,"range":{"start_line":16,"start_character":0,"end_line":16,"end_character":57},"in_reply_to":"6d46a0b1_4e61ec5c","updated":"2023-02-20 11:16:33.000000000","message":"Done","commit_id":"92e5757f8f8416195065ddc0e457d3aaf9c51303"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"ea23301350dc1cdcb4ba8e0af0c95e55793b9687","unresolved":true,"context_lines":[{"line_number":14,"context_line":"to reprocess the desired timeframe."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Closes: https://storyboard.openstack.org/#!/story/2009839"},{"line_number":17,"context_line":"Story: #2009839"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: Iad296f54f6701af84e168796aec9b1033a2a8a2d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"89716cdc_bd516cb5","line":17,"range":{"start_line":17,"start_character":0,"end_line":17,"end_character":15},"updated":"2023-02-06 13:56:23.000000000","message":"Please also add reference to task.","commit_id":"92e5757f8f8416195065ddc0e457d3aaf9c51303"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"b77b754ba772490abe140cbe930fc11aa674b558","unresolved":false,"context_lines":[{"line_number":14,"context_line":"to reprocess the desired timeframe."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Closes: https://storyboard.openstack.org/#!/story/2009839"},{"line_number":17,"context_line":"Story: #2009839"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: Iad296f54f6701af84e168796aec9b1033a2a8a2d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"d1fe6363_88f35b63","line":17,"range":{"start_line":17,"start_character":0,"end_line":17,"end_character":15},"in_reply_to":"89716cdc_bd516cb5","updated":"2023-02-20 11:16:33.000000000","message":"Done","commit_id":"92e5757f8f8416195065ddc0e457d3aaf9c51303"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"03fff0a88f87fa3bd71207fe4a54f3c66c64a40f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"bedd50e9_ce9c2c35","updated":"2023-02-06 14:36:01.000000000","message":"Could we please have gabbi tests for this?","commit_id":"92e5757f8f8416195065ddc0e457d3aaf9c51303"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"cb3999d275a4f5cfdda0f29fd469b95fb002730f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"d667dd22_197b0b47","updated":"2023-01-26 12:35:34.000000000","message":"Hello guys, \nHave you had some time to check this patch?","commit_id":"92e5757f8f8416195065ddc0e457d3aaf9c51303"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"2c20ea6057eb42f23261e25c8c315b809e0817c8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"228c877f_d7e2e493","in_reply_to":"bedd50e9_ce9c2c35","updated":"2023-03-15 17:11:53.000000000","message":"tests created.","commit_id":"92e5757f8f8416195065ddc0e457d3aaf9c51303"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"2c20ea6057eb42f23261e25c8c315b809e0817c8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"e501b7bf_58ebb4ac","updated":"2023-03-15 17:11:53.000000000","message":"Hello guys, Gabbi tests added. Can you review this patch again?","commit_id":"751125c1dcc5a4988d2a9ee1e058e8fda4a8ad29"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"5ba6edbec1e6b3df7108713d0983aeada400e87d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"a8a66dee_d49a746f","updated":"2023-03-15 16:21:56.000000000","message":"recheck","commit_id":"751125c1dcc5a4988d2a9ee1e058e8fda4a8ad29"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"d7e5e7effd66dd91c10b4fa07ea99bbe55772d44","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"641ba171_d44b4598","updated":"2023-08-17 20:06:58.000000000","message":"Hello guys, are we missing something here? the patch seems ready to be merged.","commit_id":"2ebe42489aaecc4f45c61d8a71ef02315f066853"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"4ed23703c8bbc2198c41ff872c8dbe8f7f9117bb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"31a16ad6_13f964e6","updated":"2023-08-21 14:21:03.000000000","message":"The groupby options described in commit message still don\u0027t match the release note. What about the code?","commit_id":"2ebe42489aaecc4f45c61d8a71ef02315f066853"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"1d1471f15f28f5fbdb063b565604fdb461150d34","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"467dd731_4b0c1cf2","in_reply_to":"31a16ad6_13f964e6","updated":"2023-08-22 17:22:19.000000000","message":"The code is as described in the release notes. Now I see what you mean. The commit is not talking directly about the options created, but the idea/concept. I will change them, if you think they need to be literal.","commit_id":"2ebe42489aaecc4f45c61d8a71ef02315f066853"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"e40f76369f099ba56e773bfc2d574092eaa49ce9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"7a2d66d8_0037e510","updated":"2023-08-25 13:29:58.000000000","message":"recheck","commit_id":"dc3c70bb23a55c3e67eca57b72f92100230dc531"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"c5ff1a9ceabc92a8f9cd410ed4f5f47e3eedd223","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"b85bd503_7e288f77","updated":"2023-08-23 13:04:55.000000000","message":"recheck","commit_id":"dc3c70bb23a55c3e67eca57b72f92100230dc531"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"f3a26dfe745fedbf72d2a9de41dfd1acae134050","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"c4fb3283_aa372a01","updated":"2023-08-25 14:32:35.000000000","message":"Guys, now Zuul is working again! Can you check this patch to see if we can merge it?","commit_id":"8190fc5067e5ac6aa8f434b7e0202446e52504c6"},{"author":{"_account_id":30695,"name":"Pedro Henrique Pereira Martins","email":"phpm13@gmail.com","username":"pedrohpmartins"},"change_message_id":"6b06fe2f36016f847d5252d2e7b30419a49435e6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"22632ae1_7ff7fe04","updated":"2023-08-28 15:00:04.000000000","message":"Hi Rafael, nice patch I have only a few considerations.","commit_id":"8190fc5067e5ac6aa8f434b7e0202446e52504c6"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"5aaf641646da7ec7c4df4bae9efb470307245d25","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"f0c44749_5d536a0b","updated":"2023-08-28 18:15:31.000000000","message":"Thanks for the review Pedro!","commit_id":"8190fc5067e5ac6aa8f434b7e0202446e52504c6"},{"author":{"_account_id":30695,"name":"Pedro Henrique Pereira Martins","email":"phpm13@gmail.com","username":"pedrohpmartins"},"change_message_id":"3cf7f5716ecf9134483ff9c15c9476899c822118","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"cae790fe_6cceabe3","updated":"2023-08-28 18:18:36.000000000","message":"It Looks good to me, thanks","commit_id":"7905b43f5dab020220bbe6e2d796861582d50b76"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"08f72b82fb9e9be620b7e73c53e4b54789686e28","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"bbaa3891_a0b8e851","updated":"2023-09-07 11:55:06.000000000","message":"What do you guys think? Can we merge this patch with that work around to use the latest version for the WSME? Bear in mind, that it is only needed while testing/running the docs pipeline","commit_id":"9434110afadb25e7e6671f9a722fd8f79931020a"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"4271308392092c98253a442639bc4c3bfedc4644","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"6822a935_01cf7cec","updated":"2023-10-13 19:04:44.000000000","message":"Hello guys, \nCan we move on with this one?","commit_id":"3fbde3b531c01919bf062b9aa84b8df45066bd12"},{"author":{"_account_id":25277,"name":"Mariusz Karpiarz","email":"mariusz.karpiarz@codethink.co.uk","username":"mkarpiarz"},"change_message_id":"9d8db088069af19c570f9258f324d44d97a8bd63","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"33197c05_8238cb92","updated":"2023-10-16 10:07:34.000000000","message":"In my test environment I don\u0027t have enough metrics to test `groupby\u003dtime-y` properly but everything else seems to be working as intended.\n\nOne thing I noticed is that `groupby\u003dmonth` and `groupby\u003dyear` are available even in stable Zed CloudKitty (version 17). In fact even arbitrary group-by values are accepted by this part of the code (not part of this change): https://opendev.org/openstack/cloudkitty/src/branch/stable/zed/cloudkitty/api/v2/summary/summary.py#L75-L88\n\n```\n        arguments \u003d {\n            \u0027begin\u0027: begin,\n            \u0027end\u0027: end,\n            \u0027groupby\u0027: groupby,\n            \u0027filters\u0027: filters,\n            \u0027metric_types\u0027: metric_types,\n            \u0027offset\u0027: offset,\n            \u0027limit\u0027: limit,\n            \u0027paginate\u0027: True\n        }\n        if custom_fields:\n            arguments[\u0027custom_fields\u0027] \u003d custom_fields\n\n        total \u003d self._storage.total(**arguments)\n\n```\nI take it you guys were aware of this and wanted to extend this functionality by adding those `time-*` parameters.\n\nAlso, something that is outside of the scope of this change, since stable branches behave the same, but related to the previous paragraph: Nonsensical values for the `group-by` parameter (for example \"blah\") are currently accepted and they even return results like this one:\n\n```\n\"results\": [[\"\u003cstart-date\u003e\", \"\u003cend-date\u003e\", 115.0, 57.5, \"\"]\n```\nNotice that the last value on the above list is an empty string.\nWould it make sense to restrict group-by parameters to a list of known (i.e. implemented) values or is it better to keep this part as flexible as it is right now and accept all those false positive results?","commit_id":"3fbde3b531c01919bf062b9aa84b8df45066bd12"}],"api-ref/source/v2/summary/summary_parameters.yml":[{"author":{"_account_id":30695,"name":"Pedro Henrique Pereira Martins","email":"phpm13@gmail.com","username":"pedrohpmartins"},"change_message_id":"6b06fe2f36016f847d5252d2e7b30419a49435e6","unresolved":true,"context_lines":[{"line_number":56,"context_line":"    defined in the collector YML settings. Therefore, one can group the "},{"line_number":57,"context_line":"    result using any of the `groupby` attributes defined in the collector "},{"line_number":58,"context_line":"    settings of CloudKitty. Besides those attributes, by default, starting "},{"line_number":59,"context_line":"    in CloudKitty `Antelope` release, we will have the following new groupby"},{"line_number":60,"context_line":"    options: (i) time: to group data by hourly; (ii) time-d: to group data "},{"line_number":61,"context_line":"    by day of the year; (iii) time-w: to group data by week of the year; "},{"line_number":62,"context_line":"    (iv) month: to group data by month; and, (v) year: to group data by year."}],"source_content_type":"text/x-yaml","patch_set":5,"id":"a31456dd_a0ffcb26","line":59,"range":{"start_line":59,"start_character":19,"end_line":59,"end_character":27},"updated":"2023-08-28 15:00:04.000000000","message":"I guess it is not more Antelope, right?\n\nThis feature is going to create new columns/tags/attributes in the storage backends, which are defined here https://review.opendev.org/c/openstack/cloudkitty/+/861786/5/cloudkitty/collector/__init__.py#269.\n\nSo if we groupby in the summary API by those keys, it is going to work the same as group by their short version, so:\n\n - grouping by \"time-d\" is the same as grouping by \"day_of_the_year\";\n - grouping by \"time-w\" is the same as grouping by \"week_of_the_year\";\n - grouping by \"time-m\" is the same as grouping by \"month\";\n - grouping by \"time-y\" is the same as grouping by \"year\".\n\nI think  that it would be nice to have both options here in the documentation, so users know their possibilities about using a short or complete notation to group by time.","commit_id":"92e5757f8f8416195065ddc0e457d3aaf9c51303"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"5aaf641646da7ec7c4df4bae9efb470307245d25","unresolved":false,"context_lines":[{"line_number":56,"context_line":"    defined in the collector YML settings. Therefore, one can group the "},{"line_number":57,"context_line":"    result using any of the `groupby` attributes defined in the collector "},{"line_number":58,"context_line":"    settings of CloudKitty. Besides those attributes, by default, starting "},{"line_number":59,"context_line":"    in CloudKitty `Antelope` release, we will have the following new groupby"},{"line_number":60,"context_line":"    options: (i) time: to group data by hourly; (ii) time-d: to group data "},{"line_number":61,"context_line":"    by day of the year; (iii) time-w: to group data by week of the year; "},{"line_number":62,"context_line":"    (iv) month: to group data by month; and, (v) year: to group data by year."}],"source_content_type":"text/x-yaml","patch_set":5,"id":"257fbf86_42d80f11","line":59,"range":{"start_line":59,"start_character":19,"end_line":59,"end_character":27},"in_reply_to":"a31456dd_a0ffcb26","updated":"2023-08-28 18:15:31.000000000","message":"Done","commit_id":"92e5757f8f8416195065ddc0e457d3aaf9c51303"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"cebd8813dc59f60176a5d69c7ab349a47a3711c2","unresolved":true,"context_lines":[{"line_number":57,"context_line":"    result using any of the `groupby` attributes defined in the collector"},{"line_number":58,"context_line":"    settings of CloudKitty. Besides those attributes, by default, starting"},{"line_number":59,"context_line":"    in CloudKitty `C` release, we will have the following new groupby"},{"line_number":60,"context_line":"    options: (i) time: to group data by hourly; (ii) time-d: to group data"},{"line_number":61,"context_line":"    by day of the year; (iii) time-w: to group data by week of the year;"},{"line_number":62,"context_line":"    (iv) time-m: to group data by month; and, (v) time-y: to group data by"},{"line_number":63,"context_line":"    year. If you have old data in CloudKitty and you wish to use these"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"53971681_35eeb79b","line":60,"range":{"start_line":60,"start_character":23,"end_line":60,"end_character":46},"updated":"2023-10-02 14:19:00.000000000","message":"This should say \"by hour\" or \"hourly\", but not \"by hourly\"","commit_id":"2733b0380e573aa34939fed242c1f79251d54b92"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"a47b83805df67f154acd2ad07ad4e388a6da84e5","unresolved":false,"context_lines":[{"line_number":57,"context_line":"    result using any of the `groupby` attributes defined in the collector"},{"line_number":58,"context_line":"    settings of CloudKitty. Besides those attributes, by default, starting"},{"line_number":59,"context_line":"    in CloudKitty `C` release, we will have the following new groupby"},{"line_number":60,"context_line":"    options: (i) time: to group data by hourly; (ii) time-d: to group data"},{"line_number":61,"context_line":"    by day of the year; (iii) time-w: to group data by week of the year;"},{"line_number":62,"context_line":"    (iv) time-m: to group data by month; and, (v) time-y: to group data by"},{"line_number":63,"context_line":"    year. If you have old data in CloudKitty and you wish to use these"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"f30870e4_c6d45608","line":60,"range":{"start_line":60,"start_character":23,"end_line":60,"end_character":46},"in_reply_to":"53971681_35eeb79b","updated":"2023-10-02 14:22:46.000000000","message":"Done","commit_id":"2733b0380e573aa34939fed242c1f79251d54b92"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"cebd8813dc59f60176a5d69c7ab349a47a3711c2","unresolved":true,"context_lines":[{"line_number":63,"context_line":"    year. If you have old data in CloudKitty and you wish to use these"},{"line_number":64,"context_line":"    group by methods, you will need to reprocess the desired timeframe."},{"line_number":65,"context_line":"    The `groupby` options `time-d`, `time-w`, `time-m`, `time-y` are the"},{"line_number":66,"context_line":"    short versions of the follwing `groupby` options `day_of_the_year`,"},{"line_number":67,"context_line":"    `week_of_the_year`, `month`, and `year` respectively."},{"line_number":68,"context_line":"  type: list of strings"},{"line_number":69,"context_line":"  required: false"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"684c00e3_dbf26ab5","line":66,"range":{"start_line":66,"start_character":26,"end_line":66,"end_character":34},"updated":"2023-10-02 14:19:00.000000000","message":"following","commit_id":"2733b0380e573aa34939fed242c1f79251d54b92"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"a47b83805df67f154acd2ad07ad4e388a6da84e5","unresolved":false,"context_lines":[{"line_number":63,"context_line":"    year. If you have old data in CloudKitty and you wish to use these"},{"line_number":64,"context_line":"    group by methods, you will need to reprocess the desired timeframe."},{"line_number":65,"context_line":"    The `groupby` options `time-d`, `time-w`, `time-m`, `time-y` are the"},{"line_number":66,"context_line":"    short versions of the follwing `groupby` options `day_of_the_year`,"},{"line_number":67,"context_line":"    `week_of_the_year`, `month`, and `year` respectively."},{"line_number":68,"context_line":"  type: list of strings"},{"line_number":69,"context_line":"  required: false"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"85175126_d6e06928","line":66,"range":{"start_line":66,"start_character":26,"end_line":66,"end_character":34},"in_reply_to":"684c00e3_dbf26ab5","updated":"2023-10-02 14:22:46.000000000","message":"Done","commit_id":"2733b0380e573aa34939fed242c1f79251d54b92"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"89957572507f0056a82e116f44da9886061ac759","unresolved":true,"context_lines":[{"line_number":55,"context_line":"    Optional attributes to group the summary by. The `groupby` elements are"},{"line_number":56,"context_line":"    defined in the collector YML settings. Therefore, one can group the"},{"line_number":57,"context_line":"    result using any of the `groupby` attributes defined in the collector"},{"line_number":58,"context_line":"    settings of CloudKitty. Besides those attributes, by default, starting"},{"line_number":59,"context_line":"    in CloudKitty `C` release, we will have the following new groupby"},{"line_number":60,"context_line":"    options: (i) time: to group data hourly; (ii) time-d: to group data"},{"line_number":61,"context_line":"    by day of the year; (iii) time-w: to group data by week of the year;"}],"source_content_type":"text/x-yaml","patch_set":19,"id":"1ab5ed87_e5087067","line":58,"updated":"2023-10-16 14:20:37.000000000","message":"You probably meant for all the backticks to be double backticks?","commit_id":"3fbde3b531c01919bf062b9aa84b8df45066bd12"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"8b8d912e93082f14642eb150ccf5f602d0e2d96b","unresolved":false,"context_lines":[{"line_number":55,"context_line":"    Optional attributes to group the summary by. The `groupby` elements are"},{"line_number":56,"context_line":"    defined in the collector YML settings. Therefore, one can group the"},{"line_number":57,"context_line":"    result using any of the `groupby` attributes defined in the collector"},{"line_number":58,"context_line":"    settings of CloudKitty. Besides those attributes, by default, starting"},{"line_number":59,"context_line":"    in CloudKitty `C` release, we will have the following new groupby"},{"line_number":60,"context_line":"    options: (i) time: to group data hourly; (ii) time-d: to group data"},{"line_number":61,"context_line":"    by day of the year; (iii) time-w: to group data by week of the year;"}],"source_content_type":"text/x-yaml","patch_set":19,"id":"f7d6209f_99d6cdca","line":58,"in_reply_to":"1ab5ed87_e5087067","updated":"2023-10-16 14:24:01.000000000","message":"Done","commit_id":"3fbde3b531c01919bf062b9aa84b8df45066bd12"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"809edc020baa6df572ab49a88a61d7057d65e0d2","unresolved":true,"context_lines":[{"line_number":56,"context_line":"    defined in the collector YML settings. Therefore, one can group the"},{"line_number":57,"context_line":"    result using any of the ``groupby`` attributes defined in the collector"},{"line_number":58,"context_line":"    settings of CloudKitty. Besides those attributes, by default, starting"},{"line_number":59,"context_line":"    in CloudKitty ``C`` release, we will have the following new groupby"},{"line_number":60,"context_line":"    options: (i) time: to group data hourly; (ii) time-d: to group data"},{"line_number":61,"context_line":"    by day of the year; (iii) time-w: to group data by week of the year;"},{"line_number":62,"context_line":"    (iv) time-m: to group data by month; and, (v) time-y: to group data by"}],"source_content_type":"text/x-yaml","patch_set":20,"id":"1f0d75ee_c8ca855e","line":59,"range":{"start_line":59,"start_character":18,"end_line":59,"end_character":23},"updated":"2023-11-13 14:21:40.000000000","message":"Replace by: 2024.1","commit_id":"730c8b2320af739ac02e518583bd795abd0e3bf2"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"841fc1894dfee711e728268b6ca7a61753218df5","unresolved":false,"context_lines":[{"line_number":56,"context_line":"    defined in the collector YML settings. Therefore, one can group the"},{"line_number":57,"context_line":"    result using any of the ``groupby`` attributes defined in the collector"},{"line_number":58,"context_line":"    settings of CloudKitty. Besides those attributes, by default, starting"},{"line_number":59,"context_line":"    in CloudKitty ``C`` release, we will have the following new groupby"},{"line_number":60,"context_line":"    options: (i) time: to group data hourly; (ii) time-d: to group data"},{"line_number":61,"context_line":"    by day of the year; (iii) time-w: to group data by week of the year;"},{"line_number":62,"context_line":"    (iv) time-m: to group data by month; and, (v) time-y: to group data by"}],"source_content_type":"text/x-yaml","patch_set":20,"id":"97ae81a8_0d42f711","line":59,"range":{"start_line":59,"start_character":18,"end_line":59,"end_character":23},"in_reply_to":"1f0d75ee_c8ca855e","updated":"2023-11-13 14:26:12.000000000","message":"Done","commit_id":"730c8b2320af739ac02e518583bd795abd0e3bf2"}],"cloudkitty/tests/storage/v2/test_storage_unit.py":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"f4e6ed565f190a91e860ebb334fa44ebe4226247","unresolved":true,"context_lines":[{"line_number":381,"context_line":"            groupby)"},{"line_number":382,"context_line":""},{"line_number":383,"context_line":"    def test_parse_groupby_syntax_to_groupby_elements_all_time_options(self):"},{"line_number":384,"context_line":"        groupby \u003d [\"something\", \"time\", \"time-d\", \"time-w\", \"time-m\", \"time-y\"]"},{"line_number":385,"context_line":""},{"line_number":386,"context_line":"        expected_log_calls \u003d []"},{"line_number":387,"context_line":"        for k, v in storage.v2.BaseStorage.TIME_COMMANDS_MAP.items():"}],"source_content_type":"text/x-python","patch_set":5,"id":"b774a5a7_bedbd808","line":384,"range":{"start_line":384,"start_character":32,"end_line":384,"end_character":78},"updated":"2023-02-06 14:35:00.000000000","message":"This doesn\u0027t seem to match the release note.","commit_id":"92e5757f8f8416195065ddc0e457d3aaf9c51303"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"b77b754ba772490abe140cbe930fc11aa674b558","unresolved":false,"context_lines":[{"line_number":381,"context_line":"            groupby)"},{"line_number":382,"context_line":""},{"line_number":383,"context_line":"    def test_parse_groupby_syntax_to_groupby_elements_all_time_options(self):"},{"line_number":384,"context_line":"        groupby \u003d [\"something\", \"time\", \"time-d\", \"time-w\", \"time-m\", \"time-y\"]"},{"line_number":385,"context_line":""},{"line_number":386,"context_line":"        expected_log_calls \u003d []"},{"line_number":387,"context_line":"        for k, v in storage.v2.BaseStorage.TIME_COMMANDS_MAP.items():"}],"source_content_type":"text/x-python","patch_set":5,"id":"f87ecbdd_806088c3","line":384,"range":{"start_line":384,"start_character":32,"end_line":384,"end_character":78},"in_reply_to":"b774a5a7_bedbd808","updated":"2023-02-20 11:16:33.000000000","message":"I fixed the release notes.","commit_id":"92e5757f8f8416195065ddc0e457d3aaf9c51303"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"3afb3b4f13a3d8ed7fc438d34778b42955ef6974","unresolved":true,"context_lines":[{"line_number":26,"context_line":"from cloudkitty.tests import utils as test_utils"},{"line_number":27,"context_line":"from cloudkitty.utils import tz as tzutils"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"from werkzeug import exceptions as http_exceptions"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"_ES_CLIENT_PATH \u003d (\u0027cloudkitty.storage.v2.elasticsearch\u0027"},{"line_number":32,"context_line":"                   \u0027.client.ElasticsearchClient\u0027)"}],"source_content_type":"text/x-python","patch_set":16,"id":"daf93cc8_cdf0a1cf","line":29,"updated":"2023-10-02 14:17:13.000000000","message":"This should be with the testscenarios import.","commit_id":"2733b0380e573aa34939fed242c1f79251d54b92"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"19d0f41fbcb231b484ecb0cff3e32c1b51f71ec5","unresolved":false,"context_lines":[{"line_number":26,"context_line":"from cloudkitty.tests import utils as test_utils"},{"line_number":27,"context_line":"from cloudkitty.utils import tz as tzutils"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"from werkzeug import exceptions as http_exceptions"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"_ES_CLIENT_PATH \u003d (\u0027cloudkitty.storage.v2.elasticsearch\u0027"},{"line_number":32,"context_line":"                   \u0027.client.ElasticsearchClient\u0027)"}],"source_content_type":"text/x-python","patch_set":16,"id":"4f01746b_ce7608d4","line":29,"in_reply_to":"ba49d7fb_9bf4407c","updated":"2023-10-02 14:29:32.000000000","message":"Done","commit_id":"2733b0380e573aa34939fed242c1f79251d54b92"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"8e47b03dd989d1b502de08f322e209478ab0cb43","unresolved":true,"context_lines":[{"line_number":26,"context_line":"from cloudkitty.tests import utils as test_utils"},{"line_number":27,"context_line":"from cloudkitty.utils import tz as tzutils"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"from werkzeug import exceptions as http_exceptions"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"_ES_CLIENT_PATH \u003d (\u0027cloudkitty.storage.v2.elasticsearch\u0027"},{"line_number":32,"context_line":"                   \u0027.client.ElasticsearchClient\u0027)"}],"source_content_type":"text/x-python","patch_set":16,"id":"ba49d7fb_9bf4407c","line":29,"in_reply_to":"daf93cc8_cdf0a1cf","updated":"2023-10-02 14:19:14.000000000","message":"I am not sure I follow. You mean moving the line close to the import of testscenarios?","commit_id":"2733b0380e573aa34939fed242c1f79251d54b92"}],"doc/requirements.txt":[{"author":{"_account_id":25277,"name":"Mariusz Karpiarz","email":"mariusz.karpiarz@codethink.co.uk","username":"mkarpiarz"},"change_message_id":"2ec050e63263c70f1e7720fe63e26d177c141ec0","unresolved":true,"context_lines":[{"line_number":1,"context_line":"sphinx\u003e\u003d3.3.1,\u003c5.0.0 # BSD"},{"line_number":2,"context_line":"openstackdocstheme\u003e\u003d2.2.6 # Apache-2.0"},{"line_number":3,"context_line":"sphinxcontrib-httpdomain\u003e\u003d1.7.0 # BSD"},{"line_number":4,"context_line":"sphinxcontrib-pecanwsme\u003e\u003d0.10.0 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":15,"id":"787f3f32_3740d6c8","side":"PARENT","line":1,"range":{"start_line":1,"start_character":0,"end_line":1,"end_character":15},"updated":"2023-09-12 09:29:54.000000000","message":"Are you guys willing to wait until we properly test and review the whole patch or would it be better to move this part to a separate patch we could merge in easily?","commit_id":"94630b97cd1fb4bdd9a638070ffbbe3625de8aa2"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"f33d6c7f16e050453d142cd8216c85e0bbbc6c5b","unresolved":false,"context_lines":[{"line_number":1,"context_line":"sphinx\u003e\u003d3.3.1,\u003c5.0.0 # BSD"},{"line_number":2,"context_line":"openstackdocstheme\u003e\u003d2.2.6 # Apache-2.0"},{"line_number":3,"context_line":"sphinxcontrib-httpdomain\u003e\u003d1.7.0 # BSD"},{"line_number":4,"context_line":"sphinxcontrib-pecanwsme\u003e\u003d0.10.0 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":15,"id":"90b66efd_f00787ea","side":"PARENT","line":1,"range":{"start_line":1,"start_character":0,"end_line":1,"end_character":15},"in_reply_to":"787f3f32_3740d6c8","updated":"2023-09-12 10:00:31.000000000","message":"Done. Can you check the patch: https://review.opendev.org/c/openstack/cloudkitty/+/894629?","commit_id":"94630b97cd1fb4bdd9a638070ffbbe3625de8aa2"}],"releasenotes/notes/support-group-by-timeframes-1247aa336916f3b6.yaml":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"5dc86ad073bb631498fbb81f94fb1e4d258b25a1","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":"    Introduce new default groupby options: (i) time: to group data by hourly."},{"line_number":5,"context_line":"    The actual group by process will depend on the ``period`` parameter. The"},{"line_number":6,"context_line":"    default value is ``3600``, which represents one hour; (ii) time-d: to"},{"line_number":7,"context_line":"    group data by day of the year; (iii) time-w: to group data by week of"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"b2586de8_8c1d0853","line":4,"range":{"start_line":4,"start_character":67,"end_line":4,"end_character":76},"updated":"2023-10-02 14:20:23.000000000","message":"\"by hour\" or \"hourly\"","commit_id":"2733b0380e573aa34939fed242c1f79251d54b92"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"22d15a18ff85c3574075fc7a45339cd491acb5af","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Introduce new default groupby options: (i) time: to group data by hourly."},{"line_number":5,"context_line":"    The actual group by process will depend on the ``period`` parameter. The"},{"line_number":6,"context_line":"    default value is ``3600``, which represents one hour; (ii) time-d: to"},{"line_number":7,"context_line":"    group data by day of the year; (iii) time-w: to group data by week of"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"0f99c57c_d8529d1a","line":4,"range":{"start_line":4,"start_character":67,"end_line":4,"end_character":76},"in_reply_to":"b2586de8_8c1d0853","updated":"2023-10-02 14:27:38.000000000","message":"Done","commit_id":"2733b0380e573aa34939fed242c1f79251d54b92"}]}