)]}'
{"keystone/identity/backends/ldap/core.py":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"7a78685d8090b40aa726fa110ce3da7a6bc1c512","unresolved":false,"context_lines":[{"line_number":644,"context_line":"    NotFound \u003d exception.RoleNotFound"},{"line_number":645,"context_line":"    options_name \u003d \u0027role\u0027"},{"line_number":646,"context_line":"    attribute_mapping \u003d {\u0027name\u0027: \u0027ou\u0027,"},{"line_number":647,"context_line":"                         #\u0027serviceId\u0027: \u0027service_id\u0027,"},{"line_number":648,"context_line":"                         }"},{"line_number":649,"context_line":"    model \u003d models.Role"},{"line_number":650,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPX%2F%2Fv88%3D","line":647,"updated":"2013-06-11 13:15:21.000000000","message":"Should line 647 be removed? Maybe in a separate commit?","commit_id":"bba4ea7177d3f943e763bde88538f1085a854e30"},{"author":{"_account_id":6456,"name":"Sahdev Zala","email":"spzala@us.ibm.com","username":"spzala"},"change_message_id":"bdda5796c8b8ccd54c097a2d76da6f0468a84934","unresolved":false,"context_lines":[{"line_number":644,"context_line":"    NotFound \u003d exception.RoleNotFound"},{"line_number":645,"context_line":"    options_name \u003d \u0027role\u0027"},{"line_number":646,"context_line":"    attribute_mapping \u003d {\u0027name\u0027: \u0027ou\u0027,"},{"line_number":647,"context_line":"                         #\u0027serviceId\u0027: \u0027service_id\u0027,"},{"line_number":648,"context_line":"                         }"},{"line_number":649,"context_line":"    model \u003d models.Role"},{"line_number":650,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPX%2F%2Fv6g%3D","line":647,"in_reply_to":"AAAAPX%2F%2Fv88%3D","updated":"2013-06-14 15:27:05.000000000","message":"Hi Lance, thanks for the review. I am not sure if line 647 is commented out for any reference purposes, but I believe nothing wrong if you want to open a bug to remove the line and see what feedback you get.","commit_id":"bba4ea7177d3f943e763bde88538f1085a854e30"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"1047f6052fea80721cf2a9cb6e0520f048fab951","unresolved":false,"context_lines":[{"line_number":644,"context_line":"    NotFound \u003d exception.RoleNotFound"},{"line_number":645,"context_line":"    options_name \u003d \u0027role\u0027"},{"line_number":646,"context_line":"    attribute_mapping \u003d {\u0027name\u0027: \u0027ou\u0027,"},{"line_number":647,"context_line":"                         #\u0027serviceId\u0027: \u0027service_id\u0027,"},{"line_number":648,"context_line":"                         }"},{"line_number":649,"context_line":"    model \u003d models.Role"},{"line_number":650,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPX%2F%2FmqE%3D","line":647,"in_reply_to":"AAAAPX%2F%2Fv88%3D","updated":"2013-06-14 15:14:06.000000000","message":"yeah commented out code should never make it through review...","commit_id":"bba4ea7177d3f943e763bde88538f1085a854e30"}]}