)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"1171bdca68c225cc054ca09820ffac83b7c3de00","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4278abcb_06fd8b9a","updated":"2025-08-25 06:33:49.000000000","message":"a test would be nice.","commit_id":"2cb4522d8126f3ce1baf521b85212147eb1444f3"},{"author":{"_account_id":37598,"name":"Ivan Anfimov","display_name":"Ivan Anfimov","email":"lazekteam@gmail.com","username":"anfimovir"},"change_message_id":"f7c30ec6be4c2ed21e5636f4ebd967b307b9ef61","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"7d6036af_8d4851ef","updated":"2025-08-27 15:27:02.000000000","message":"@leonie@stackhpc.com hello, localization file not need, translation provide via Zanata after merge code ;) please remove it.","commit_id":"193cf34b7db83e7517438ddbacde0012118d0cc1"},{"author":{"_account_id":38288,"name":"Leonie Chamberlin-Medd","display_name":"Leonie Chamberlin-Medd","email":"leonie@stackhpc.com","username":"LChams"},"change_message_id":"7e25049b6c238923cc89f16b64935655524b16e8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"33b1565d_18bd8462","in_reply_to":"7d6036af_8d4851ef","updated":"2025-08-28 06:50:21.000000000","message":"Sorted, thanks for letting me know!","commit_id":"193cf34b7db83e7517438ddbacde0012118d0cc1"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"0e322cbff99b93f8b5d50867e02cff981a21c593","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"3a8d3dab_05626ef1","updated":"2025-09-03 07:03:49.000000000","message":"Would you be able to come up with a test that, so that another change would not unintentionally destroy the desired outcome of this change here?","commit_id":"4f13282c811b2c7d778cf81a9a9172318c2ee58b"},{"author":{"_account_id":38288,"name":"Leonie Chamberlin-Medd","display_name":"Leonie Chamberlin-Medd","email":"leonie@stackhpc.com","username":"LChams"},"change_message_id":"877ab7a796925ab5810549e01037d7486dcaa610","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"b0ddedaf_4b1e44e9","in_reply_to":"3a8d3dab_05626ef1","updated":"2025-09-04 10:07:58.000000000","message":"Hi Matthias, \n\nPlease could you provide some clarification on what exactly the tests could entail? It changes existing API calls so I\u0027m not sure what tests would be useful to add. \n\nThanks","commit_id":"4f13282c811b2c7d778cf81a9a9172318c2ee58b"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"0e3258788a9f08a527d249d2339f7c7835b86975","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"f6954cdf_c5db0185","in_reply_to":"b0ddedaf_4b1e44e9","updated":"2025-09-04 10:25:31.000000000","message":"Right. And you\u0027d want to make sure that these API calls stay as they are, no? You are doing this change for a reason, and would want to catch if a change made by you or someone else accidentally breaks what you implemented.\n\nFor reference, horizon has e.g this (and more) tests to actually check if the returned data is what it is supposed to look like https://github.com/openstack/horizon/blob/master/openstack_dashboard/test/integration_tests/pages/admin/volume/grouptypespage.py\n\n\nI get you, the tests for cloudkitty dashboard are not as extensive as they are in horizon.\nIt may be a bit too much to ask to add tests like the ones with horizon though, and I would not block this change due to missing tests.","commit_id":"4f13282c811b2c7d778cf81a9a9172318c2ee58b"},{"author":{"_account_id":38288,"name":"Leonie Chamberlin-Medd","display_name":"Leonie Chamberlin-Medd","email":"leonie@stackhpc.com","username":"LChams"},"change_message_id":"8d53156763be619e810e6c6e0806e64e638ee569","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"a22efd6a_df8d3d6e","in_reply_to":"f6954cdf_c5db0185","updated":"2025-09-05 09:30:11.000000000","message":"I\u0027m with you on this. I think this is something that would be good to add in a follow-up patch. Thank you","commit_id":"4f13282c811b2c7d778cf81a9a9172318c2ee58b"},{"author":{"_account_id":38288,"name":"Leonie Chamberlin-Medd","display_name":"Leonie Chamberlin-Medd","email":"leonie@stackhpc.com","username":"LChams"},"change_message_id":"ffedb52773073f2e6748e9857738767ebfa7a9c5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"95d8489a_ee80d4a2","updated":"2025-11-27 14:33:53.000000000","message":"Will sort out relation chain and a potential mistake I made in resolving the merge conflict ASAP, sorry for the delay","commit_id":"0ef7470cd14e4632d9a5c3af0ab9659ec34303d8"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"461885c29dd65b06793cdfee78229ca31a9be59a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"d024f3d9_1a7e9afb","updated":"2026-02-16 14:30:26.000000000","message":"We need a release note for this one, right?\n\nAlso, if the backend is not using V2, they would have an impact, and the panel would not work anymore.","commit_id":"2de67613bbce72b8525e4114a849ad671d530c16"},{"author":{"_account_id":38288,"name":"Leonie Chamberlin-Medd","display_name":"Leonie Chamberlin-Medd","email":"leonie@stackhpc.com","username":"LChams"},"change_message_id":"31778b7df36ce805b01fa514980d5453d1382ce7","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"db4e0cd4_d2efba2c","in_reply_to":"695ede88_baeb027c","updated":"2026-02-18 11:01:39.000000000","message":"Also worth noting that the project/rating panel already uses the v2 API for the data retrieval so this matches that.","commit_id":"2de67613bbce72b8525e4114a849ad671d530c16"},{"author":{"_account_id":38288,"name":"Leonie Chamberlin-Medd","display_name":"Leonie Chamberlin-Medd","email":"leonie@stackhpc.com","username":"LChams"},"change_message_id":"9b2fcb174be6ebf558c79984d5f40e014f1f4efa","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"695ede88_baeb027c","in_reply_to":"d024f3d9_1a7e9afb","updated":"2026-02-18 10:32:53.000000000","message":"Hi Rafael,\n\nYes I agree! Adding a release note is on my to-do list, should get it done soon. \n\nIt would be important to note that those using the Gazpacho release need to upgrade to V2 in the release notes. Let me know what you think. \n\nThanks!","commit_id":"2de67613bbce72b8525e4114a849ad671d530c16"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"83524afc8fcb4fc0be91f085fd500c2fb7483f70","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"dbad3d91_e7299fc8","updated":"2026-03-02 14:25:55.000000000","message":"still looks good to me","commit_id":"8af1dbba1ea1922f7481f29510bc9db72ec1b68e"}],"cloudkittydashboard/dashboards/admin/summary/tables.py":[{"author":{"_account_id":37598,"name":"Ivan Anfimov","display_name":"Ivan Anfimov","email":"lazekteam@gmail.com","username":"anfimovir"},"change_message_id":"2ea2b193737e37426a40f94e61adaf6a34afa5bc","unresolved":true,"context_lines":[{"line_number":37,"context_line":""},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"class TenantSummaryTable(tables.DataTable):"},{"line_number":40,"context_line":"    res_type \u003d tables.Column(\u0027type\u0027, verbose_name\u003d_(\"Res Type\"))"},{"line_number":41,"context_line":"    rate \u003d tables.Column(\u0027rate\u0027, verbose_name\u003d_(\"Rate\"))"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    class Meta(object):"}],"source_content_type":"text/x-python","patch_set":3,"id":"0a0a11b6_2db9c32b","line":40,"updated":"2025-08-25 00:43:42.000000000","message":"```suggestion\n    res_type \u003d tables.Column(\u0027type\u0027, verbose_name\u003d_(\"Result Type\"))\n```\n\nHello, may be this more friendly.","commit_id":"2cb4522d8126f3ce1baf521b85212147eb1444f3"},{"author":{"_account_id":38288,"name":"Leonie Chamberlin-Medd","display_name":"Leonie Chamberlin-Medd","email":"leonie@stackhpc.com","username":"LChams"},"change_message_id":"371e2d8a85ae3ed286bb56cef3e008c5b71bdda3","unresolved":false,"context_lines":[{"line_number":37,"context_line":""},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"class TenantSummaryTable(tables.DataTable):"},{"line_number":40,"context_line":"    res_type \u003d tables.Column(\u0027type\u0027, verbose_name\u003d_(\"Res Type\"))"},{"line_number":41,"context_line":"    rate \u003d tables.Column(\u0027rate\u0027, verbose_name\u003d_(\"Rate\"))"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    class Meta(object):"}],"source_content_type":"text/x-python","patch_set":3,"id":"ff61a913_cae5cd3b","line":40,"in_reply_to":"0a0a11b6_2db9c32b","updated":"2025-08-27 14:47:09.000000000","message":"Hi Ivan, I agree this is friendlier, I have changed it to Resource Type.","commit_id":"2cb4522d8126f3ce1baf521b85212147eb1444f3"}],"cloudkittydashboard/dashboards/admin/summary/views.py":[{"author":{"_account_id":37598,"name":"Ivan Anfimov","display_name":"Ivan Anfimov","email":"lazekteam@gmail.com","username":"anfimovir"},"change_message_id":"2ea2b193737e37426a40f94e61adaf6a34afa5bc","unresolved":true,"context_lines":[{"line_number":53,"context_line":"            tenant[\u0027name\u0027] \u003d tenants.get(tenant.id, \u0027-\u0027)"},{"line_number":54,"context_line":"            tenant[\u0027rate\u0027] \u003d utils.formatRate(tenant[\u0027rate\u0027],"},{"line_number":55,"context_line":"                                              rate_prefix, rate_postfix)"},{"line_number":56,"context_line":"            data[-1][\u0027name\u0027] \u003d \u0027Cloud Total\u0027"},{"line_number":57,"context_line":"        return data"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"b64a1992_b0bfa472","line":56,"updated":"2025-08-25 00:43:42.000000000","message":"May be add in translation system?","commit_id":"2cb4522d8126f3ce1baf521b85212147eb1444f3"},{"author":{"_account_id":38288,"name":"Leonie Chamberlin-Medd","display_name":"Leonie Chamberlin-Medd","email":"leonie@stackhpc.com","username":"LChams"},"change_message_id":"371e2d8a85ae3ed286bb56cef3e008c5b71bdda3","unresolved":false,"context_lines":[{"line_number":53,"context_line":"            tenant[\u0027name\u0027] \u003d tenants.get(tenant.id, \u0027-\u0027)"},{"line_number":54,"context_line":"            tenant[\u0027rate\u0027] \u003d utils.formatRate(tenant[\u0027rate\u0027],"},{"line_number":55,"context_line":"                                              rate_prefix, rate_postfix)"},{"line_number":56,"context_line":"            data[-1][\u0027name\u0027] \u003d \u0027Cloud Total\u0027"},{"line_number":57,"context_line":"        return data"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"92f99446_a1297bbc","line":56,"in_reply_to":"b64a1992_b0bfa472","updated":"2025-08-27 14:47:09.000000000","message":"Done","commit_id":"2cb4522d8126f3ce1baf521b85212147eb1444f3"}]}
