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
joaopamaral · Oct 21, 2024 · c507c7a · c507c7a
1 parent dd8258f
commit c507c7a
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