)]}'
{"keystone/identity/mapping_backends/sql.py":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"bde5da1a1a552e4acf5fa56f574a4d63dc4aa10b","unresolved":false,"context_lines":[{"line_number":110,"context_line":"            for row in rows:"},{"line_number":111,"context_line":"                if row.entity_type \u003d\u003d identity_mapping.EntityType.GROUP:"},{"line_number":112,"context_line":"                    try:"},{"line_number":113,"context_line":"                        self.identity_api.get_group(row.public_id)"},{"line_number":114,"context_line":"                    except exception.UserNotFound:"},{"line_number":115,"context_line":"                        projects \u003d self.assignment_api.list_projects_for_group("},{"line_number":116,"context_line":"                            row.public_id"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff346bd7_137c6070","line":113,"updated":"2017-07-26 20:55:24.000000000","message":"Is it possible to invoke these bits from keystone-manage or cli.py instead of in a driver? It\u0027s odd to have a backend have this many dependencies on managers.","commit_id":"29293af2ab839544f1bd8bc7e505e5eec5d76a71"},{"author":{"_account_id":16465,"name":"Kristi Nikolla","email":"knikolla@bu.edu","username":"knikolla"},"change_message_id":"b1426c0475b88ee30781906ddae7c7ce6db9ed13","unresolved":false,"context_lines":[{"line_number":110,"context_line":"            for row in rows:"},{"line_number":111,"context_line":"                if row.entity_type \u003d\u003d identity_mapping.EntityType.GROUP:"},{"line_number":112,"context_line":"                    try:"},{"line_number":113,"context_line":"                        self.identity_api.get_group(row.public_id)"},{"line_number":114,"context_line":"                    except exception.UserNotFound:"},{"line_number":115,"context_line":"                        projects \u003d self.assignment_api.list_projects_for_group("},{"line_number":116,"context_line":"                            row.public_id"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff346bd7_73c4cc5f","line":113,"in_reply_to":"ff346bd7_137c6070","updated":"2017-07-26 21:04:08.000000000","message":"Will look into it.","commit_id":"29293af2ab839544f1bd8bc7e505e5eec5d76a71"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"5082193854e685d7354cf3bdb5b8f1ce6ce8abb6","unresolved":false,"context_lines":[{"line_number":109,"context_line":"                for row in rows:"},{"line_number":110,"context_line":"                    if row.entity_type \u003d\u003d identity_mapping.EntityType.GROUP:"},{"line_number":111,"context_line":"                        try:"},{"line_number":112,"context_line":"                            self.identity_api.get_group(row.public_id)"},{"line_number":113,"context_line":"                            # Note(knikolla): If we get past the above call"},{"line_number":114,"context_line":"                            # without an exception, the mapping is still"},{"line_number":115,"context_line":"                            # valid and we remove it from the list."}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa0c359_e0ea5a0d","line":112,"range":{"start_line":112,"start_character":28,"end_line":112,"end_character":33},"updated":"2018-02-08 22:58:56.000000000","message":"This needs to be refactored since we should be using PROVIDERS. We also should try and work this type of logic into the manager since calling a manager from the backend isn\u0027t really something we should be doing at this layer.","commit_id":"5f071a8aaf13fdaea4ae9c905e9b7eeddd0c24cf"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"5082193854e685d7354cf3bdb5b8f1ce6ce8abb6","unresolved":false,"context_lines":[{"line_number":121,"context_line":""},{"line_number":122,"context_line":"                    elif row.entity_type \u003d\u003d identity_mapping.EntityType.USER:"},{"line_number":123,"context_line":"                        try:"},{"line_number":124,"context_line":"                            self.identity_api.get_user(row.public_id)"},{"line_number":125,"context_line":"                            query \u003d query.filter("},{"line_number":126,"context_line":"                                IDMapping.public_id !\u003d row.public_id"},{"line_number":127,"context_line":"                            )"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa0c359_00f0a6fd","line":124,"updated":"2018-02-08 22:58:56.000000000","message":"Same comment here.","commit_id":"5f071a8aaf13fdaea4ae9c905e9b7eeddd0c24cf"}]}