)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":31721,"name":"Felipe Rodrigues","email":"felipefuty01@gmail.com","username":"felipefutty"},"change_message_id":"707ef6cc80bffba7dc882a7f95eb5c2b25862ec3","unresolved":true,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Signed-off-by: Goutham Pacha Ravi \u003cgouthampravi@gmail.com\u003e"},{"line_number":29,"context_line":"Change-Id: I5d3c1041738214ec8382edfd5494c10ff0be351a"},{"line_number":30,"context_line":"Signed-off-by: Goutham Pacha Ravi \u003cgouthampravi@gmail.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"f1e00a26_0f1d04f8","line":30,"range":{"start_line":30,"start_character":1,"end_line":30,"end_character":58},"updated":"2022-09-15 20:42:27.000000000","message":"repeated name","commit_id":"755a1503187a29f9b4f6ecbf369acb781c3e95e7"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"11710272687e6aab2557da085c3d119a88e19684","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"90dc0e12_c2922a96","updated":"2022-09-14 20:11:56.000000000","message":"recheck\n\nfailures are unrelated","commit_id":"fbe1124b70ef6675988d8500401dbbf3f0893135"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"2f60df4c98400d4562cc2e0aad177028d6e7d470","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"51df4ba7_9a6660ae","updated":"2022-09-15 14:23:40.000000000","message":"Thank you for working on this change. It is looking good, only found one possible issue. Please take a look at the comments inline :)","commit_id":"1e71c1cc3ab14200d55f5e70f1f01293e0fca7b4"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"f50580d86e4d8ffdd7061b4401e76e3b811f4e3f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"c4d2d9c4_bad0b012","updated":"2022-09-15 20:27:45.000000000","message":"Can\u0027t spot anything wrong on this change. LGTM, thanks","commit_id":"755a1503187a29f9b4f6ecbf369acb781c3e95e7"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"9f9ea0cda7be20bacdf77e44474663a7e3ce20eb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"f99ce5e1_67ab9ea2","updated":"2022-09-15 17:52:48.000000000","message":"LGTM, thank you for the changes :)","commit_id":"755a1503187a29f9b4f6ecbf369acb781c3e95e7"},{"author":{"_account_id":31721,"name":"Felipe Rodrigues","email":"felipefuty01@gmail.com","username":"felipefutty"},"change_message_id":"707ef6cc80bffba7dc882a7f95eb5c2b25862ec3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"6bc641f3_8a2edcbe","updated":"2022-09-15 20:42:27.000000000","message":"LGTM, thanks Goutham!","commit_id":"755a1503187a29f9b4f6ecbf369acb781c3e95e7"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"ff0df4f6aedf1731663a10d167d3742e06dfb4ca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"b9f090aa_a4a5b812","updated":"2022-09-15 21:52:12.000000000","message":"Thank you for pursuing this, Goutham!","commit_id":"755a1503187a29f9b4f6ecbf369acb781c3e95e7"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"5ca6c90f1909c4c67daf23004bb7913f7f3c2237","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"3ed9eb43_903a3be8","updated":"2022-09-15 15:36:29.000000000","message":"Thanks Carlos - good catch indeed!","commit_id":"755a1503187a29f9b4f6ecbf369acb781c3e95e7"}],"manila/policies/share_snapshot.py":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"2f60df4c98400d4562cc2e0aad177028d6e7d470","unresolved":true,"context_lines":[{"line_number":254,"context_line":"    ),"},{"line_number":255,"context_line":"    policy.DocumentedRuleDefault("},{"line_number":256,"context_line":"        name\u003dBASE_POLICY_NAME % \u0027get_metadata\u0027,"},{"line_number":257,"context_line":"        check_str\u003dbase.ADMIN_OR_PROJECT_MEMBER,"},{"line_number":258,"context_line":"        scope_types\u003d[\u0027system\u0027, \u0027project\u0027],"},{"line_number":259,"context_line":"        description\u003d\"Get snapshot metadata.\","},{"line_number":260,"context_line":"        operations\u003d["}],"source_content_type":"text/x-python","patch_set":6,"id":"71ef6b48_a11e0b49","line":257,"range":{"start_line":257,"start_character":23,"end_line":257,"end_character":46},"updated":"2022-09-15 14:23:40.000000000","message":"this is get_metadata... isn\u0027t it supposed to follow the same behavior as before, but change only from system_or_project_reader to admin_or_project_reader?\nThis would have the same behavior as get_snapshots and get_all_snapshots","commit_id":"1e71c1cc3ab14200d55f5e70f1f01293e0fca7b4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"5ca6c90f1909c4c67daf23004bb7913f7f3c2237","unresolved":false,"context_lines":[{"line_number":254,"context_line":"    ),"},{"line_number":255,"context_line":"    policy.DocumentedRuleDefault("},{"line_number":256,"context_line":"        name\u003dBASE_POLICY_NAME % \u0027get_metadata\u0027,"},{"line_number":257,"context_line":"        check_str\u003dbase.ADMIN_OR_PROJECT_MEMBER,"},{"line_number":258,"context_line":"        scope_types\u003d[\u0027system\u0027, \u0027project\u0027],"},{"line_number":259,"context_line":"        description\u003d\"Get snapshot metadata.\","},{"line_number":260,"context_line":"        operations\u003d["}],"source_content_type":"text/x-python","patch_set":6,"id":"385a1d56_4c6a0d3f","line":257,"range":{"start_line":257,"start_character":23,"end_line":257,"end_character":46},"in_reply_to":"71ef6b48_a11e0b49","updated":"2022-09-15 15:36:29.000000000","message":"good catch! yes! thank you","commit_id":"1e71c1cc3ab14200d55f5e70f1f01293e0fca7b4"}],"releasenotes/notes/zed-secure-rbac-direction-change-2329bbf442b9a2da.yaml":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"2f60df4c98400d4562cc2e0aad177028d6e7d470","unresolved":true,"context_lines":[{"line_number":2,"context_line":"prelude: \u003e"},{"line_number":3,"context_line":"    RBAC defaults of all Shared File System service (manila) APIs have been"},{"line_number":4,"context_line":"    updated to remove \"system\" scope personas. This is being done in concert"},{"line_number":5,"context_line":"    with other OpenStack services and in-reaction to operator feedback that"},{"line_number":6,"context_line":"    the use of system \"scope\" introduces backwards incompatibility in existing"},{"line_number":7,"context_line":"    workflows. The new defaults support the use of \"scope\", however, no RBAC"},{"line_number":8,"context_line":"    rule by default includes \"system\" scope. At this time, we do not recommend"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"5ba45803_9c28e665","line":5,"range":{"start_line":5,"start_character":32,"end_line":5,"end_character":35},"updated":"2022-09-15 14:23:40.000000000","message":"nit: maybe add a comma here","commit_id":"1e71c1cc3ab14200d55f5e70f1f01293e0fca7b4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"5ca6c90f1909c4c67daf23004bb7913f7f3c2237","unresolved":false,"context_lines":[{"line_number":2,"context_line":"prelude: \u003e"},{"line_number":3,"context_line":"    RBAC defaults of all Shared File System service (manila) APIs have been"},{"line_number":4,"context_line":"    updated to remove \"system\" scope personas. This is being done in concert"},{"line_number":5,"context_line":"    with other OpenStack services and in-reaction to operator feedback that"},{"line_number":6,"context_line":"    the use of system \"scope\" introduces backwards incompatibility in existing"},{"line_number":7,"context_line":"    workflows. The new defaults support the use of \"scope\", however, no RBAC"},{"line_number":8,"context_line":"    rule by default includes \"system\" scope. At this time, we do not recommend"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"bba838a5_817a94bc","line":5,"range":{"start_line":5,"start_character":32,"end_line":5,"end_character":35},"in_reply_to":"5ba45803_9c28e665","updated":"2022-09-15 15:36:29.000000000","message":"Done","commit_id":"1e71c1cc3ab14200d55f5e70f1f01293e0fca7b4"}]}
