)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"89e432f530ba4ec7e8dc47a7b3255e171ffcf30a","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This is the most basic way to move the oauth1 migrations into"},{"line_number":10,"context_line":"the keystone core migrations path. Check to see if the migration was"},{"line_number":11,"context_line":"performed already via the extension and if so skip performing it again."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I8cea6748e7a5ae2b01320b4d6a9866e60ff2ea74"},{"line_number":14,"context_line":"Implements: bp move-extensions"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"da85f559_cbb1bf48","line":12,"updated":"2015-11-11 16:00:00.000000000","message":"This should contain a DocImpact, right? \n\nDocImpact: You should no longer run the migrations for this extension","commit_id":"e5019e138a49e0ba8401a0d71ec1d36bfe2cf241"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"ff6cf8f3998635bea1825e1e6108baff77786e41","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This is the most basic way to move the oauth1 migrations into"},{"line_number":10,"context_line":"the keystone core migrations path. Check to see if the migration was"},{"line_number":11,"context_line":"performed already via the extension and if so skip performing it again."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I8cea6748e7a5ae2b01320b4d6a9866e60ff2ea74"},{"line_number":14,"context_line":"Implements: bp move-extensions"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"ba8a016a_0a0073a9","line":12,"in_reply_to":"da85f559_cbb1bf48","updated":"2015-11-17 19:45:13.000000000","message":"Done","commit_id":"e5019e138a49e0ba8401a0d71ec1d36bfe2cf241"}],"keystone/tests/unit/test_sql_migrate_extensions.py":[{"author":{"_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},"change_message_id":"cef011187b27fcfe30599ac6486b77f00690b74d","unresolved":false,"context_lines":[{"line_number":45,"context_line":"        self.upgrade(1, repository\u003dself.repo_path)"},{"line_number":46,"context_line":"        self.assertTableColumns(\u0027example\u0027, [\u0027id\u0027, \u0027type\u0027, \u0027extra\u0027])"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"class SqlUpgradeOAuth1Extension(test_sql_upgrade.SqlMigrateBase):"},{"line_number":50,"context_line":"    def repo_package(self):"},{"line_number":51,"context_line":"        return oauth1"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a29b11f_7c76d43f","side":"PARENT","line":48,"updated":"2015-10-19 08:57:35.000000000","message":"same question as with federation migrations - shouldn\u0027t we simply move logic somewhere else?","commit_id":"8ed914c570e38de30cb9541ff93107542c49af7b"}],"keystone/tests/unit/test_sql_upgrade.py":[{"author":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"change_message_id":"e60ee50ce5ff83306c330d180b9d7bd551316449","unresolved":false,"context_lines":[{"line_number":657,"context_line":"        mock_oauth1.assert_any_call(extension\u003d\u0027oauth1\u0027, engine\u003dmock.ANY)"},{"line_number":658,"context_line":""},{"line_number":659,"context_line":"        # It won\u0027t exist because we are mocking it, but we can verify"},{"line_number":660,"context_line":"        # that 082 did not create the table."},{"line_number":661,"context_line":"        self.assertTableDoesNotExist(\u0027consumer\u0027)"},{"line_number":662,"context_line":"        self.assertTableDoesNotExist(\u0027request_token\u0027)"},{"line_number":663,"context_line":"        self.assertTableDoesNotExist(\u0027access_token\u0027)"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a26ad4f_8ee80fb6","line":660,"updated":"2015-10-30 03:06:07.000000000","message":"82 should be 83","commit_id":"1689934ac6c8f299a7660ea81b3ccc00a015dfd4"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"19bf6658618a020a34931e51d50ae110801e3997","unresolved":false,"context_lines":[{"line_number":620,"context_line":"        self.assertTableDoesNotExist(\u0027idp_remote_ids\u0027)"},{"line_number":621,"context_line":""},{"line_number":622,"context_line":"    def test_create_oauth_tables(self):"},{"line_number":623,"context_line":"        self.assertTableDoesNotExist(\u0027consumer\u0027)"},{"line_number":624,"context_line":"        self.assertTableDoesNotExist(\u0027request_token\u0027)"},{"line_number":625,"context_line":"        self.assertTableDoesNotExist(\u0027access_token\u0027)"},{"line_number":626,"context_line":"        self.upgrade(83)"}],"source_content_type":"text/x-python","patch_set":10,"id":"da85f559_0b5587f8","line":623,"updated":"2015-11-11 15:57:16.000000000","message":"nit: pull \u0027consumer\u0027, \u0027request_token\u0027, and \u0027access_token\u0027 out in variables since they are used later.","commit_id":"e5019e138a49e0ba8401a0d71ec1d36bfe2cf241"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"ff6cf8f3998635bea1825e1e6108baff77786e41","unresolved":false,"context_lines":[{"line_number":620,"context_line":"        self.assertTableDoesNotExist(\u0027idp_remote_ids\u0027)"},{"line_number":621,"context_line":""},{"line_number":622,"context_line":"    def test_create_oauth_tables(self):"},{"line_number":623,"context_line":"        self.assertTableDoesNotExist(\u0027consumer\u0027)"},{"line_number":624,"context_line":"        self.assertTableDoesNotExist(\u0027request_token\u0027)"},{"line_number":625,"context_line":"        self.assertTableDoesNotExist(\u0027access_token\u0027)"},{"line_number":626,"context_line":"        self.upgrade(83)"}],"source_content_type":"text/x-python","patch_set":10,"id":"ba8a016a_6a59fffd","line":623,"in_reply_to":"da85f559_0b5587f8","updated":"2015-11-17 19:45:13.000000000","message":"Done","commit_id":"e5019e138a49e0ba8401a0d71ec1d36bfe2cf241"}]}