jason810496 commented on PR #47433: URL: https://github.com/apache/airflow/pull/47433#issuecomment-2703749450
> As expected full tests break and need fixing. Also while testing the UI I realized that the server is not happy with this change (can't load dags internal error 500) and we will need to wait for @vincbeck fix. `FabAuthManager` is crashing on `get_permitted_dag_ids`. This is a bug in the auth manager implementation and not detected by our tests at the moment because unit tests use the `SimpleAuthManager` where the dev setup use `FabAuthManager` by default. Just to double-check, do you mean that for this PR, I don’t need to make any changes but should wait for `FabAuthManager` to be fixed and then rebase to verify everything works correctly? If that’s the case, should https://github.com/apache/airflow/pull/47388 also be reverted? ( Since you mentioned that the dev setup use `FabAuthManager` by default, I initially thought the issue was caused by not adding a JWT token to the API client. ) -- 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]
