Fix pre-commit for auto update of fab migration versions (apache#42382)

* Fix pre-commit for auto update of fab migration versions This fixes the pre-commit for updating the version in FAB migration and also, updated the actual version as 1.4 since that's going to be the next version At the same time, added Daniel's fix for airflow alembic.ini file * fixup! Fix pre-commit for auto update of fab migration versions * fixup! fixup! Fix pre-commit for auto update of fab migration versions
aws-mwaa · Sep 20, 2024 · 6a527c9 · 6a527c9
1 parent 8741e9c
commit 6a527c9
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 5 deletions.
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -510,7 +510,9 @@ repos:
           (?x)
           ^scripts/ci/pre_commit/version_heads_map\.py$|
           ^airflow/migrations/versions/.*$|^airflow/migrations/versions|
-          ^airflow/utils/db.py$
+          ^airflow/providers/fab/migrations/versions/.*$|^airflow/providers/fab/migrations/versions|
+          ^airflow/utils/db.py$|
+          ^airflow/providers/fab/auth_manager/models/db.py$
         additional_dependencies: ['packaging','google-re2']
       - id: update-version
         name: Update versions in docs

diff --git a/airflow/providers/fab/alembic.ini b/airflow/providers/fab/alembic.ini
@@ -20,7 +20,7 @@
 [alembic]
 # path to migration scripts
 # Use forward slashes (/) also on windows to provide an os agnostic path
-script_location = migrations
+script_location = %(here)s/migrations
 
 # template used to generate migration file names; The default value is %%(rev)s_%%(slug)s
 # Uncomment the line below if you want the files to be prepended with date and time

diff --git a/airflow/providers/fab/auth_manager/models/db.py b/airflow/providers/fab/auth_manager/models/db.py
@@ -28,7 +28,7 @@
 PACKAGE_DIR = os.path.dirname(airflow.__file__)
 
 _REVISION_HEADS_MAP: dict[str, str] = {
-    "1.3.0": "6709f7a774b9",
+    "1.4.0": "6709f7a774b9",
 }
 
 

diff --git a/...sions/0001_1_3_0_placeholder_migration.py → ...sions/0001_1_4_0_placeholder_migration.py b/...sions/0001_1_3_0_placeholder_migration.py → ...sions/0001_1_4_0_placeholder_migration.py
@@ -36,7 +36,7 @@
 down_revision = None
 branch_labels = None
 depends_on = None
-fab_version = "1.3.0"
+fab_version = "1.4.0"
 
 
 def upgrade() -> None: ...

diff --git a/docs/apache-airflow-providers-fab/migrations-ref.rst b/docs/apache-airflow-providers-fab/migrations-ref.rst
@@ -34,7 +34,7 @@ Here's the list of all the Database Migrations that are executed via when you ru
 +-------------------------+--------------+---------------+------------------------+
 | Revision ID             | Revises ID   | Fab Version   | Description            |
 +=========================+==============+===============+========================+
-| ``6709f7a774b9`` (head) | ``None``     | ``1.3.0``     | placeholder migration. |
+| ``6709f7a774b9`` (head) | ``None``     | ``1.4.0``     | placeholder migration. |
 +-------------------------+--------------+---------------+------------------------+
 
  .. End of auto-generated table