)]}'
{"keystone/contrib/federation/migrate_repo/versions/007_add_remote_id_table.py":[{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"c4c5e5b86395e2766d9cc90b2935fb811240e5e5","unresolved":false,"context_lines":[{"line_number":35,"context_line":"    select \u003d orm.sql.select([idp_table.c.id, idp_table.c.remote_id])"},{"line_number":36,"context_line":"    for identity in migrate_engine.execute(select):"},{"line_number":37,"context_line":"        if not identity.remote_id:"},{"line_number":38,"context_line":"            continue"},{"line_number":39,"context_line":"        remote_idp_entry \u003d {\u0027idp_id\u0027: identity.id,"},{"line_number":40,"context_line":"                            \u0027remote_id\u0027: identity.remote_id}"},{"line_number":41,"context_line":"        remote_id_table.insert(remote_idp_entry).execute()"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a50d1a3_36bbfae6","line":38,"updated":"2015-07-28 16:47:29.000000000","message":"this will work, but you could also just add a WHERE clause to the select object on L35 to let SQL do the lifting","commit_id":"bc8e4d0c6d4264bd00cbd1dddfcd032a51ea6f16"},{"author":{"_account_id":8623,"name":"Kaitlin Farr","email":"kaitlin.farr@jhuapl.edu","username":"kaitlin.farr"},"change_message_id":"f770006af82506b6a520f050e1ddf5a4c4fb1bd6","unresolved":false,"context_lines":[{"line_number":35,"context_line":"    select \u003d orm.sql.select([idp_table.c.id, idp_table.c.remote_id])"},{"line_number":36,"context_line":"    for identity in migrate_engine.execute(select):"},{"line_number":37,"context_line":"        if not identity.remote_id:"},{"line_number":38,"context_line":"            continue"},{"line_number":39,"context_line":"        remote_idp_entry \u003d {\u0027idp_id\u0027: identity.id,"},{"line_number":40,"context_line":"                            \u0027remote_id\u0027: identity.remote_id}"},{"line_number":41,"context_line":"        remote_id_table.insert(remote_idp_entry).execute()"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a50d1a3_f1a11cf8","line":38,"in_reply_to":"3a50d1a3_36bbfae6","updated":"2015-07-28 16:59:16.000000000","message":"+1","commit_id":"bc8e4d0c6d4264bd00cbd1dddfcd032a51ea6f16"},{"author":{"_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},"change_message_id":"e2bc9773457cc8f21ef83bef350028f8d5b2eb47","unresolved":false,"context_lines":[{"line_number":35,"context_line":"    select \u003d orm.sql.select([idp_table.c.id, idp_table.c.remote_id])"},{"line_number":36,"context_line":"    for identity in migrate_engine.execute(select):"},{"line_number":37,"context_line":"        if not identity.remote_id:"},{"line_number":38,"context_line":"            continue"},{"line_number":39,"context_line":"        remote_idp_entry \u003d {\u0027idp_id\u0027: identity.id,"},{"line_number":40,"context_line":"                            \u0027remote_id\u0027: identity.remote_id}"},{"line_number":41,"context_line":"        remote_id_table.insert(remote_idp_entry).execute()"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a50d1a3_6e48c2b3","line":38,"in_reply_to":"3a50d1a3_36bbfae6","updated":"2015-07-29 06:45:45.000000000","message":"yeah, actually you are right!","commit_id":"bc8e4d0c6d4264bd00cbd1dddfcd032a51ea6f16"},{"author":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"change_message_id":"b4ae217823e2b4e18c03a4499d595e808330bfc9","unresolved":false,"context_lines":[{"line_number":35,"context_line":"    select \u003d orm.sql.select([idp_table.c.id, idp_table.c.remote_id])"},{"line_number":36,"context_line":"    for identity in migrate_engine.execute(select):"},{"line_number":37,"context_line":"        if not identity.remote_id:"},{"line_number":38,"context_line":"            continue"},{"line_number":39,"context_line":"        remote_idp_entry \u003d {\u0027idp_id\u0027: identity.id,"},{"line_number":40,"context_line":"                            \u0027remote_id\u0027: identity.remote_id}"},{"line_number":41,"context_line":"        remote_id_table.insert(remote_idp_entry).execute()"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a50d1a3_9ce53339","line":38,"in_reply_to":"3a50d1a3_f1a11cf8","updated":"2015-07-29 06:49:44.000000000","message":"++","commit_id":"bc8e4d0c6d4264bd00cbd1dddfcd032a51ea6f16"}],"keystone/tests/unit/test_sql_migrate_extensions.py":[{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"b5cec511449d80c529b454a33411977537ba79cf","unresolved":false,"context_lines":[{"line_number":312,"context_line":"        self.assertEqual(\u0027\u0027, sp.sp_url)"},{"line_number":313,"context_line":""},{"line_number":314,"context_line":"    def test_propagate_remote_id_to_separate_column(self):"},{"line_number":315,"context_line":"        \"\"\"Make sure empty remote_id is not propagated."},{"line_number":316,"context_line":"        Test scenario:"},{"line_number":317,"context_line":"        - Upgrade database to version 6 where identity_provider table has a"},{"line_number":318,"context_line":"          remote_id column"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a1a9d01_cafc2a74","line":315,"updated":"2015-09-30 21:47:51.000000000","message":"p.s. i wish we had pep257 enforcement","commit_id":"1438cfeb4cf56773998e03c9801efee5f9673a75"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"240b983614d7b5805c3444c8cf466e58141a6580","unresolved":false,"context_lines":[{"line_number":370,"context_line":"            remote_id_table.c.idp_id \u003d\u003d idp3[\u0027id\u0027])"},{"line_number":371,"context_line":"        # NOTE(marek-denis): As idp3 had empty \u0027remote_id\u0027 attribute we expect"},{"line_number":372,"context_line":"        # not to find it in the \u0027remote_id_table\u0027 table, hence count should be"},{"line_number":373,"context_line":"        # 0.real"},{"line_number":374,"context_line":"        self.assertEqual(0, idp.count())"},{"line_number":375,"context_line":""},{"line_number":376,"context_line":"    def test_add_relay_state_column(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a1a9d01_0ae0e2cb","line":373,"updated":"2015-09-30 21:46:19.000000000","message":"keepin it real, i see","commit_id":"1438cfeb4cf56773998e03c9801efee5f9673a75"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"e1124bace2ede0e0eaf5f934f930d22ec5db9ed9","unresolved":false,"context_lines":[{"line_number":370,"context_line":"            remote_id_table.c.idp_id \u003d\u003d idp3[\u0027id\u0027])"},{"line_number":371,"context_line":"        # NOTE(marek-denis): As idp3 had empty \u0027remote_id\u0027 attribute we expect"},{"line_number":372,"context_line":"        # not to find it in the \u0027remote_id_table\u0027 table, hence count should be"},{"line_number":373,"context_line":"        # 0.real"},{"line_number":374,"context_line":"        self.assertEqual(0, idp.count())"},{"line_number":375,"context_line":""},{"line_number":376,"context_line":"    def test_add_relay_state_column(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a1a9d01_6ad5dee2","line":373,"updated":"2015-09-30 21:47:16.000000000","message":"not sure why the .real is here","commit_id":"1438cfeb4cf56773998e03c9801efee5f9673a75"}]}