adjust DB schema revision path

gnocchixyz · Feb 13, 2025 · 681eec9 · 681eec9
1 parent 2c35409
commit 681eec9
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 12 deletions.
diff --git a/gnocchi/indexer/alembic/versions/f89ed2e3c2ec_create_last_measure_timestamp_column.py b/gnocchi/indexer/alembic/versions/f89ed2e3c2ec_create_last_measure_timestamp_column.py
@@ -15,7 +15,7 @@
 
 """Create last measure push timestamp column
 Revision ID: f89ed2e3c2ec
-Revises: 18fff4509e3e
+Revises: 94544ca86c7e
 Create Date: 2024-04-24 09:16:00
 """
 
@@ -26,7 +26,7 @@
 
 # revision identifiers, used by Alembic.
 revision = 'f89ed2e3c2ec'
-down_revision = '18fff4509e3e'
+down_revision = '94544ca86c7e'
 branch_labels = None
 depends_on = None
 

diff --git a/gnocchi/tests/indexer/sqlalchemy/test_migrations.py b/gnocchi/tests/indexer/sqlalchemy/test_migrations.py
@@ -92,18 +92,20 @@ def compare_server_default(self, ctxt, ins_col, meta_col, insp_def, meta_def, re
         :param meta_def: column default value from model
         :param rendered_meta_def: rendered column default value (from model)
 
+        When the column has server_default=sqlalchemy.sql.func.now(), the diff includes the followings diff
+         [ [ ( 'modify_default',
+               None,
+               'metric',
+               'last_measure_timestamp',
+               { 'existing_comment': None,
+                 'existing_nullable': False,
+                 'existing_type': DATETIME()},
+               DefaultClause(<sqlalchemy.sql.elements.TextClause object at 0x7f0100b24b50>, for_update=False),
+               DefaultClause(<sqlalchemy.sql.functions.now at 0x7f01010b08d0; now>, for_update=False))]]
+
         """
 
-        # When the column has server_default=sqlalchemy.sql.func.now(), the diff includes the followings diff
-        # [ [ ( 'modify_default',
-        #       None,
-        #       'metric',
-        #       'last_measure_timestamp',
-        #       { 'existing_comment': None,
-        #         'existing_nullable': False,
-        #         'existing_type': DATETIME()},
-        #       DefaultClause(<sqlalchemy.sql.elements.TextClause object at 0x7f0100b24b50>, for_update=False),
-        #       DefaultClause(<sqlalchemy.sql.functions.now at 0x7f01010b08d0; now>, for_update=False))]]
+
         method_return = super(ModelsMigrationsSync, self).compare_server_default(ctxt, ins_col, meta_col, insp_def,
                                                                                  meta_def, rendered_meta_def)