)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"77754c50d19cd2f8d193736f66957d3a4eec2d7a","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"image: Support for stores_info"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Updated the ``stores()`` to make it work similar to glanceclient,"},{"line_number":10,"context_line":"also updated path for stores-info detail."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: If436e58d410e2b2670415ef962b5a7758103970b"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ffd8f7e5_8fe298cd","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":20},"updated":"2023-07-11 08:13:24.000000000","message":"Can you add/explain detail about the difference between two?","commit_id":"70eac2db2a9065d9c1fe21f4aeae57f5447f281f"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"c6bbc88a4875826e4f19497d3ae30a75f64cbd54","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Mridula Joshi \u003cmrjoshi@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2023-07-28 09:41:31 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"image: Support for stores_info"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Updated the ``stores()`` by updating base-path for"},{"line_number":10,"context_line":"stores-info detail."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"fba0da04_150b9e69","line":7,"range":{"start_line":7,"start_character":19,"end_line":7,"end_character":30},"updated":"2023-07-31 18:12:26.000000000","message":"Didn\u0027t we already have support for that? With the only issue being a bug (wrong URL)? This commit looks more like a bugfix than a new feature, unless I\u0027m mistaken.","commit_id":"d2398e5ea2da6f092cf9475c24a61559de169efd"},{"author":{"_account_id":33765,"name":"Mridula Joshi","email":"mrjoshi@redhat.com","username":"mrjoshi"},"change_message_id":"291f5869a892f46c3ab838d55d18293dcad55dbb","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Mridula Joshi \u003cmrjoshi@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2023-07-28 09:41:31 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"image: Support for stores_info"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Updated the ``stores()`` by updating base-path for"},{"line_number":10,"context_line":"stores-info detail."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"c8d5ab99_745fd145","line":7,"range":{"start_line":7,"start_character":19,"end_line":7,"end_character":30},"in_reply_to":"c5677d80_8ab26f0d","updated":"2023-08-01 12:36:53.000000000","message":"Ack","commit_id":"d2398e5ea2da6f092cf9475c24a61559de169efd"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"750dc72f1dde0d49cc89f3f3bc5c1add0267b196","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Mridula Joshi \u003cmrjoshi@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2023-07-28 09:41:31 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"image: Support for stores_info"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Updated the ``stores()`` by updating base-path for"},{"line_number":10,"context_line":"stores-info detail."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"c5677d80_8ab26f0d","line":7,"range":{"start_line":7,"start_character":19,"end_line":7,"end_character":30},"in_reply_to":"fba0da04_150b9e69","updated":"2023-08-01 10:15:20.000000000","message":"Yeah, the commit message needs a bit of a rework","commit_id":"d2398e5ea2da6f092cf9475c24a61559de169efd"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"c6bbc88a4875826e4f19497d3ae30a75f64cbd54","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Updated the ``stores()`` by updating base-path for"},{"line_number":10,"context_line":"stores-info detail."},{"line_number":11,"context_line":"It should be ``info/stores/details"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: If436e58d410e2b2670415ef962b5a7758103970b"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9b002783_9d500b73","line":11,"range":{"start_line":11,"start_character":27,"end_line":11,"end_character":34},"updated":"2023-07-31 18:12:26.000000000","message":"I think you mean \"detail\" (singular).","commit_id":"d2398e5ea2da6f092cf9475c24a61559de169efd"},{"author":{"_account_id":33765,"name":"Mridula Joshi","email":"mrjoshi@redhat.com","username":"mrjoshi"},"change_message_id":"291f5869a892f46c3ab838d55d18293dcad55dbb","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Updated the ``stores()`` by updating base-path for"},{"line_number":10,"context_line":"stores-info detail."},{"line_number":11,"context_line":"It should be ``info/stores/details"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: If436e58d410e2b2670415ef962b5a7758103970b"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"d62f2fdb_b305dbd7","line":11,"range":{"start_line":11,"start_character":27,"end_line":11,"end_character":34},"in_reply_to":"9b002783_9d500b73","updated":"2023-08-01 12:36:53.000000000","message":"yes, right","commit_id":"d2398e5ea2da6f092cf9475c24a61559de169efd"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"ceafc8bc348a8bc43b5ebb4df99fafab84943476","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d5bc0e2c_0d1d3ae5","updated":"2023-07-11 10:02:11.000000000","message":"This seems incorrect. The API returns a list of objects with a consistent format. That is by definition a list API. What you\u0027ve done here is already correct. This would break that.\n\nAlso, once again, let me restate that making things similar to a legacy client is a nice-to-have but absolutely *not* a goal. The goal is consistency across SDK (or OSC) APIs. Put another way, changing something to \"make it work similarly to other SDK resources\" makes sense, but changing it to \"make it work similarly to glanceclient\" (and by extension, making it work _differently_ to other SDK resources) is not.","commit_id":"70eac2db2a9065d9c1fe21f4aeae57f5447f281f"},{"author":{"_account_id":33765,"name":"Mridula Joshi","email":"mrjoshi@redhat.com","username":"mrjoshi"},"change_message_id":"c107ae815b9aa64d4ad81778a2d178e42cc47e58","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e6a12a81_399960c7","in_reply_to":"d5bc0e2c_0d1d3ae5","updated":"2023-07-28 10:09:52.000000000","message":"ack, corrected","commit_id":"70eac2db2a9065d9c1fe21f4aeae57f5447f281f"}],"openstack/image/v2/service_info.py":[{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"3cf11fc7794989ab762e9550f74474a35a50cc82","unresolved":true,"context_lines":[{"line_number":37,"context_line":"    #: default"},{"line_number":38,"context_line":"    is_default \u003d resource.Body(\u0027default\u0027, type\u003dbool)"},{"line_number":39,"context_line":"    #: properties"},{"line_number":40,"context_line":"    properties \u003d resource.Body(\u0027properties\u0027)"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    def delete_image(self, session, image, *, ignore_missing\u003dFalse):"},{"line_number":43,"context_line":"        \"\"\"Delete image from store"}],"source_content_type":"text/x-python","patch_set":4,"id":"4f81d18c_2d679420","line":40,"updated":"2023-08-09 09:09:57.000000000","message":"can you please add tests for new attribute? I think also you should set type\u003ddict to properly parse the content","commit_id":"8fd7f5c429417afdd123fe22d95aa257301966c1"}]}
