jscheffl commented on issue #64886:
URL: https://github.com/apache/airflow/issues/64886#issuecomment-4209442768

   Thanks Jarek for preparing providers!
   
   +1 (binding) - Checked SVN, Check in Docker, Reproducible package build, 
Licenses, Signatures
   
   Had a bit of challenges in testing as two bugs blocked me: (1) the cooldown 
period prevented me to test in docker as "new pre-release packages" were not 
found in docker build and (2) there is a broken symlink in the repo which made 
RAT license check fail. Fixed in PR https://github.com/apache/airflow/pull/64920
   
   But this is non blocking as the providers itselfs are good.
   
   Tested my usual round via breeze with Airflow 3.0.6, 3.1.8, 3.2.0, main and 
EdgeExecutor + FAB with Integration Test Dag. All looking good.
   
   Minor nit / glitches I saw:
   
   - Running all providers and also common.ai in Airflow 3.0.6 with 
SimpleAuthManager as well as FabAuthMgr generated the exception (is printed in 
stdout as well as in all task logs!):
   ```
       [2026-04-08T20:05:08.273+0000] {plugins_manager.py:262} ERROR - Failed 
to import plugin hitl_review
       Traceback (most recent call last):
         File 
"/usr/python/lib/python3.12/site-packages/airflow/plugins_manager.py", line 
254, in load_entrypoint_plugins
           plugin_class = entry_point.load()
                          ^^^^^^^^^^^^^^^^^^
         File "/usr/python/lib/python3.12/importlib/metadata/__init__.py", line 
205, in load
           module = import_module(match.group('module'))
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
         File "/usr/python/lib/python3.12/importlib/__init__.py", line 90, in 
import_module
           return _bootstrap._gcd_import(name[level:], package, level)
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
         File "<frozen importlib._bootstrap>", line 1387, in _gcd_import
         File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
         File "<frozen importlib._bootstrap>", line 1331, in 
_find_and_load_unlocked
         File "<frozen importlib._bootstrap>", line 935, in _load_unlocked
         File "<frozen importlib._bootstrap_external>", line 999, in exec_module
         File "<frozen importlib._bootstrap>", line 488, in 
_call_with_frames_removed
         File 
"/usr/python/lib/python3.12/site-packages/airflow/providers/common/ai/plugins/hitl_review.py",
 line 252, in <module>
           dependencies=[Depends(requires_access_dag(method="GET", 
access_entity=DagAccessEntity.HITL_DETAIL))],
                                                                                
 ^^^^^^^^^^^^^^^^^^^^^^^^^^^
       AttributeError: type object 'DagAccessEntity' has no attribute 
'HITL_DETAIL'
   ```
   
   To remove the glitch maybe we should mark common-ai only be supporting 
Airflow 3.1.0+?
   
   Jens


-- 
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]

Reply via email to