)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9126,"name":"Chris Buccella","email":"chris@buccella.org","username":"hdd"},"change_message_id":"7e69fe9e7dd14aad772e2c5ceec2d9e0222cad1c","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Swapnil Kulkarni \u003cswapnilkulkarni2608@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-01-25 18:38:50 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Update incosistencies in tenant options in glanceclient"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Updated glance client for using --tenant option by default,"},{"line_number":10,"context_line":"keeping existign --tenant-id option deprecated."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7adec928_9f03d075","line":7,"updated":"2014-05-21 19:26:14.000000000","message":"incosistencies -\u003e inconsistencies","commit_id":"76455720d1f02d6a2a8739e42b6b3db33ba864ad"},{"author":{"_account_id":9126,"name":"Chris Buccella","email":"chris@buccella.org","username":"hdd"},"change_message_id":"7e69fe9e7dd14aad772e2c5ceec2d9e0222cad1c","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Update incosistencies in tenant options in glanceclient"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Updated glance client for using --tenant option by default,"},{"line_number":10,"context_line":"keeping existign --tenant-id option deprecated."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ic12b5df0af1806f6d88a9b0b4b3a9f5b2285d04a"},{"line_number":13,"context_line":"Closes-Bug: #1228115"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7adec928_1ff7c08d","line":10,"updated":"2014-05-21 19:26:14.000000000","message":"existign -\u003e existing","commit_id":"76455720d1f02d6a2a8739e42b6b3db33ba864ad"},{"author":{"_account_id":2537,"name":"Nikhil Komawar","email":"nik.komawar@gmail.com","username":"nikhil-komawar"},"change_message_id":"346924e42d0067e470f17c96cbfd614d7de08d9b","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Update incosistencies in tenant options in glanceclient"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Updated glance client for using --tenant option by default,"},{"line_number":10,"context_line":"keeping existign --tenant-id option deprecated."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ic12b5df0af1806f6d88a9b0b4b3a9f5b2285d04a"},{"line_number":13,"context_line":"Closes-Bug: #1228115"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1ae5cdf2_3c3c32bf","line":10,"updated":"2014-06-11 02:03:28.000000000","message":"s/keeping/making/","commit_id":"76455720d1f02d6a2a8739e42b6b3db33ba864ad"}],"doc/source/index.rst":[{"author":{"_account_id":2537,"name":"Nikhil Komawar","email":"nik.komawar@gmail.com","username":"nikhil-komawar"},"change_message_id":"346924e42d0067e470f17c96cbfd614d7de08d9b","unresolved":false,"context_lines":[{"line_number":21,"context_line":"In order to use the CLI, you must provide your OpenStack username, password, tenant, and auth endpoint. Use the corresponding configuration options (``--os-username``, ``--os-password``, ``--os-tenant-id``, and ``--os-auth-url``) or set them in environment variables::"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    export OS_USERNAME\u003duser"},{"line_number":24,"context_line":"    export OS_PASSWORD\u003dpass"},{"line_number":25,"context_line":"    export OS_TENANT_ID\u003db363706f891f48019483f8bd6503c54b"},{"line_number":26,"context_line":"    export OS_AUTH_URL\u003dhttp://auth.example.com:5000/v2.0"},{"line_number":27,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"1ae5cdf2_9c3706b0","line":24,"updated":"2014-06-11 02:03:28.000000000","message":"Can you please confirm if all projects are keeping this environment var as is and just using the new standard arg of --tenant? That seems inconsistent. We should change this everywhere in that case.","commit_id":"76455720d1f02d6a2a8739e42b6b3db33ba864ad"}],"glanceclient/v1/shell.py":[{"author":{"_account_id":2537,"name":"Nikhil Komawar","email":"nik.komawar@gmail.com","username":"nikhil-komawar"},"change_message_id":"346924e42d0067e470f17c96cbfd614d7de08d9b","unresolved":false,"context_lines":[{"line_number":344,"context_line":"def do_member_list(gc, args):"},{"line_number":345,"context_line":"    \"\"\"Describe sharing permissions by image or tenant.\"\"\""},{"line_number":346,"context_line":"    if args.tenant is not None:"},{"line_number":347,"context_line":"        args.tenant_id \u003d args.tenant"},{"line_number":348,"context_line":""},{"line_number":349,"context_line":"    if args.image_id and args.tenant_id:"},{"line_number":350,"context_line":"        print(\u0027Unable to filter members by both --image-id and --tenant.\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1ae5cdf2_5c901e95","line":347,"updated":"2014-06-11 02:03:28.000000000","message":"Please add a note here mentioning that this is a workaround until tenant-id is completely removed. Ideally, we should have used args.tenant with this change however, that would not make much sense to do atm.\n\nSo, if you add a #TODO(your_irc_nick) saying that this hack should be removed later it would be great!","commit_id":"76455720d1f02d6a2a8739e42b6b3db33ba864ad"},{"author":{"_account_id":11391,"name":"Mikhail Fedosin","email":"mfedosin@redhat.com","username":"fedosinme"},"change_message_id":"d11e68253c7a9db6baea9d53a2929ddc0a64d76f","unresolved":false,"context_lines":[{"line_number":357,"context_line":"    if args.tenant is not None:"},{"line_number":358,"context_line":"        args.tenant_id \u003d args.tenant"},{"line_number":359,"context_line":"    if args.image_id and args.tenant_id:"},{"line_number":360,"context_line":"        print(\u0027Unable to filter members by both --image-id and --tenant-id.\u0027)"},{"line_number":361,"context_line":"        sys.exit(1)"},{"line_number":362,"context_line":"    elif args.image_id:"},{"line_number":363,"context_line":"        kwargs \u003d {\u0027image\u0027: args.image_id}"}],"source_content_type":"text/x-python","patch_set":3,"id":"da86d52c_1fe1c929","line":360,"updated":"2015-02-06 11:46:28.000000000","message":"Maybe, since --tenant-id is going to be deprecated, should change --tenant-id to --tenant here too?","commit_id":"61a322eaa312cee2bd0501bb7c9a7845f9fae5ec"},{"author":{"_account_id":11356,"name":"Louis Taylor","email":"louis@kragniz.eu","username":"kragniz"},"change_message_id":"3bf985e34e7215715c43bc9bb0fc71065f0c4a2b","unresolved":false,"context_lines":[{"line_number":386,"context_line":"        args.tenant_id \u003d args.tenant"},{"line_number":387,"context_line":"    if args.image_id and args.tenant_id:"},{"line_number":388,"context_line":"        print(\u0027Unable to filter members by both --image-id and --tenant.\u0027)"},{"line_number":389,"context_line":"        sys.exit(1)"},{"line_number":390,"context_line":"    elif args.image_id:"},{"line_number":391,"context_line":"        kwargs \u003d {\u0027image\u0027: args.image_id}"},{"line_number":392,"context_line":"    elif args.tenant_id:"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a0c5dc1_5e4f3abd","line":389,"updated":"2015-05-13 19:44:37.000000000","message":"I\u0027m going to be mean here and ask you to update this to use utils.exit instead, like:\n\n    utils.exit(\u0027Unable to filter members by both --image-id and --tenant.\u0027)\n\nThis does the print + exit with error code using a common function.","commit_id":"f824effcdd93d8c0c7e5438648724fe7d9eec52a"},{"author":{"_account_id":11356,"name":"Louis Taylor","email":"louis@kragniz.eu","username":"kragniz"},"change_message_id":"3bf985e34e7215715c43bc9bb0fc71065f0c4a2b","unresolved":false,"context_lines":[{"line_number":393,"context_line":"        kwargs \u003d {\u0027member\u0027: args.tenant_id}"},{"line_number":394,"context_line":"    else:"},{"line_number":395,"context_line":"        print(\u0027Unable to list all members. Specify --image-id or --tenant\u0027)"},{"line_number":396,"context_line":"        sys.exit(1)"},{"line_number":397,"context_line":""},{"line_number":398,"context_line":"    members \u003d gc.image_members.list(**kwargs)"},{"line_number":399,"context_line":"    columns \u003d [\u0027Image ID\u0027, \u0027Member ID\u0027, \u0027Can Share\u0027]"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a0c5dc1_1ec8a221","line":396,"updated":"2015-05-13 19:44:37.000000000","message":"Likewise here.","commit_id":"f824effcdd93d8c0c7e5438648724fe7d9eec52a"},{"author":{"_account_id":12807,"name":"Steve Lewis (stevelle)","email":"stevelle@gmail.com","username":"stevelle"},"change_message_id":"a8148d8afe23d396c99e350d1c2f5b89156a8c33","unresolved":false,"context_lines":[{"line_number":381,"context_line":"           help\u003dargparse.SUPPRESS)"},{"line_number":382,"context_line":"def do_member_list(gc, args):"},{"line_number":383,"context_line":"    \"\"\"Describe sharing permissions by image or tenant.\"\"\""},{"line_number":384,"context_line":"    # TODO(coolsvap) --tenant-id needs to removed once deprecated"},{"line_number":385,"context_line":"    if args.tenant is not None:"},{"line_number":386,"context_line":"        args.tenant_id \u003d args.tenant"},{"line_number":387,"context_line":"    if args.image_id and args.tenant_id:"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a016987_8b9c5f5e","line":384,"updated":"2015-05-15 21:38:30.000000000","message":"as a nit, should say probably \u0027if\u0027 deprecated instead.","commit_id":"b12d74b7a291c62a7b8edd663dc12510fea59bfe"}]}