)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"36e82604ff27aba3b8f317eb3d57584ba6c755fa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"aa52d6bf_16b5689b","updated":"2021-12-03 09:15:51.000000000","message":"LGTM","commit_id":"6943b31697779d2bd86fb7c0cfb4f099b14cb20b"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"630d556b2a9ec9357bc4dbfa1f325d3c79e8ab8d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"698aa5be_3519e9a1","updated":"2021-12-03 13:29:57.000000000","message":"LGTM. This will unblock our CI and switch our setup to the canonical approach.\nI have one question inline though. If the content of that comment is an issue, I think we can address in the next PS, which will be after the share group type change has been merged.","commit_id":"6943b31697779d2bd86fb7c0cfb4f099b14cb20b"}],"devstack/plugin.sh":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"630d556b2a9ec9357bc4dbfa1f325d3c79e8ab8d","unresolved":true,"context_lines":[{"line_number":414,"context_line":"        local flavor_exists\u003d$( openstack --os-cloud devstack-admin flavor list | grep \" $MANILA_SERVICE_VM_FLAVOR_NAME \" )"},{"line_number":415,"context_line":"        if [[ -z $flavor_exists ]]; then"},{"line_number":416,"context_line":"            # Create flavor for Manila\u0027s service VM"},{"line_number":417,"context_line":"            openstack --os-cloud devstack-admin --os-cloud devstack-admin flavor create \\"},{"line_number":418,"context_line":"                $MANILA_SERVICE_VM_FLAVOR_NAME \\"},{"line_number":419,"context_line":"                --id $MANILA_SERVICE_VM_FLAVOR_REF \\"},{"line_number":420,"context_line":"                --ram $MANILA_SERVICE_VM_FLAVOR_RAM \\"}],"source_content_type":"text/x-sh","patch_set":3,"id":"3cf427c3_8f76ab12","line":417,"range":{"start_line":417,"start_character":22,"end_line":417,"end_character":73},"updated":"2021-12-03 13:29:57.000000000","message":"Just out of curiosity: is there a special need to set this twice?","commit_id":"6943b31697779d2bd86fb7c0cfb4f099b14cb20b"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"3a161cc0dd68b8e70c2eeb322f9adec6599296c6","unresolved":true,"context_lines":[{"line_number":414,"context_line":"        local flavor_exists\u003d$( openstack --os-cloud devstack-admin flavor list | grep \" $MANILA_SERVICE_VM_FLAVOR_NAME \" )"},{"line_number":415,"context_line":"        if [[ -z $flavor_exists ]]; then"},{"line_number":416,"context_line":"            # Create flavor for Manila\u0027s service VM"},{"line_number":417,"context_line":"            openstack --os-cloud devstack-admin --os-cloud devstack-admin flavor create \\"},{"line_number":418,"context_line":"                $MANILA_SERVICE_VM_FLAVOR_NAME \\"},{"line_number":419,"context_line":"                --id $MANILA_SERVICE_VM_FLAVOR_REF \\"},{"line_number":420,"context_line":"                --ram $MANILA_SERVICE_VM_FLAVOR_RAM \\"}],"source_content_type":"text/x-sh","patch_set":3,"id":"8e0fd5b4_1b96d361","line":417,"range":{"start_line":417,"start_character":22,"end_line":417,"end_character":73},"in_reply_to":"3cf427c3_8f76ab12","updated":"2021-12-03 14:38:42.000000000","message":"nope, a copy pasta error. Will remove it along with the group type change.. thanks for calling it out!","commit_id":"6943b31697779d2bd86fb7c0cfb4f099b14cb20b"}]}
