)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32411,"name":"Eduardo Santos","email":"eduardo.experimental@gmail.com","username":"ecsantos"},"change_message_id":"8bed5b3f20c1e0636148b03467c15d9b681b84c5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5d773284_4b035797","updated":"2022-05-19 18:21:42.000000000","message":"Thanks for working on this OSC implementation, Ashley!\n\nThe code looks good to me, I didn\u0027t really spot anything that could be improved.\n\nIf I were to give a suggestion, it\u0027d be to use template strings [1] instead of the printf-style string formatting [2], since the official Python documentation prefers f-strings and I noticed some other hackathon changes are using them. [3][4][5]\n\n[1] https://docs.python.org/3/library/string.html#template-strings\n[2] https://docs.python.org/3/library/stdtypes.html#printf-style-string-formatting\n[3] https://review.opendev.org/c/openstack/python-manilaclient/+/836834\n[4] https://review.opendev.org/c/openstack/python-manilaclient/+/841993\n[5] https://review.opendev.org/c/openstack/python-manilaclient/+/842225","commit_id":"a294853195544798baaa1b3027d61b9a51e78306"},{"author":{"_account_id":30025,"name":"Vida Haririan","display_name":"Vida Haririan","email":"vhariria@redhat.com","username":"vhari"},"change_message_id":"be271dfef2900617c7977b4fbc3c4162d2903b7a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"c82f92d8_479599f6","updated":"2022-06-03 17:52:23.000000000","message":"Test validation, looks good thanks.","commit_id":"452ce0df1f0c2f710e5de45069036d5775639805"},{"author":{"_account_id":31213,"name":"Maari Tamm","email":"maari.tamm@citynetwork.eu","username":"maaritamm"},"change_message_id":"abdf6139c88c1b72ad7bb769a5a695dd9cbf21e9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"7ecd8ece_2c5e2fbb","updated":"2022-05-27 18:35:02.000000000","message":"Thanks for working on this, Ashley!\nI\u0027d probably use the json format output for the \u0027show\u0027 commands after set/unset operations to better assert the keys/values in the properties. But that\u0027s more refactoring and could be just my personal preference and not a blocker for merging this :) \nThanks again!","commit_id":"452ce0df1f0c2f710e5de45069036d5775639805"},{"author":{"_account_id":31213,"name":"Maari Tamm","email":"maari.tamm@citynetwork.eu","username":"maaritamm"},"change_message_id":"9e671979bfbd535c4b9cff1e61b3a6e30a2fbef4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"3bba16d9_46c0aeac","updated":"2022-06-09 09:07:16.000000000","message":"Thanks Ashley!","commit_id":"fdac503cadd01d275f30f4dbfcd42ac46db5d7d0"},{"author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"change_message_id":"20dc3260c1c8f28c43d8c9cd4264bcb0ee4e1e99","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"401e9b8c_e05a9de9","updated":"2022-06-10 16:38:53.000000000","message":"Thanks Ashley, looks good!","commit_id":"fdac503cadd01d275f30f4dbfcd42ac46db5d7d0"}],"manilaclient/api_versions.py":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"90ee8f672a5e3e0c57b0cd36b39041a512e8f409","unresolved":true,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"MAX_VERSION \u003d \u00272.71\u0027"},{"line_number":31,"context_line":"MIN_VERSION \u003d \u00272.0\u0027"},{"line_number":32,"context_line":"DEPRECATED_VERSION \u003d \u00271.0\u0027"},{"line_number":33,"context_line":"_VERSIONED_METHOD_MAP \u003d {}"}],"source_content_type":"text/x-python","patch_set":5,"id":"76fd5ab5_e2e215b1","line":30,"range":{"start_line":30,"start_character":0,"end_line":30,"end_character":20},"updated":"2022-05-31 20:05:01.000000000","message":"I\u0027m unsure if we should really bump the version on this change. We could use a separate one for doing so...","commit_id":"452ce0df1f0c2f710e5de45069036d5775639805"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"695b3bdd71e2ba903aa751e5d25a02e1424c09fb","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"MAX_VERSION \u003d \u00272.71\u0027"},{"line_number":31,"context_line":"MIN_VERSION \u003d \u00272.0\u0027"},{"line_number":32,"context_line":"DEPRECATED_VERSION \u003d \u00271.0\u0027"},{"line_number":33,"context_line":"_VERSIONED_METHOD_MAP \u003d {}"}],"source_content_type":"text/x-python","patch_set":5,"id":"73dc33be_ba95315d","line":30,"range":{"start_line":30,"start_character":0,"end_line":30,"end_character":20},"in_reply_to":"76fd5ab5_e2e215b1","updated":"2022-06-06 16:54:11.000000000","message":"Sure thing, I\u0027ll roll back.","commit_id":"452ce0df1f0c2f710e5de45069036d5775639805"}]}