)]}'
{"keystoneclient/v2_0/shell.py":[{"author":{"_account_id":6,"name":"Joe Heck","email":"heckj@mac.com","username":"heckj"},"change_message_id":"4a1acb497049f91f184b1f199ea4792afbadd35c","unresolved":false,"context_lines":[{"line_number":233,"context_line":"@utils.arg(\u0027user_id\u0027, metavar\u003d\u0027\u003cuser_id\u003e\u0027, help\u003d\u0027ID of User\u0027)"},{"line_number":234,"context_line":"@utils.arg(\u0027role_id\u0027, metavar\u003d\u0027\u003crole_id\u003e\u0027, help\u003d\u0027ID of Role\u0027)"},{"line_number":235,"context_line":"@utils.arg(\u0027tenant\u0027, metavar\u003d\u0027\u003ctenant_id\u003e\u0027,"},{"line_number":236,"context_line":"        help\u003d\u0027ID of Tenant\u0027, nargs\u003d\u0027?\u0027)"},{"line_number":237,"context_line":"def do_add_user_role(kc, args):"},{"line_number":238,"context_line":"    kc.roles.add_user_role(args.user_id, args.role_id, args.tenant)"},{"line_number":239,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAEn%2F%2F5qw%3D","line":236,"updated":"2012-02-17 03:10:34.000000000","message":"would you mind lining up the arguments?","commit_id":"d9469d95457fb1676c09c10ea4cc6a93fe55f39a"},{"author":{"_account_id":6,"name":"Joe Heck","email":"heckj@mac.com","username":"heckj"},"change_message_id":"4a1acb497049f91f184b1f199ea4792afbadd35c","unresolved":false,"context_lines":[{"line_number":242,"context_line":"@utils.arg(\u0027user_id\u0027, metavar\u003d\u0027\u003cuser_id\u003e\u0027, help\u003d\u0027ID of User\u0027)"},{"line_number":243,"context_line":"@utils.arg(\u0027role_id\u0027, metavar\u003d\u0027\u003crole_id\u003e\u0027, help\u003d\u0027ID of Role\u0027)"},{"line_number":244,"context_line":"@utils.arg(\u0027tenant\u0027, metavar\u003d\u0027\u003ctenant_id\u003e\u0027,"},{"line_number":245,"context_line":"        help\u003d\u0027ID of Tenant\u0027, nargs\u003d\u0027?\u0027)"},{"line_number":246,"context_line":"def do_remove_user_role(kc, args):"},{"line_number":247,"context_line":"    kc.roles.remove_user_role(args.user_id, args.role_id, args.tenant)"},{"line_number":248,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAEn%2F%2F5qs%3D","line":245,"updated":"2012-02-17 03:10:34.000000000","message":"lining up the arguments?","commit_id":"d9469d95457fb1676c09c10ea4cc6a93fe55f39a"},{"author":{"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},"change_message_id":"4d99562677fa6225bca83fc9c73f84a9f21e83ab","unresolved":false,"context_lines":[{"line_number":45,"context_line":"    utils.print_dict(user._info)"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"@utils.arg(\u0027id\u0027, metavar\u003d\u0027\u003cuser_id\u003e\u0027, help\u003d\u0027User ID to update email.\u0027)"},{"line_number":49,"context_line":"@utils.arg(\u0027name\u0027, metavar\u003d\u0027\u003cname\u003e\u0027, nargs\u003d\u0027?\u0027,"},{"line_number":50,"context_line":"           help\u003d\u0027New desired user name.\u0027)"},{"line_number":51,"context_line":"@utils.arg(\u0027email\u0027, metavar\u003d\u0027\u003cemail\u003e\u0027, nargs\u003d\u0027?\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAEn%2F%2F5pw%3D","line":48,"updated":"2012-02-17 05:20:37.000000000","message":"I think you can revert the help text change here.","commit_id":"7b50a1abf3fd0ae13e5ad2cd2a9ac9f95fbd537b"},{"author":{"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},"change_message_id":"4d99562677fa6225bca83fc9c73f84a9f21e83ab","unresolved":false,"context_lines":[{"line_number":141,"context_line":"@utils.arg(\u0027--enabled\u0027, metavar\u003d\u0027\u003cTrue/False\u003e\u0027, nargs\u003d\u0027?\u0027, const\u003dTrue,"},{"line_number":142,"context_line":"           help\u003d\u0027Enable/disable tenant\u0027)"},{"line_number":143,"context_line":"@utils.arg(\u0027id\u0027, metavar\u003d\u0027\u003ctenant_id\u003e\u0027, help\u003d\u0027Tenant ID to update\u0027)"},{"line_number":144,"context_line":"def do_tenant_update(kc, args):"},{"line_number":145,"context_line":"    \"\"\"Update tenant name, description, enabled status\"\"\""},{"line_number":146,"context_line":"    tenant \u003d kc.tenants.get(args.id)"},{"line_number":147,"context_line":"    kwargs \u003d {}"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAEn%2F%2F5p8%3D","line":144,"updated":"2012-02-17 05:20:37.000000000","message":"I\u0027d like to see tenant_enable/disable go away with this patch.","commit_id":"7b50a1abf3fd0ae13e5ad2cd2a9ac9f95fbd537b"},{"author":{"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},"change_message_id":"4d99562677fa6225bca83fc9c73f84a9f21e83ab","unresolved":false,"context_lines":[{"line_number":152,"context_line":"    if args.enabled:"},{"line_number":153,"context_line":"        new_enable \u003d args.enabled.lower() in [\u0027true\u0027, \u0027yes\u0027, \u00271\u0027]"},{"line_number":154,"context_line":"        kwargs.update({\u0027enabled\u0027: new_enable})"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"    try:"},{"line_number":157,"context_line":"        tenant.update(**kwargs)"},{"line_number":158,"context_line":"        print \u0027Tenant has been updated.\u0027"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAEn%2F%2F5p4%3D","line":155,"updated":"2012-02-17 05:20:37.000000000","message":"How about we fail if we don\u0027t get one of the three attributes above? It is misleading to print out \u0027Tenant has been updated\u0027 when you don\u0027t end up changing anything.","commit_id":"7b50a1abf3fd0ae13e5ad2cd2a9ac9f95fbd537b"},{"author":{"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},"change_message_id":"4d99562677fa6225bca83fc9c73f84a9f21e83ab","unresolved":false,"context_lines":[{"line_number":157,"context_line":"        tenant.update(**kwargs)"},{"line_number":158,"context_line":"        print \u0027Tenant has been updated.\u0027"},{"line_number":159,"context_line":"    except Exception, e:"},{"line_number":160,"context_line":"        print \u0027Unable to update tenant: %s\u0027 % e"},{"line_number":161,"context_line":""},{"line_number":162,"context_line":""},{"line_number":163,"context_line":"@utils.arg(\u0027id\u0027, metavar\u003d\u0027\u003ctenant_id\u003e\u0027, help\u003d\u0027Tenant ID to delete\u0027)"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAEn%2F%2F5p0%3D","line":160,"updated":"2012-02-17 05:20:37.000000000","message":"So you can\u0027t update a tenant that wasn\u0027t given a description upon creation unless you first update its description. Can you see if that\u0027s an easy thing to handle here?","commit_id":"7b50a1abf3fd0ae13e5ad2cd2a9ac9f95fbd537b"},{"author":{"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},"change_message_id":"3f36ddc7fb5b89bb4779e6d63204bbb31def683d","unresolved":false,"context_lines":[{"line_number":136,"context_line":"        kwargs.update({\u0027enabled\u0027: new_enable})"},{"line_number":137,"context_line":""},{"line_number":138,"context_line":"    if kwargs \u003d\u003d {}:"},{"line_number":139,"context_line":"        print \"Tenant not updated, no arguemnts present.\""},{"line_number":140,"context_line":"        return"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"    try:"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAAEn%2F%2F5pg%3D","line":139,"updated":"2012-02-17 05:50:53.000000000","message":"Whoops, missed the typo here: \u0027arguemnts\u0027.","commit_id":"d3a88f31e571f19fa4a51220b7780b7aa7476f93"}],"keystoneclient/v2_0/tenants.py":[{"author":{"_account_id":6,"name":"Joe Heck","email":"heckj@mac.com","username":"heckj"},"change_message_id":"4a1acb497049f91f184b1f199ea4792afbadd35c","unresolved":false,"context_lines":[{"line_number":34,"context_line":""},{"line_number":35,"context_line":"        try:"},{"line_number":36,"context_line":"            retval \u003d self.manager.update(self.id, tenant_name\u003dnew_name,"},{"line_number":37,"context_line":"                    description\u003dnew_description,"},{"line_number":38,"context_line":"                    enabled\u003dnew_enabled)"},{"line_number":39,"context_line":"            self \u003d retval"},{"line_number":40,"context_line":"        except Exception, e:"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAEn%2F%2F5qo%3D","line":37,"updated":"2012-02-17 03:10:34.000000000","message":"line up the args?","commit_id":"d9469d95457fb1676c09c10ea4cc6a93fe55f39a"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"3545ae2cc8fdc1bf6023ab5c745e7d1724cc8543","unresolved":false,"context_lines":[{"line_number":38,"context_line":"                                         enabled\u003dnew_enabled)"},{"line_number":39,"context_line":"            self \u003d retval"},{"line_number":40,"context_line":"        except Exception, e:"},{"line_number":41,"context_line":"            retval \u003d None"},{"line_number":42,"context_line":"        return retval"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    def add_user(self, user, role):"}],"source_content_type":"text/x-python","patch_set":9,"id":"9ad9bd40_12eea8b4","line":41,"updated":"2014-05-02 18:16:26.000000000","message":"... wondering what the reasoning is behind the above 3 lines of code ...","commit_id":"bbd2561507538282c5fb1c549b298226399237a1"}]}