ci: enable alembic check in migration tests

Author: guan404ming

.github/actions/alembic_check/action.yml

@@ -0,0 +1,41 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+#
+---
+name: 'Run Alembic check'
+description: 'Runs Alembic check'
+runs:
+  using: "composite"
+  steps:
+    - name: "Run Alembic check: ${{env.BACKEND}}"
+      shell: bash
+      run: |
+        breeze shell "cd airflow-core/src/airflow && alembic check"
+      env:
+        COMPOSE_PROJECT_NAME: "docker-compose"
+        AIRFLOW__DATABASE__EXTERNAL_DB_MANAGERS: "airflow.providers.fab.auth_manager.models.db.FABDBManager"
+        DB_RESET: "false"
+      if: env.BACKEND != 'sqlite'
+    - name: "Bring any containers left down"
+      shell: bash
+      run: breeze down
+      env:
+        COMPOSE_PROJECT_NAME: "docker-compose"
+    - name: "Dump logs on failure ${{env.BACKEND}}"
+      shell: bash
+      run: docker ps -q | xargs docker logs
+      if: failure()

.github/workflows/ci.yml

@@ -359,6 +359,7 @@ jobs:
         ${{ needs.build-info.outputs.core-test-types-list-as-strings-in-json }}
       include-success-outputs: ${{ needs.build-info.outputs.include-success-outputs }}
       run-migration-tests: "true"
+      run-alembic-check: "true"
       run-coverage: ${{ needs.build-info.outputs.run-coverage }}
       debug-resources: ${{ needs.build-info.outputs.debug-resources }}
       skip-providers-tests: ${{ needs.build-info.outputs.skip-providers-tests }}
@@ -414,6 +415,7 @@ jobs:
       include-success-outputs: ${{ needs.build-info.outputs.include-success-outputs }}
       run-coverage: ${{ needs.build-info.outputs.run-coverage }}
       run-migration-tests: "true"
+      run-alembic-check: "true"
       debug-resources: ${{ needs.build-info.outputs.debug-resources }}
       skip-providers-tests: ${{ needs.build-info.outputs.skip-providers-tests }}
       use-uv: ${{ needs.build-info.outputs.use-uv }}
@@ -471,6 +473,7 @@ jobs:
       include-success-outputs: ${{ needs.build-info.outputs.include-success-outputs }}
       run-coverage: ${{ needs.build-info.outputs.run-coverage }}
       run-migration-tests: "true"
+      run-alembic-check: "true"
       debug-resources: ${{ needs.build-info.outputs.debug-resources }}
       skip-providers-tests: ${{ needs.build-info.outputs.skip-providers-tests }}
       use-uv: ${{ needs.build-info.outputs.use-uv }}

.github/workflows/run-unit-tests.yml

@@ -70,6 +70,11 @@ on:  # yamllint disable-line rule:truthy
         required: false
         default: "false"
         type: string
+      run-alembic-check:
+        description: "Whether to run alembic check or not (true/false)"
+        required: false
+        default: "false"
+        type: string
       run-coverage:
         description: "Whether to run coverage or not (true/false)"
         required: true
@@ -171,6 +176,10 @@ jobs:
           platform: "linux/amd64"
           python: ${{ matrix.python-version }}
           use-uv: ${{ inputs.use-uv }}
+      - name: >
+          Alembic Check: ${{ matrix.python-version }}:${{ env.PARALLEL_TEST_TYPES }}
+        uses: ./.github/actions/alembic_check
+        if: inputs.run-alembic-check == 'true' && inputs.test-group == 'core'
       - name: >
           Migration Tests: ${{ matrix.python-version }}:${{ env.PARALLEL_TEST_TYPES }}
         uses: ./.github/actions/migration_tests

airflow-core/src/airflow/models/dag.py

@@ -2173,7 +2173,7 @@ class DagModel(Base):
     # Description of the dag
     description = Column(Text)
     # Timetable summary
-    timetable_summary = Column(Text, nullable=True)
+    timetable_summary = Column(String(5000), nullable=False)
     # Timetable description
     timetable_description = Column(String(1000), nullable=True)
     # Asset expression based on asset triggers