)]}'
{"tempest/cmd/account_generator.py":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"111a13c85958b4763e736c85ac6e107859aea025","unresolved":false,"context_lines":[{"line_number":163,"context_line":"        spec.append([CONF.object_storage.operator_role])"},{"line_number":164,"context_line":"        spec.append([CONF.object_storage.reseller_admin_role])"},{"line_number":165,"context_line":"    if admin:"},{"line_number":166,"context_line":"        spec.append(\u0027admin\u0027)"},{"line_number":167,"context_line":"    resources \u003d []"},{"line_number":168,"context_line":"    for cred_type in spec:"},{"line_number":169,"context_line":"        resources.append((cred_type, cred_provider.get_credentials("}],"source_content_type":"text/x-python","patch_set":2,"id":"3f79a3b5_a3143deb","line":166,"range":{"start_line":166,"start_character":20,"end_line":166,"end_character":27},"updated":"2018-10-01 13:14:02.000000000","message":"As an aside, unless there is something about special swift roles, this should be CONF.identity.admin_role.","commit_id":"a794e853042d10d9fc70d4ca3e8b3341b573aff1"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"96b2790b1fa86346c645fe1bb02d5a71f9939cc5","unresolved":false,"context_lines":[{"line_number":163,"context_line":"        spec.append([CONF.object_storage.operator_role])"},{"line_number":164,"context_line":"        spec.append([CONF.object_storage.reseller_admin_role])"},{"line_number":165,"context_line":"    if admin:"},{"line_number":166,"context_line":"        spec.append(\u0027admin\u0027)"},{"line_number":167,"context_line":"    resources \u003d []"},{"line_number":168,"context_line":"    for cred_type in spec:"},{"line_number":169,"context_line":"        resources.append((cred_type, cred_provider.get_credentials("}],"source_content_type":"text/x-python","patch_set":2,"id":"3f79a3b5_3acb604a","line":166,"range":{"start_line":166,"start_character":20,"end_line":166,"end_character":27},"in_reply_to":"3f79a3b5_5e2e5eab","updated":"2018-10-02 02:49:41.000000000","message":"Yes that would be nice","commit_id":"a794e853042d10d9fc70d4ca3e8b3341b573aff1"},{"author":{"_account_id":27078,"name":"Manik Bindlish","email":"manikbindlish19@gmail.com","username":"manik"},"change_message_id":"740fd2efb2cc2e4117881c2927aeb22e6d8c404d","unresolved":false,"context_lines":[{"line_number":163,"context_line":"        spec.append([CONF.object_storage.operator_role])"},{"line_number":164,"context_line":"        spec.append([CONF.object_storage.reseller_admin_role])"},{"line_number":165,"context_line":"    if admin:"},{"line_number":166,"context_line":"        spec.append(\u0027admin\u0027)"},{"line_number":167,"context_line":"    resources \u003d []"},{"line_number":168,"context_line":"    for cred_type in spec:"},{"line_number":169,"context_line":"        resources.append((cred_type, cred_provider.get_credentials("}],"source_content_type":"text/x-python","patch_set":2,"id":"3f79a3b5_5e2e5eab","line":166,"range":{"start_line":166,"start_character":20,"end_line":166,"end_character":27},"in_reply_to":"3f79a3b5_a3143deb","updated":"2018-10-01 14:41:21.000000000","message":"Thanks for the review Felipe,\nShould I create a following patch for this change?","commit_id":"a794e853042d10d9fc70d4ca3e8b3341b573aff1"}]}