)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"01be5407afcf9eb0abf34e71788b76856a869ed4","unresolved":false,"context_lines":[{"line_number":12,"context_line":"the addition of separate entry points for listing role assignments"},{"line_number":13,"context_line":"for a tree in the follow on patch."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"This patch, therfore, rationalizes the routes so that this"},{"line_number":16,"context_line":"is set up for just the list method, and removes the unused code"},{"line_number":17,"context_line":"stubs."},{"line_number":18,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7a2fa921_38bf13d2","line":15,"updated":"2015-10-05 22:11:45.000000000","message":"change \"therforer\" to \"therefore\"","commit_id":"e2e1ccfff94fd7779cd2339caa3c843ed2f2afd5"}],"keystone/assignment/routers.py":[{"author":{"_account_id":17123,"name":"Tom Cocozzello (tjcocozz)","email":"tjcocozz@us.ibm.com","username":"tcocozzello"},"change_message_id":"579781bf25585228bc7df5eb21e43f0706b7188e","unresolved":false,"context_lines":[{"line_number":71,"context_line":""},{"line_number":72,"context_line":"        routers.append("},{"line_number":73,"context_line":"            router.Router(controllers.RoleV3(), \u0027roles\u0027, \u0027role\u0027,"},{"line_number":74,"context_line":"                          resource_descriptions\u003dself.v3_resources))"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"        grant_controller \u003d controllers.GrantAssignmentV3()"},{"line_number":77,"context_line":"        self._add_resource("}],"source_content_type":"text/x-python","patch_set":2,"id":"da20952f_ffe9f31a","line":74,"updated":"2015-09-04 15:00:45.000000000","message":"Can we do a similar change to this router, like the router bellow that you changed?  If not can you explain why?","commit_id":"cc4369eced436fe477f448de052a6d0e6024f793"},{"author":{"_account_id":17123,"name":"Tom Cocozzello (tjcocozz)","email":"tjcocozz@us.ibm.com","username":"tcocozzello"},"change_message_id":"1f1479052f23c78f1ecd2d6172f2d7aeb7c7e43c","unresolved":false,"context_lines":[{"line_number":71,"context_line":""},{"line_number":72,"context_line":"        routers.append("},{"line_number":73,"context_line":"            router.Router(controllers.RoleV3(), \u0027roles\u0027, \u0027role\u0027,"},{"line_number":74,"context_line":"                          resource_descriptions\u003dself.v3_resources))"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"        grant_controller \u003d controllers.GrantAssignmentV3()"},{"line_number":77,"context_line":"        self._add_resource("}],"source_content_type":"text/x-python","patch_set":2,"id":"da20952f_3f552b1d","line":74,"in_reply_to":"da20952f_df0d0f90","updated":"2015-09-04 15:10:34.000000000","message":"Thank you for explaining :)","commit_id":"cc4369eced436fe477f448de052a6d0e6024f793"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"10a049e7f41f104a0a016a39baa5fee7fe3a0d0f","unresolved":false,"context_lines":[{"line_number":71,"context_line":""},{"line_number":72,"context_line":"        routers.append("},{"line_number":73,"context_line":"            router.Router(controllers.RoleV3(), \u0027roles\u0027, \u0027role\u0027,"},{"line_number":74,"context_line":"                          resource_descriptions\u003dself.v3_resources))"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"        grant_controller \u003d controllers.GrantAssignmentV3()"},{"line_number":77,"context_line":"        self._add_resource("}],"source_content_type":"text/x-python","patch_set":2,"id":"da20952f_df0d0f90","line":74,"in_reply_to":"da20952f_ffe9f31a","updated":"2015-09-04 15:04:02.000000000","message":"Well the RolesV3() object is a fully fledged v3 class with get, delete etc....so we want all these routes set up.  For role_assignments, we only have list....and in the follow on patch I want to change this to something other than the standard list_role_assignments.","commit_id":"cc4369eced436fe477f448de052a6d0e6024f793"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"01be5407afcf9eb0abf34e71788b76856a869ed4","unresolved":false,"context_lines":[{"line_number":163,"context_line":"            router.Router(controllers.RoleAssignmentV3(),"},{"line_number":164,"context_line":"                          \u0027role_assignments\u0027, \u0027role_assignment\u0027,"},{"line_number":165,"context_line":"                          resource_descriptions\u003dself.v3_resources,"},{"line_number":166,"context_line":"                          is_entity_implemented\u003dFalse))"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":"        if CONF.os_inherit.enabled:"},{"line_number":169,"context_line":"            self._add_resource("}],"source_content_type":"text/x-python","patch_set":3,"id":"7a2fa921_3894f335","side":"PARENT","line":166,"updated":"2015-10-05 22:11:45.000000000","message":"we should be able to simplify router.Router to get rid of this since this was the only user.","commit_id":"d80df88f4735eaa68113f66423cfffc75a3d6d9c"}]}