ColtenOuO opened a new pull request, #71334:
URL: https://github.com/apache/airflow/pull/71334
### Sumarry
Two steps in `.github/actions/migration_tests/action.yml` reference
`${DB_MANGERS}`
while the surrounding `env:` block defines `DB_MANAGERS`. The variable is
undefined,
so the export resolves to `AIRFLOW__DATABASE__EXTERNAL_DB_MANAGERS=`
(empty). The
other two steps in the same file spell it correctly.
Nothing fails: GitHub's default `shell: bash` runs without `-u`, `export X=`
on its
own line is valid, and the migration commands on the next line run normally
— so the
steps have stayed green since the typo was introduced in #50343.
**Current impact: none.**
Since #62308, `RunDBManager` starts from DB managers
auto-discovered via `provider.yaml`, and the FAB provider declares its own,
so the
empty config is simply skipped. This is a cleanup: it removes a silent
failure that
would come back if discovery were ever not in play, and stops the file
contradicting
itself.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
