)]}'
{"neutron/tests/functional/db/test_migrations.py":[{"author":{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},"change_message_id":"ed5c091821e4e0c6d388710acecf8365a50d46a6","unresolved":false,"context_lines":[{"line_number":581,"context_line":"        self._migrate_up(config, engine, \u0027c3a73f615e4\u0027,"},{"line_number":582,"context_line":"                         \u0027dce3ec7a25c9\u0027, True)"},{"line_number":583,"context_line":""},{"line_number":584,"context_line":"    def _pre_upgrade_c3a73f615e4(self, engine):"},{"line_number":585,"context_line":"        connection \u003d engine.connect()"},{"line_number":586,"context_line":"        data \u003d connection.execute(\"show columns from vips\")"},{"line_number":587,"context_line":"        return data"},{"line_number":588,"context_line":""},{"line_number":589,"context_line":"    def _check_c3a73f615e4(self, engine, data):"},{"line_number":590,"context_line":"        pass"},{"line_number":591,"context_line":""},{"line_number":592,"context_line":""},{"line_number":593,"context_line":"class TestWalkMigrationsPsql(_TestWalkMigrations):"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa0719c6_98cdf85f","line":590,"range":{"start_line":584,"start_character":0,"end_line":590,"end_character":12},"updated":"2016-03-23 04:53:48.000000000","message":"The following worked for me:\n\n\n    def _pre_upgrade_c3a73f615e4(self, engine):\n        address_scopes \u003d oslo_utils.get_table(engine, \u0027address_scopes\u0027)\n        return address_scopes.columns\n  \n    def _check_c3a73f615e4(self, engine, pre_upgrade_columns):\n        self.assertNotIn(\u0027ip_version\u0027, pre_upgrade_columns)\n        address_scopes \u003d oslo_utils.get_table(engine, \u0027address_scopes\u0027)\n        post_upgrade_columns \u003d address_scopes.columns\n        self.assertIn(\u0027ip_version\u0027, post_upgrade_columns)","commit_id":"341f8dd5814fe4e2e8e9768e1e07e0db9d45303c"},{"author":{"_account_id":17120,"name":"Manjeet Singh Bhatia","email":"manjeet.s.bhatia@intel.com","username":"manjeets"},"change_message_id":"3bb9336fd0c65613424cc99002349f27bbcb5596","unresolved":false,"context_lines":[{"line_number":581,"context_line":"        self._migrate_up(config, engine, \u0027c3a73f615e4\u0027,"},{"line_number":582,"context_line":"                         \u0027dce3ec7a25c9\u0027, True)"},{"line_number":583,"context_line":""},{"line_number":584,"context_line":"    def _pre_upgrade_c3a73f615e4(self, engine):"},{"line_number":585,"context_line":"        connection \u003d engine.connect()"},{"line_number":586,"context_line":"        data \u003d connection.execute(\"show columns from vips\")"},{"line_number":587,"context_line":"        return data"},{"line_number":588,"context_line":""},{"line_number":589,"context_line":"    def _check_c3a73f615e4(self, engine, data):"},{"line_number":590,"context_line":"        pass"},{"line_number":591,"context_line":""},{"line_number":592,"context_line":""},{"line_number":593,"context_line":"class TestWalkMigrationsPsql(_TestWalkMigrations):"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa0719c6_546bfaf0","line":590,"range":{"start_line":584,"start_character":0,"end_line":590,"end_character":12},"in_reply_to":"fa0719c6_98cdf85f","updated":"2016-03-23 19:01:03.000000000","message":"Used assertColumninTable instead of \nusing \n\naddress_scopes \u003d oslo_utils.get_table(engine, \u0027address_scopes\u0027)\n        post_upgrade_columns \u003d address_scopes.columns\n        self.assertIn(\u0027ip_version\u0027, post_upgrade_columns)","commit_id":"341f8dd5814fe4e2e8e9768e1e07e0db9d45303c"},{"author":{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},"change_message_id":"a06d461664d30b09caf9c53c153946067ae41bde","unresolved":false,"context_lines":[{"line_number":575,"context_line":"        self._reset_databases()"},{"line_number":576,"context_line":"        self._walk_versions(config, engine, False, False)"},{"line_number":577,"context_line":""},{"line_number":578,"context_line":"    def test_mysql_revise(self):"},{"line_number":579,"context_line":"        connect_string \u003d oslo_utils.get_connect_string(self.BACKEND,"},{"line_number":580,"context_line":"            \"openstack_citest\", user\u003d\"openstack_citest\","},{"line_number":581,"context_line":"            passwd\u003d\"openstack_citest\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"da0c15f0_2181804b","line":578,"updated":"2016-03-25 02:49:29.000000000","message":"Make this a private method that takes dest and curr as parameters.\n\n    def _test_mysql_revise(self, dest, curr):","commit_id":"3abba021adb8b7fa1a66a62f36a39af4d4b7a0dc"},{"author":{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},"change_message_id":"a06d461664d30b09caf9c53c153946067ae41bde","unresolved":false,"context_lines":[{"line_number":584,"context_line":"        self.engines[\"mysqlcitest\"] \u003d engine"},{"line_number":585,"context_line":"        self.test_databases[\"mysqlcitest\"] \u003d connect_string"},{"line_number":586,"context_line":"        self._reset_databases()"},{"line_number":587,"context_line":"        self._migrate_up(config, engine, \u0027c3a73f615e4\u0027,"},{"line_number":588,"context_line":"                         \u0027dce3ec7a25c9\u0027, True)"},{"line_number":589,"context_line":""},{"line_number":590,"context_line":"    def _pre_upgrade_c3a73f615e4(self, engine):"},{"line_number":591,"context_line":"        address_scopes \u003d oslo_utils.get_table(engine, \u0027address_scopes\u0027)"}],"source_content_type":"text/x-python","patch_set":5,"id":"da0c15f0_a18d907d","line":588,"range":{"start_line":587,"start_character":0,"end_line":588,"end_character":46},"updated":"2016-03-25 02:49:29.000000000","message":"Replace with the following:\n\n\n    # First migrate the schema from base to curr\n    self._migrate_up(config, engine, dest\u003dcurr, curr\u003dNone)\n    # Now migrate from curr to dest with data\n    self._migrate_up(config, engine, dest\u003ddest, curr\u003dcurr, with_data\u003dTrue)","commit_id":"3abba021adb8b7fa1a66a62f36a39af4d4b7a0dc"},{"author":{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},"change_message_id":"a06d461664d30b09caf9c53c153946067ae41bde","unresolved":false,"context_lines":[{"line_number":586,"context_line":"        self._reset_databases()"},{"line_number":587,"context_line":"        self._migrate_up(config, engine, \u0027c3a73f615e4\u0027,"},{"line_number":588,"context_line":"                         \u0027dce3ec7a25c9\u0027, True)"},{"line_number":589,"context_line":""},{"line_number":590,"context_line":"    def _pre_upgrade_c3a73f615e4(self, engine):"},{"line_number":591,"context_line":"        address_scopes \u003d oslo_utils.get_table(engine, \u0027address_scopes\u0027)"},{"line_number":592,"context_line":"        return address_scopes.columns"}],"source_content_type":"text/x-python","patch_set":5,"id":"da0c15f0_e1d4a848","line":589,"updated":"2016-03-25 02:49:29.000000000","message":"Create the test case like this:\n\n  def test_revise_dce3ec7a25c9_to_c3a73f615e4(self):\n      self._test_mysql_revise(dest\u003d\u0027c3a73f615e4\u0027, curr\u003d\u0027dce3ec7a25c9\u0027)","commit_id":"3abba021adb8b7fa1a66a62f36a39af4d4b7a0dc"},{"author":{"_account_id":17120,"name":"Manjeet Singh Bhatia","email":"manjeet.s.bhatia@intel.com","username":"manjeets"},"change_message_id":"262b6d8d78124bcb8cc8efa93a4c35601552bdb2","unresolved":false,"context_lines":[{"line_number":11,"context_line":"#    WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the"},{"line_number":12,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":13,"context_line":"#    under the License."},{"line_number":14,"context_line":"import pdb"},{"line_number":15,"context_line":"TOX \u003d True  # disable breakpoints when straight tox"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"import abc"}],"source_content_type":"text/x-python","patch_set":9,"id":"9a061dce_953c6ec9","line":14,"range":{"start_line":14,"start_character":1,"end_line":14,"end_character":9},"updated":"2016-04-06 23:59:04.000000000","message":"this should be removed !","commit_id":"e767ac161119baf7020e21a53da49082531d1350"},{"author":{"_account_id":17120,"name":"Manjeet Singh Bhatia","email":"manjeet.s.bhatia@intel.com","username":"manjeets"},"change_message_id":"262b6d8d78124bcb8cc8efa93a4c35601552bdb2","unresolved":false,"context_lines":[{"line_number":693,"context_line":""},{"line_number":694,"context_line":"class TestWalkMigrationsDualBackends(TestWalkMigrationsMysql,"},{"line_number":695,"context_line":"                                     TestWalkMigrationsPsql,"},{"line_number":696,"context_line":"                                     _TestWalkMigrations):"},{"line_number":697,"context_line":"    if not TOX:"},{"line_number":698,"context_line":"        pdb.set_trace()"},{"line_number":699,"context_line":"    # BACKEND \u003d [\u0027postgresql\u0027, \u0027mysql+pymysql\u0027]"}],"source_content_type":"text/x-python","patch_set":9,"id":"9a061dce_7540ba34","line":696,"range":{"start_line":696,"start_character":37,"end_line":696,"end_character":56},"updated":"2016-04-06 23:59:04.000000000","message":"not sure if you need to add this i guess its parent class already inherited ones.","commit_id":"e767ac161119baf7020e21a53da49082531d1350"}]}