)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"ff0ce5a07ad80f32556dd0c2c8680ec627006408","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e2695f27_c85b0536","updated":"2025-10-24 22:19:17.000000000","message":"Hi Riane,\n\nIgnore the CI failure here because grenade jobs are broken for an unrelated reason.. but, the schema you added needs to be used in a validation, for example, see the schema for availability zones here [1], and its validation here [2]\n\n[1] https://github.com/openstack/manila/blob/8ae12adc99ffd6977036a476f032d4957653bbc2/manila/api/schemas/availability_zones.py#L13-L19\n[2] https://github.com/openstack/manila/blob/8ae12adc99ffd6977036a476f032d4957653bbc2/manila/api/v2/availability_zones.py#L49-L50","commit_id":"b0763bc17469aec5fb226a702de80dca954fc7d4"},{"author":{"_account_id":38038,"name":"Riane TOrres","display_name":"Riane Torres","email":"riane.torres@gmail.com","username":"realrtorres"},"change_message_id":"0a5693e107a2cec2842a17e79826b5312d5c5016","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2340f989_3b99365d","updated":"2025-11-14 00:10:16.000000000","message":"Resolved comments","commit_id":"75e8f148e2d40249adc62208167e24b49645a408"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"2d7ee9c10f1fb089f1eb71ff3e34c82e2197ca29","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"fa5d764e_ad795aed","updated":"2026-01-26 19:24:42.000000000","message":"Thanks for working on this @riane.torres@gmail.com","commit_id":"75e8f148e2d40249adc62208167e24b49645a408"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"860ab32ba2c48aa4865118a31f1d5bdf9e7eb347","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7059fefe_c50cb925","updated":"2026-03-05 18:57:00.000000000","message":"LGTM, thanks","commit_id":"a5e675c38e6dcb288f953c2ae88067fbf5a5c1c8"}],"manila/api/schemas/services.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"ff0ce5a07ad80f32556dd0c2c8680ec627006408","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"408faec5_86b2e0fd","line":37,"updated":"2025-10-24 22:19:17.000000000","message":"Could you also add the response schema? (it\u0027s null, but, see example here: https://github.com/openstack/manila/blob/8ae12adc99ffd6977036a476f032d4957653bbc2/manila/api/schemas/resource_locks.py#L326-L328)","commit_id":"b0763bc17469aec5fb226a702de80dca954fc7d4"},{"author":{"_account_id":38038,"name":"Riane TOrres","display_name":"Riane Torres","email":"riane.torres@gmail.com","username":"realrtorres"},"change_message_id":"0a5693e107a2cec2842a17e79826b5312d5c5016","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c89e4131_079345ea","line":37,"in_reply_to":"408faec5_86b2e0fd","updated":"2025-11-14 00:10:16.000000000","message":"Added it, but I accidentally made a whole new commit here https://review.opendev.org/c/openstack/manila/+/966375 and forgot to amend this commit.","commit_id":"b0763bc17469aec5fb226a702de80dca954fc7d4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"8c4dfe70f06b2eff2a8ad8e949882d799a7c5351","unresolved":true,"context_lines":[{"line_number":10,"context_line":"# License for the specific language governing permissions and limitations"},{"line_number":11,"context_line":"# under the License."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"from oslo_config import cfg"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"CONF \u003d cfg.CONF"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"ensure_shares_request_body \u003d {"}],"source_content_type":"text/x-python","patch_set":2,"id":"ac92f425_6b0e4295","line":15,"range":{"start_line":13,"start_character":0,"end_line":15,"end_character":15},"updated":"2026-01-26 19:22:12.000000000","message":"unused","commit_id":"75e8f148e2d40249adc62208167e24b49645a408"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"2d7ee9c10f1fb089f1eb71ff3e34c82e2197ca29","unresolved":false,"context_lines":[{"line_number":10,"context_line":"# License for the specific language governing permissions and limitations"},{"line_number":11,"context_line":"# under the License."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"from oslo_config import cfg"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"CONF \u003d cfg.CONF"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"ensure_shares_request_body \u003d {"}],"source_content_type":"text/x-python","patch_set":2,"id":"8ef77335_5b822d32","line":15,"range":{"start_line":13,"start_character":0,"end_line":15,"end_character":15},"in_reply_to":"ac92f425_6b0e4295","updated":"2026-01-26 19:24:42.000000000","message":"Done","commit_id":"75e8f148e2d40249adc62208167e24b49645a408"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"8c4dfe70f06b2eff2a8ad8e949882d799a7c5351","unresolved":true,"context_lines":[{"line_number":23,"context_line":"            \u0027properties\u0027: {"},{"line_number":24,"context_line":"                \u0027host\u0027: {"},{"line_number":25,"context_line":"                    \u0027type\u0027: \u0027string\u0027,"},{"line_number":26,"context_line":"                    \u0027description\u0027: \u0027Backend host to ensure shares on (e.g. \u0027"},{"line_number":27,"context_line":"                                   \u0027openstackhost@generic)\u0027"},{"line_number":28,"context_line":"                }"},{"line_number":29,"context_line":"            },"},{"line_number":30,"context_line":"            \u0027required\u0027: [\u0027host\u0027],"}],"source_content_type":"text/x-python","patch_set":2,"id":"06afef97_35d15363","line":27,"range":{"start_line":26,"start_character":35,"end_line":27,"end_character":59},"updated":"2026-01-26 19:22:12.000000000","message":"can be referenced from parameters.yaml","commit_id":"75e8f148e2d40249adc62208167e24b49645a408"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"2d7ee9c10f1fb089f1eb71ff3e34c82e2197ca29","unresolved":false,"context_lines":[{"line_number":23,"context_line":"            \u0027properties\u0027: {"},{"line_number":24,"context_line":"                \u0027host\u0027: {"},{"line_number":25,"context_line":"                    \u0027type\u0027: \u0027string\u0027,"},{"line_number":26,"context_line":"                    \u0027description\u0027: \u0027Backend host to ensure shares on (e.g. \u0027"},{"line_number":27,"context_line":"                                   \u0027openstackhost@generic)\u0027"},{"line_number":28,"context_line":"                }"},{"line_number":29,"context_line":"            },"},{"line_number":30,"context_line":"            \u0027required\u0027: [\u0027host\u0027],"}],"source_content_type":"text/x-python","patch_set":2,"id":"6ece1518_d01c1320","line":27,"range":{"start_line":26,"start_character":35,"end_line":27,"end_character":59},"in_reply_to":"06afef97_35d15363","updated":"2026-01-26 19:24:42.000000000","message":"Done","commit_id":"75e8f148e2d40249adc62208167e24b49645a408"}]}