)]}'
{"openstackclient/identity/v3/group.py":[{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"52cf98078f17bdbb984d03c19c14a2ab9fd9c422","unresolved":false,"context_lines":[{"line_number":47,"context_line":"            help\u003d\u0027New group description\u0027,"},{"line_number":48,"context_line":"        )"},{"line_number":49,"context_line":"        parser.add_argument("},{"line_number":50,"context_line":"            \u0027--domain_id\u0027,"},{"line_number":51,"context_line":"            metavar\u003d\u0027\u003cgroup-domain_id\u003e\u0027,"},{"line_number":52,"context_line":"            help\u003d\u0027New group domain id\u0027,"},{"line_number":53,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2Fe4g%3D","line":50,"updated":"2013-01-19 05:57:25.000000000","message":"--domain-id?","commit_id":"db2169a7da4bd6665c1f450dccd367d255b25eb1"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"47f75f4bf345191296f5c47911e00707c3130398","unresolved":false,"context_lines":[{"line_number":47,"context_line":"            help\u003d\u0027New group description\u0027,"},{"line_number":48,"context_line":"        )"},{"line_number":49,"context_line":"        parser.add_argument("},{"line_number":50,"context_line":"            \u0027--domain_id\u0027,"},{"line_number":51,"context_line":"            metavar\u003d\u0027\u003cgroup-domain_id\u003e\u0027,"},{"line_number":52,"context_line":"            help\u003d\u0027New group domain id\u0027,"},{"line_number":53,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2Feos%3D","line":50,"updated":"2013-01-20 02:55:24.000000000","message":"Is there a textual identifier for domain?  I want to allow human-meaningful arguments whenever possible, so I\u0027d call this --domain.","commit_id":"db2169a7da4bd6665c1f450dccd367d255b25eb1"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"52cf98078f17bdbb984d03c19c14a2ab9fd9c422","unresolved":false,"context_lines":[{"line_number":49,"context_line":"        parser.add_argument("},{"line_number":50,"context_line":"            \u0027--domain_id\u0027,"},{"line_number":51,"context_line":"            metavar\u003d\u0027\u003cgroup-domain_id\u003e\u0027,"},{"line_number":52,"context_line":"            help\u003d\u0027New group domain id\u0027,"},{"line_number":53,"context_line":"        )"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"        return parser"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2Fe4c%3D","line":52,"updated":"2013-01-19 05:57:25.000000000","message":"id -\u003e ID","commit_id":"db2169a7da4bd6665c1f450dccd367d255b25eb1"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"52cf98078f17bdbb984d03c19c14a2ab9fd9c422","unresolved":false,"context_lines":[{"line_number":103,"context_line":"        parser.add_argument("},{"line_number":104,"context_line":"            \u0027--user\u0027,"},{"line_number":105,"context_line":"            metavar\u003d\u0027\u003cuser\u003e\u0027,"},{"line_number":106,"context_line":"            help\u003d\u0027Name or ID of user to filter groups\u0027,"},{"line_number":107,"context_line":"        )"},{"line_number":108,"context_line":"        parser.add_argument("},{"line_number":109,"context_line":"            \u0027--long\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2Fe4M%3D","line":106,"updated":"2013-01-19 05:57:25.000000000","message":"I assume this is intended to list the groups that a user belongs to? help doesn\u0027t really make that clear? in the API, they\u0027re both list operations on collections of groups, but they\u0027re different operations to the end user -- they should maybe have separate CLI commands?","commit_id":"db2169a7da4bd6665c1f450dccd367d255b25eb1"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"47f75f4bf345191296f5c47911e00707c3130398","unresolved":false,"context_lines":[{"line_number":106,"context_line":"            help\u003d\u0027Name or ID of user to filter groups\u0027,"},{"line_number":107,"context_line":"        )"},{"line_number":108,"context_line":"        parser.add_argument("},{"line_number":109,"context_line":"            \u0027--long\u0027,"},{"line_number":110,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":111,"context_line":"            default\u003dFalse,"},{"line_number":112,"context_line":"            help\u003d\u0027Additional fields are listed in output\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2Feoo%3D","line":109,"updated":"2013-01-20 02:55:24.000000000","message":"--verbose already exists as a global option; we\u0027ve been using --long for displaying additional columns in list outputs.","commit_id":"db2169a7da4bd6665c1f450dccd367d255b25eb1"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"52cf98078f17bdbb984d03c19c14a2ab9fd9c422","unresolved":false,"context_lines":[{"line_number":106,"context_line":"            help\u003d\u0027Name or ID of user to filter groups\u0027,"},{"line_number":107,"context_line":"        )"},{"line_number":108,"context_line":"        parser.add_argument("},{"line_number":109,"context_line":"            \u0027--long\u0027,"},{"line_number":110,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":111,"context_line":"            default\u003dFalse,"},{"line_number":112,"context_line":"            help\u003d\u0027Additional fields are listed in output\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2Fe4E%3D","line":109,"updated":"2013-01-19 05:57:25.000000000","message":"I assume --verbose is taken?","commit_id":"db2169a7da4bd6665c1f450dccd367d255b25eb1"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"52cf98078f17bdbb984d03c19c14a2ab9fd9c422","unresolved":false,"context_lines":[{"line_number":116,"context_line":"    def take_action(self, parsed_args):"},{"line_number":117,"context_line":"        self.log.debug(\u0027take_action(%s)\u0027 % parsed_args)"},{"line_number":118,"context_line":"        if parsed_args.long:"},{"line_number":119,"context_line":"            columns \u003d (\u0027ID\u0027, \u0027Name\u0027, \u0027UserId\u0027, \u0027Description\u0027)"},{"line_number":120,"context_line":"        else:"},{"line_number":121,"context_line":"            columns \u003d (\u0027ID\u0027, \u0027Name\u0027)"},{"line_number":122,"context_line":"        data \u003d self.app.client_manager.identity.groups.list()"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2Fe38%3D","line":119,"updated":"2013-01-19 05:57:25.000000000","message":"kind of lost on how a group could have a user id?","commit_id":"db2169a7da4bd6665c1f450dccd367d255b25eb1"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"52cf98078f17bdbb984d03c19c14a2ab9fd9c422","unresolved":false,"context_lines":[{"line_number":129,"context_line":""},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"class SetGroup(command.Command):"},{"line_number":132,"context_line":"    \"\"\"Set group command\"\"\""},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"    api \u003d \u0027identity\u0027"},{"line_number":135,"context_line":"    log \u003d logging.getLogger(__name__ + \u0027.SetGroup\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2Fe4Q%3D","line":132,"updated":"2013-01-19 05:57:25.000000000","message":"Set -\u003e Update? Set makes it sound like you\u0027re replacing the object (PUT), not just partially updating it.","commit_id":"db2169a7da4bd6665c1f450dccd367d255b25eb1"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"47f75f4bf345191296f5c47911e00707c3130398","unresolved":false,"context_lines":[{"line_number":129,"context_line":""},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"class SetGroup(command.Command):"},{"line_number":132,"context_line":"    \"\"\"Set group command\"\"\""},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"    api \u003d \u0027identity\u0027"},{"line_number":135,"context_line":"    log \u003d logging.getLogger(__name__ + \u0027.SetGroup\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2Feo0%3D","line":132,"in_reply_to":"AAAAM3%2F%2Fe4Q%3D","updated":"2013-01-20 02:55:24.000000000","message":"Yes, set is the new update.  See http://wiki.openstack.org/UnifiedCLI/Mapping for the verb list and the expected behaviours.  The verbs are not defined in REST-like terms but in the actions the user can expect to take place.","commit_id":"db2169a7da4bd6665c1f450dccd367d255b25eb1"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"52cf98078f17bdbb984d03c19c14a2ab9fd9c422","unresolved":false,"context_lines":[{"line_number":165,"context_line":"            kwargs[\u0027description\u0027] \u003d parsed_args.description"},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"        if not len(kwargs):"},{"line_number":168,"context_line":"            #stdout.write(\"Group not updated, no arguments present\")"},{"line_number":169,"context_line":"            print \"Group not updated, no arguments present\""},{"line_number":170,"context_line":"            return"},{"line_number":171,"context_line":"        identity_client.groups.update(group.id, **kwargs)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2Fe4Y%3D","line":168,"updated":"2013-01-19 05:57:25.000000000","message":"remove?","commit_id":"db2169a7da4bd6665c1f450dccd367d255b25eb1"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"47f75f4bf345191296f5c47911e00707c3130398","unresolved":false,"context_lines":[{"line_number":166,"context_line":""},{"line_number":167,"context_line":"        if not len(kwargs):"},{"line_number":168,"context_line":"            #stdout.write(\"Group not updated, no arguments present\")"},{"line_number":169,"context_line":"            print \"Group not updated, no arguments present\""},{"line_number":170,"context_line":"            return"},{"line_number":171,"context_line":"        identity_client.groups.update(group.id, **kwargs)"},{"line_number":172,"context_line":"        return"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2Feok%3D","line":169,"updated":"2013-01-20 02:55:24.000000000","message":"Need to use stdout.write for PyV3 future-proofing.  Yes, for this project we\u0027re shooting to be v3 clean as much as possible.  I should add that to the wiki...","commit_id":"db2169a7da4bd6665c1f450dccd367d255b25eb1"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"ad0c88a1c6ee2aa64dd4640dd9ce4c5ac01a44db","unresolved":false,"context_lines":[{"line_number":107,"context_line":"            help\u003d\u0027Additional fields are listed in output\u0027,"},{"line_number":108,"context_line":"        )"},{"line_number":109,"context_line":"        return parser"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"    def take_action(self, parsed_args):"},{"line_number":112,"context_line":"        self.log.debug(\u0027take_action(%s)\u0027 % parsed_args)"},{"line_number":113,"context_line":"        if parsed_args.long:"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2Fdo0%3D","line":110,"updated":"2013-01-21 17:08:50.000000000","message":"removed the option to filter by user id for now, I was thinking we could make a second list method, like the one seen in openstackclient/identity/v2/role.py; where there is a ListRole and ListUserRole?","commit_id":"2f14c4905775b6064b8eaac1ea2c8de1dfd3d151"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"7e8341f9d723ae33e41ee42f6fc3388095138dc8","unresolved":false,"context_lines":[{"line_number":47,"context_line":"            help\u003d\u0027New group description\u0027,"},{"line_number":48,"context_line":"        )"},{"line_number":49,"context_line":"        parser.add_argument("},{"line_number":50,"context_line":"            \u0027--domain\u0027,"},{"line_number":51,"context_line":"            metavar\u003d\u0027\u003cgroup-domain\u003e\u0027,"},{"line_number":52,"context_line":"            help\u003d\u0027New group domain ID\u0027,"},{"line_number":53,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2FdJU%3D","line":50,"updated":"2013-01-21 22:17:47.000000000","message":"the new argument name make it *look* like you can pass a domain name, but i don\u0027t see a domain name -\u003e id resolution taking place anywhere...","commit_id":"58285bee8b388d788526807edb7ac1a69e504838"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"caa3a2eff7cec8aa8e70cccc52aa92eb6dfea7a9","unresolved":false,"context_lines":[{"line_number":47,"context_line":"            help\u003d\u0027New group description\u0027,"},{"line_number":48,"context_line":"        )"},{"line_number":49,"context_line":"        parser.add_argument("},{"line_number":50,"context_line":"            \u0027--domain\u0027,"},{"line_number":51,"context_line":"            metavar\u003d\u0027\u003cgroup-domain\u003e\u0027,"},{"line_number":52,"context_line":"            help\u003d\u0027New group domain ID\u0027,"},{"line_number":53,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2Fcnk%3D","line":50,"in_reply_to":"AAAAM3%2F%2FdJU%3D","updated":"2013-01-22 06:37:26.000000000","message":"Added name-\u003eid resolution","commit_id":"58285bee8b388d788526807edb7ac1a69e504838"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"7e8341f9d723ae33e41ee42f6fc3388095138dc8","unresolved":false,"context_lines":[{"line_number":49,"context_line":"        parser.add_argument("},{"line_number":50,"context_line":"            \u0027--domain\u0027,"},{"line_number":51,"context_line":"            metavar\u003d\u0027\u003cgroup-domain\u003e\u0027,"},{"line_number":52,"context_line":"            help\u003d\u0027New group domain ID\u0027,"},{"line_number":53,"context_line":"        )"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"        return parser"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2FdJc%3D","line":52,"updated":"2013-01-21 22:17:47.000000000","message":"help still reflects that an ID is expected","commit_id":"58285bee8b388d788526807edb7ac1a69e504838"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"caa3a2eff7cec8aa8e70cccc52aa92eb6dfea7a9","unresolved":false,"context_lines":[{"line_number":49,"context_line":"        parser.add_argument("},{"line_number":50,"context_line":"            \u0027--domain\u0027,"},{"line_number":51,"context_line":"            metavar\u003d\u0027\u003cgroup-domain\u003e\u0027,"},{"line_number":52,"context_line":"            help\u003d\u0027New group domain ID\u0027,"},{"line_number":53,"context_line":"        )"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"        return parser"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2Fcng%3D","line":52,"in_reply_to":"AAAAM3%2F%2FdJc%3D","updated":"2013-01-22 06:37:26.000000000","message":"updated to refer to the fact that both id and name can be used","commit_id":"58285bee8b388d788526807edb7ac1a69e504838"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"7e8341f9d723ae33e41ee42f6fc3388095138dc8","unresolved":false,"context_lines":[{"line_number":59,"context_line":"        identity_client \u003d self.app.client_manager.identity"},{"line_number":60,"context_line":"        group \u003d identity_client.groups.create("},{"line_number":61,"context_line":"            parsed_args.name,"},{"line_number":62,"context_line":"            parsed_args.domain,"},{"line_number":63,"context_line":"            parsed_args.description,"},{"line_number":64,"context_line":"        )"},{"line_number":65,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2FdJg%3D","line":62,"updated":"2013-01-21 22:17:47.000000000","message":"the .create() call is expecting either a domain ref or domain_id","commit_id":"58285bee8b388d788526807edb7ac1a69e504838"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"caa3a2eff7cec8aa8e70cccc52aa92eb6dfea7a9","unresolved":false,"context_lines":[{"line_number":59,"context_line":"        identity_client \u003d self.app.client_manager.identity"},{"line_number":60,"context_line":"        group \u003d identity_client.groups.create("},{"line_number":61,"context_line":"            parsed_args.name,"},{"line_number":62,"context_line":"            parsed_args.domain,"},{"line_number":63,"context_line":"            parsed_args.description,"},{"line_number":64,"context_line":"        )"},{"line_number":65,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2Fcnc%3D","line":62,"in_reply_to":"AAAAM3%2F%2FdJg%3D","updated":"2013-01-22 06:37:26.000000000","message":"Done","commit_id":"58285bee8b388d788526807edb7ac1a69e504838"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"f91b67853d692678b38fa17dccc8b1c2cc79e237","unresolved":false,"context_lines":[{"line_number":65,"context_line":"        group \u003d identity_client.groups.create("},{"line_number":66,"context_line":"            parsed_args.name,"},{"line_number":67,"context_line":"            domain_id\u003ddomain_id,"},{"line_number":68,"context_line":"            parsed_args.description,"},{"line_number":69,"context_line":"        )"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"        info \u003d {}"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAM3%2F%2Fbmw%3D","line":68,"updated":"2013-01-23 21:11:17.000000000","message":"This needs to be a keyword arg.","commit_id":"b69c9faceaf7b9d1a9cfb7d361eb2976e97e5b42"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"8a0fda9518938569f23d96b19460cf87c8047465","unresolved":false,"context_lines":[{"line_number":65,"context_line":"        group \u003d identity_client.groups.create("},{"line_number":66,"context_line":"            parsed_args.name,"},{"line_number":67,"context_line":"            domain_id\u003ddomain_id,"},{"line_number":68,"context_line":"            parsed_args.description,"},{"line_number":69,"context_line":"        )"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"        info \u003d {}"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAM3%2F%2FWsU%3D","line":68,"in_reply_to":"AAAAM3%2F%2Fbmw%3D","updated":"2013-01-25 20:31:42.000000000","message":"+1","commit_id":"b69c9faceaf7b9d1a9cfb7d361eb2976e97e5b42"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"53164e2aae7f8f3b4bd8083f80c0a7b1c20bbab4","unresolved":false,"context_lines":[{"line_number":65,"context_line":"        group \u003d identity_client.groups.create("},{"line_number":66,"context_line":"            parsed_args.name,"},{"line_number":67,"context_line":"            domain_id\u003ddomain_id,"},{"line_number":68,"context_line":"            parsed_args.description,"},{"line_number":69,"context_line":"        )"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"        info \u003d {}"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAM3%2F%2FWyc%3D","line":68,"in_reply_to":"AAAAM3%2F%2Fbmw%3D","updated":"2013-01-25 20:40:02.000000000","message":"Done","commit_id":"b69c9faceaf7b9d1a9cfb7d361eb2976e97e5b42"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"8d834482e1106c78a7cb6792269980d0b50c7e37","unresolved":false,"context_lines":[{"line_number":116,"context_line":"    def take_action(self, parsed_args):"},{"line_number":117,"context_line":"        self.log.debug(\u0027take_action(%s)\u0027 % parsed_args)"},{"line_number":118,"context_line":"        if parsed_args.long:"},{"line_number":119,"context_line":"            columns \u003d (\u0027ID\u0027, \u0027Name\u0027, \u0027Description\u0027)"},{"line_number":120,"context_line":"        else:"},{"line_number":121,"context_line":"            columns \u003d (\u0027ID\u0027, \u0027Name\u0027)"},{"line_number":122,"context_line":"        data \u003d self.app.client_manager.identity.groups.list()"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAM3%2F%2FZCA%3D","line":119,"updated":"2013-01-24 11:24:48.000000000","message":"Do we not need domain_id here?","commit_id":"b69c9faceaf7b9d1a9cfb7d361eb2976e97e5b42"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"8a0fda9518938569f23d96b19460cf87c8047465","unresolved":false,"context_lines":[{"line_number":116,"context_line":"    def take_action(self, parsed_args):"},{"line_number":117,"context_line":"        self.log.debug(\u0027take_action(%s)\u0027 % parsed_args)"},{"line_number":118,"context_line":"        if parsed_args.long:"},{"line_number":119,"context_line":"            columns \u003d (\u0027ID\u0027, \u0027Name\u0027, \u0027Description\u0027)"},{"line_number":120,"context_line":"        else:"},{"line_number":121,"context_line":"            columns \u003d (\u0027ID\u0027, \u0027Name\u0027)"},{"line_number":122,"context_line":"        data \u003d self.app.client_manager.identity.groups.list()"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAM3%2F%2FWsc%3D","line":119,"in_reply_to":"AAAAM3%2F%2FZCA%3D","updated":"2013-01-25 20:31:42.000000000","message":"+1","commit_id":"b69c9faceaf7b9d1a9cfb7d361eb2976e97e5b42"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"53164e2aae7f8f3b4bd8083f80c0a7b1c20bbab4","unresolved":false,"context_lines":[{"line_number":116,"context_line":"    def take_action(self, parsed_args):"},{"line_number":117,"context_line":"        self.log.debug(\u0027take_action(%s)\u0027 % parsed_args)"},{"line_number":118,"context_line":"        if parsed_args.long:"},{"line_number":119,"context_line":"            columns \u003d (\u0027ID\u0027, \u0027Name\u0027, \u0027Description\u0027)"},{"line_number":120,"context_line":"        else:"},{"line_number":121,"context_line":"            columns \u003d (\u0027ID\u0027, \u0027Name\u0027)"},{"line_number":122,"context_line":"        data \u003d self.app.client_manager.identity.groups.list()"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAM3%2F%2FWx4%3D","line":119,"in_reply_to":"AAAAM3%2F%2FZCA%3D","updated":"2013-01-25 20:40:02.000000000","message":"Done","commit_id":"b69c9faceaf7b9d1a9cfb7d361eb2976e97e5b42"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"8d834482e1106c78a7cb6792269980d0b50c7e37","unresolved":false,"context_lines":[{"line_number":162,"context_line":"        if parsed_args.name:"},{"line_number":163,"context_line":"            kwargs[\u0027name\u0027] \u003d parsed_args.name"},{"line_number":164,"context_line":"        if parsed_args.description:"},{"line_number":165,"context_line":"            kwargs[\u0027description\u0027] \u003d parsed_args.description"},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"        if not len(kwargs):"},{"line_number":168,"context_line":"            stdout.write(\"Group not updated, no arguments present\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAM3%2F%2FZB8%3D","line":165,"updated":"2013-01-24 11:24:48.000000000","message":"You should be able to change the domain_id as well","commit_id":"b69c9faceaf7b9d1a9cfb7d361eb2976e97e5b42"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"699863c553dcf66f30de7f71e5fd8b93859bfc23","unresolved":false,"context_lines":[{"line_number":162,"context_line":"        if parsed_args.name:"},{"line_number":163,"context_line":"            kwargs[\u0027name\u0027] \u003d parsed_args.name"},{"line_number":164,"context_line":"        if parsed_args.description:"},{"line_number":165,"context_line":"            kwargs[\u0027description\u0027] \u003d parsed_args.description"},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"        if not len(kwargs):"},{"line_number":168,"context_line":"            stdout.write(\"Group not updated, no arguments present\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAM3%2F%2FWNU%3D","line":165,"in_reply_to":"AAAAM3%2F%2FWqo%3D","updated":"2013-01-27 00:59:34.000000000","message":"I agree that right now, regular users can\u0027t do this....but admin can...and I think we will need to allow admin to, for instance, move projects/users that are migrated from Folsom into the default domain into their own domains....which will involve updating the domain_id.","commit_id":"b69c9faceaf7b9d1a9cfb7d361eb2976e97e5b42"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"53164e2aae7f8f3b4bd8083f80c0a7b1c20bbab4","unresolved":false,"context_lines":[{"line_number":162,"context_line":"        if parsed_args.name:"},{"line_number":163,"context_line":"            kwargs[\u0027name\u0027] \u003d parsed_args.name"},{"line_number":164,"context_line":"        if parsed_args.description:"},{"line_number":165,"context_line":"            kwargs[\u0027description\u0027] \u003d parsed_args.description"},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"        if not len(kwargs):"},{"line_number":168,"context_line":"            stdout.write(\"Group not updated, no arguments present\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAM3%2F%2FWqo%3D","line":165,"in_reply_to":"AAAAM3%2F%2FWsE%3D","updated":"2013-01-25 20:40:02.000000000","message":"Checked in a new patch before reading this comment, I\u0027ll update with another patch - if we conclude that domain_id should be immutable.","commit_id":"b69c9faceaf7b9d1a9cfb7d361eb2976e97e5b42"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"8a0fda9518938569f23d96b19460cf87c8047465","unresolved":false,"context_lines":[{"line_number":162,"context_line":"        if parsed_args.name:"},{"line_number":163,"context_line":"            kwargs[\u0027name\u0027] \u003d parsed_args.name"},{"line_number":164,"context_line":"        if parsed_args.description:"},{"line_number":165,"context_line":"            kwargs[\u0027description\u0027] \u003d parsed_args.description"},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"        if not len(kwargs):"},{"line_number":168,"context_line":"            stdout.write(\"Group not updated, no arguments present\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAM3%2F%2FWsE%3D","line":165,"in_reply_to":"AAAAM3%2F%2FZB8%3D","updated":"2013-01-25 20:31:42.000000000","message":"henry: i\u0027m starting to think that group.domain_id, project.domain_id, user.domain_id should perhaps be immutable, since we can\u0027t offer multi-domain-scoped tokens to authorize the change?","commit_id":"b69c9faceaf7b9d1a9cfb7d361eb2976e97e5b42"}],"openstackclient/identity/v3/project.py":[{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"7e8341f9d723ae33e41ee42f6fc3388095138dc8","unresolved":false,"context_lines":[{"line_number":64,"context_line":"            dest\u003d\u0027enabled\u0027,"},{"line_number":65,"context_line":"            action\u003d\u0027store_false\u0027,"},{"line_number":66,"context_line":"            help\u003d\u0027Disable project\u0027,"},{"line_number":67,"context_line":"        )"},{"line_number":68,"context_line":"        return parser"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"    def take_action(self, parsed_args):"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2FdHI%3D","line":67,"updated":"2013-01-21 22:17:47.000000000","message":"Given that things are created in the enabled state by default, and that --enable is completely redundant and can conflict with --disable (if they\u0027re both provided), I\u0027d rather see a single parameter on create calls to opt-in to creating objects in the disabled state, rather than having a completely useless option (--enable):\n\n  object-create --disabled\n\n--enable/--disable make sense on update because you\u0027re setting a toggle that could be in either position:\n\n  object-update --enable\n  object-update --disable","commit_id":"58285bee8b388d788526807edb7ac1a69e504838"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"79daf0a85302d8c015cf391dfd584611966cb07f","unresolved":false,"context_lines":[{"line_number":64,"context_line":"            dest\u003d\u0027enabled\u0027,"},{"line_number":65,"context_line":"            action\u003d\u0027store_false\u0027,"},{"line_number":66,"context_line":"            help\u003d\u0027Disable project\u0027,"},{"line_number":67,"context_line":"        )"},{"line_number":68,"context_line":"        return parser"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"    def take_action(self, parsed_args):"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2Fcxo%3D","line":67,"in_reply_to":"AAAAM3%2F%2FdHI%3D","updated":"2013-01-22 02:35:14.000000000","message":"I\u0027ve done this on the other create commands for completeness and, well, symmetry.  I assume the chances of the default changing are slim.  I suppose then I need to go change the others...","commit_id":"58285bee8b388d788526807edb7ac1a69e504838"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"7e8341f9d723ae33e41ee42f6fc3388095138dc8","unresolved":false,"context_lines":[{"line_number":74,"context_line":"            parsed_args.project_name,"},{"line_number":75,"context_line":"            domain\u003dparsed_args.domain,"},{"line_number":76,"context_line":"            description\u003dparsed_args.description,"},{"line_number":77,"context_line":"            enabled\u003dparsed_args.enabled,"},{"line_number":78,"context_line":"        )"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"        info \u003d {}"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2FdHA%3D","line":77,"updated":"2013-01-21 22:17:47.000000000","message":"enabled\u003dnot parsed_args.disabled,","commit_id":"58285bee8b388d788526807edb7ac1a69e504838"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"caa3a2eff7cec8aa8e70cccc52aa92eb6dfea7a9","unresolved":false,"context_lines":[{"line_number":74,"context_line":"            parsed_args.project_name,"},{"line_number":75,"context_line":"            domain\u003dparsed_args.domain,"},{"line_number":76,"context_line":"            description\u003dparsed_args.description,"},{"line_number":77,"context_line":"            enabled\u003dparsed_args.enabled,"},{"line_number":78,"context_line":"        )"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"        info \u003d {}"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2Fclw%3D","line":77,"in_reply_to":"AAAAM3%2F%2FdHA%3D","updated":"2013-01-22 06:37:26.000000000","message":"I will update the enabled/disabled arguments for v3, once Dean has updated them for v2. Just to keep things similar.","commit_id":"58285bee8b388d788526807edb7ac1a69e504838"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"7e8341f9d723ae33e41ee42f6fc3388095138dc8","unresolved":false,"context_lines":[{"line_number":158,"context_line":"        parser.add_argument("},{"line_number":159,"context_line":"            \u0027--domain\u0027,"},{"line_number":160,"context_line":"            metavar\u003d\u0027\u003cproject-domain\u003e\u0027,"},{"line_number":161,"context_line":"            help\u003d\u0027New project domain\u0027,"},{"line_number":162,"context_line":"        )"},{"line_number":163,"context_line":"        parser.add_argument("},{"line_number":164,"context_line":"            \u0027--description\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2FdG8%3D","line":161,"updated":"2013-01-21 22:17:47.000000000","message":"name or ID","commit_id":"58285bee8b388d788526807edb7ac1a69e504838"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"caa3a2eff7cec8aa8e70cccc52aa92eb6dfea7a9","unresolved":false,"context_lines":[{"line_number":158,"context_line":"        parser.add_argument("},{"line_number":159,"context_line":"            \u0027--domain\u0027,"},{"line_number":160,"context_line":"            metavar\u003d\u0027\u003cproject-domain\u003e\u0027,"},{"line_number":161,"context_line":"            help\u003d\u0027New project domain\u0027,"},{"line_number":162,"context_line":"        )"},{"line_number":163,"context_line":"        parser.add_argument("},{"line_number":164,"context_line":"            \u0027--description\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2Fcl0%3D","line":161,"in_reply_to":"AAAAM3%2F%2FdG8%3D","updated":"2013-01-22 06:37:26.000000000","message":"Looked into the update method of projects, it does not expect a domain argument in kwargs, is this correct?","commit_id":"58285bee8b388d788526807edb7ac1a69e504838"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"8d834482e1106c78a7cb6792269980d0b50c7e37","unresolved":false,"context_lines":[{"line_number":161,"context_line":"            \u0027--name\u0027,"},{"line_number":162,"context_line":"            metavar\u003d\u0027\u003cnew-project-name\u003e\u0027,"},{"line_number":163,"context_line":"            help\u003d\u0027New project name\u0027,"},{"line_number":164,"context_line":"        )"},{"line_number":165,"context_line":"        parser.add_argument("},{"line_number":166,"context_line":"            \u0027--description\u0027,"},{"line_number":167,"context_line":"            metavar\u003d\u0027\u003cproject-description\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAM3%2F%2FZBs%3D","line":164,"updated":"2013-01-24 11:24:48.000000000","message":"Should be able to change domain","commit_id":"b69c9faceaf7b9d1a9cfb7d361eb2976e97e5b42"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"53164e2aae7f8f3b4bd8083f80c0a7b1c20bbab4","unresolved":false,"context_lines":[{"line_number":161,"context_line":"            \u0027--name\u0027,"},{"line_number":162,"context_line":"            metavar\u003d\u0027\u003cnew-project-name\u003e\u0027,"},{"line_number":163,"context_line":"            help\u003d\u0027New project name\u0027,"},{"line_number":164,"context_line":"        )"},{"line_number":165,"context_line":"        parser.add_argument("},{"line_number":166,"context_line":"            \u0027--description\u0027,"},{"line_number":167,"context_line":"            metavar\u003d\u0027\u003cproject-description\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAM3%2F%2FWxc%3D","line":164,"in_reply_to":"AAAAM3%2F%2FZBs%3D","updated":"2013-01-25 20:40:02.000000000","message":"I wasn\u0027t sure if that was a valid property to change, thanks for verifying.","commit_id":"b69c9faceaf7b9d1a9cfb7d361eb2976e97e5b42"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"53164e2aae7f8f3b4bd8083f80c0a7b1c20bbab4","unresolved":false,"context_lines":[{"line_number":195,"context_line":"            kwargs[\u0027description\u0027] \u003d parsed_args.description"},{"line_number":196,"context_line":"        if \u0027enabled\u0027 in parsed_args:"},{"line_number":197,"context_line":"            kwargs[\u0027enabled\u0027] \u003d parsed_args.enabled"},{"line_number":198,"context_line":""},{"line_number":199,"context_line":"        if kwargs \u003d\u003d {}:"},{"line_number":200,"context_line":"            stdout.write(\"Project not updated, no arguments present\")"},{"line_number":201,"context_line":"            return"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAM3%2F%2FWxY%3D","line":198,"updated":"2013-01-25 20:40:02.000000000","message":"Added \n\nif parsed_args.domain:\n            domain \u003d utils.find_resource(\n                identity_client.domains, parsed_args.domain).id\n            kwargs[\u0027domain\u0027] \u003d domain\n\nJust, making sure this is the right idea...","commit_id":"b69c9faceaf7b9d1a9cfb7d361eb2976e97e5b42"}],"setup.py":[{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"79daf0a85302d8c015cf391dfd584611966cb07f","unresolved":false,"context_lines":[{"line_number":109,"context_line":"            \u0027list_user\u003dopenstackclient.identity.v2_0.user:ListUser\u0027,"},{"line_number":110,"context_line":"            \u0027set_user\u003dopenstackclient.identity.v2_0.user:SetUser\u0027,"},{"line_number":111,"context_line":"            \u0027show_user\u003dopenstackclient.identity.v2_0.user:ShowUser\u0027,"},{"line_number":112,"context_line":"            \u0027list_user-role\u003dopenstackclient.identity.v2_0.role:ListUserRole\u0027,"},{"line_number":113,"context_line":"            \u0027list_image\u003dopenstackclient.image.v2.image:ListImage\u0027,"},{"line_number":114,"context_line":"            \u0027show_image\u003dopenstackclient.image.v2.image:ShowImage\u0027,"},{"line_number":115,"context_line":"            \u0027save_image\u003dopenstackclient.image.v2.image:SaveImage\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2FcyM%3D","line":112,"updated":"2013-01-22 02:35:14.000000000","message":"This is correct.  Although I\u0027m still not sure about the order of the \u0027subject\u0027 words...","commit_id":"58285bee8b388d788526807edb7ac1a69e504838"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"6880815feafb6fa9e0f8e0567c28f4d284e7851f","unresolved":false,"context_lines":[{"line_number":109,"context_line":"            \u0027list_user\u003dopenstackclient.identity.v2_0.user:ListUser\u0027,"},{"line_number":110,"context_line":"            \u0027set_user\u003dopenstackclient.identity.v2_0.user:SetUser\u0027,"},{"line_number":111,"context_line":"            \u0027show_user\u003dopenstackclient.identity.v2_0.user:ShowUser\u0027,"},{"line_number":112,"context_line":"            \u0027list_user-role\u003dopenstackclient.identity.v2_0.role:ListUserRole\u0027,"},{"line_number":113,"context_line":"            \u0027list_image\u003dopenstackclient.image.v2.image:ListImage\u0027,"},{"line_number":114,"context_line":"            \u0027show_image\u003dopenstackclient.image.v2.image:ShowImage\u0027,"},{"line_number":115,"context_line":"            \u0027save_image\u003dopenstackclient.image.v2.image:SaveImage\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2FdGY%3D","line":112,"updated":"2013-01-21 22:18:58.000000000","message":"random hyphen","commit_id":"58285bee8b388d788526807edb7ac1a69e504838"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"383c6feaededcb1d415845b229f9cb8979a8b467","unresolved":false,"context_lines":[{"line_number":124,"context_line":"             #\u0027show_credential\u003d\u0027 +"},{"line_number":125,"context_line":"             #   \u0027openstackclient.identity.v3.credential:ShowCredential\u0027,"},{"line_number":126,"context_line":"             #\u0027list_credential\u003d\u0027 +"},{"line_number":127,"context_line":"             #   \u0027openstackclient.identity.v3.credential:ListCredential\u0027,"},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"             #\u0027create_domain\u003dopenstackclient.identity.v3.domain:CreateDomain\u0027,"},{"line_number":130,"context_line":"             #\u0027delete_domain\u003dopenstackclient.identity.v3.domain:DeleteDomain\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAM3%2F%2FaMI%3D","line":127,"updated":"2013-01-23 17:23:45.000000000","message":"I don\u0027t like to see code that\u0027s commented out that doesn\u0027t have an explanation of why it\u0027s commented out. I think this should just be removed.","commit_id":"b69c9faceaf7b9d1a9cfb7d361eb2976e97e5b42"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"53164e2aae7f8f3b4bd8083f80c0a7b1c20bbab4","unresolved":false,"context_lines":[{"line_number":124,"context_line":"             #\u0027show_credential\u003d\u0027 +"},{"line_number":125,"context_line":"             #   \u0027openstackclient.identity.v3.credential:ShowCredential\u0027,"},{"line_number":126,"context_line":"             #\u0027list_credential\u003d\u0027 +"},{"line_number":127,"context_line":"             #   \u0027openstackclient.identity.v3.credential:ListCredential\u0027,"},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"             #\u0027create_domain\u003dopenstackclient.identity.v3.domain:CreateDomain\u0027,"},{"line_number":130,"context_line":"             #\u0027delete_domain\u003dopenstackclient.identity.v3.domain:DeleteDomain\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAM3%2F%2FWw4%3D","line":127,"in_reply_to":"AAAAM3%2F%2FaAs%3D","updated":"2013-01-25 20:40:02.000000000","message":"Removed the commented out lines","commit_id":"b69c9faceaf7b9d1a9cfb7d361eb2976e97e5b42"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"7975d98cfaf568d753f03ab5d530d36c5a5d5c73","unresolved":false,"context_lines":[{"line_number":124,"context_line":"             #\u0027show_credential\u003d\u0027 +"},{"line_number":125,"context_line":"             #   \u0027openstackclient.identity.v3.credential:ShowCredential\u0027,"},{"line_number":126,"context_line":"             #\u0027list_credential\u003d\u0027 +"},{"line_number":127,"context_line":"             #   \u0027openstackclient.identity.v3.credential:ListCredential\u0027,"},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"             #\u0027create_domain\u003dopenstackclient.identity.v3.domain:CreateDomain\u0027,"},{"line_number":130,"context_line":"             #\u0027delete_domain\u003dopenstackclient.identity.v3.domain:DeleteDomain\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAM3%2F%2FaAs%3D","line":127,"in_reply_to":"AAAAM3%2F%2FaMI%3D","updated":"2013-01-23 19:06:28.000000000","message":"i assume these are going to be implemented in a different patch. either they should be removed until then or include a TODO","commit_id":"b69c9faceaf7b9d1a9cfb7d361eb2976e97e5b42"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"e94cbd268281b440753c18e6a44b9aa9de4c572c","unresolved":false,"context_lines":[{"line_number":112,"context_line":"            \u0027list_image\u003dopenstackclient.image.v2.image:ListImage\u0027,"},{"line_number":113,"context_line":"            \u0027show_image\u003dopenstackclient.image.v2.image:ShowImage\u0027,"},{"line_number":114,"context_line":"            \u0027save_image\u003dopenstackclient.image.v2.image:SaveImage\u0027,"},{"line_number":115,"context_line":"             \u0027create_group\u003dopenstackclient.identity.v3.group:CreateGroup\u0027,"},{"line_number":116,"context_line":"             \u0027delete_group\u003dopenstackclient.identity.v3.group:DeleteGroup\u0027,"},{"line_number":117,"context_line":"             \u0027set_group\u003dopenstackclient.identity.v3.group:SetGroup\u0027,"},{"line_number":118,"context_line":"             \u0027show_group\u003dopenstackclient.identity.v3.group:ShowGroup\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAM3%2F%2FVEU%3D","line":115,"updated":"2013-01-28 19:58:03.000000000","message":"...minor nit...is this meant to be indented one character?","commit_id":"e861d4bd5411595239555434aec6d1dd82eaf416"}]}