LarytheLord opened a new pull request, #62954:
URL: https://github.com/apache/airflow/pull/62954
## Summary
Fixes #55140.
When two plugins share the same `AirflowPlugin.name`, Airflow currently logs
a warning and silently skips the duplicate. This can hide the real problem from
users (especially when only one plugin appears in UI/API).
This change keeps the current non-blocking behavior (subsequent plugins
still load), but also records duplicate plugin-name conflicts as import errors
so the issue is visible and actionable.
## Changes
- In `airflow.plugins_manager._get_plugins()`:
- track loaded plugin names with their source
- on duplicate name:
- keep warning log
- add an explicit entry to `import_errors`
- Update unit test coverage in `test_plugins_manager.py`:
-
`test_duplicate_plugin_name_is_reported_and_does_not_prevent_loading_subsequent_plugins`
- verifies duplicate is reported in `import_errors` while unique plugins
still load
## Why this approach
- Surfaces the conflict as a concrete error instead of only a warning.
- Preserves existing resilience: one bad/duplicate plugin does not block
other plugins.
--
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]
