)]}'
{"docker/kolla-toolbox/kolla_keystone_user.py":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"6ff50e87773073527d22e98a01b4a98711c366cb","unresolved":false,"context_lines":[{"line_number":75,"context_line":"        if not assignment:"},{"line_number":76,"context_line":"            cloud.grant_role(role,"},{"line_number":77,"context_line":"                             user\u003duser,"},{"line_number":78,"context_line":"                             project\u003dproject)"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"        module.exit_json(changed\u003dchanged)"},{"line_number":81,"context_line":"    except Exception:"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_3291e354","line":78,"updated":"2018-09-11 14:47:06.000000000","message":"If the role has changed, don\u0027t we need to revoke the old role?","commit_id":"1b83b78e7a37dd38c38fd171671c2a47883bfa18"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"7d81a47474d4bd7df3a5cc2782a66a6e448ec1a5","unresolved":false,"context_lines":[{"line_number":75,"context_line":"        if not assignment:"},{"line_number":76,"context_line":"            cloud.grant_role(role,"},{"line_number":77,"context_line":"                             user\u003duser,"},{"line_number":78,"context_line":"                             project\u003dproject)"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"        module.exit_json(changed\u003dchanged)"},{"line_number":81,"context_line":"    except Exception:"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_0a54a1fd","line":78,"updated":"2018-09-12 03:00:10.000000000","message":"i do not think so. because we do not know the old role, and the role may also be assignmented by operator manually.","commit_id":"1b83b78e7a37dd38c38fd171671c2a47883bfa18"}]}