ephraimbuddy commented on PR #35926:
URL: https://github.com/apache/airflow/pull/35926#issuecomment-1857875064

   > > I am happy to do those cherry-picks, if it makes things easier :)
   > 
   > Can this PR be cherry picked to 2.8 branch? It changed import path in 
existed migration script 
`airflow/migrations/versions/0073_2_0_0_prefix_dag_permissions.py` to import 
from fab provider but fab provider can be used only with airflow>=2.9.0 If we 
cherry pick this PR to 2.8 branch thus it will be part of 2.8.x release, won't 
it fail any airflow db upgrade ?
   
   We shouldn't add more PRs to 2.8 branch. It's late now. 
   And concerning future cherrypicks, it's not to cherry-pick this particular 
one but I'm just thinking that if there's an issue in Auth manager in 2.8.0, 
how do we fix it for 2.8.1 since the authmanager is now moved to the provider. 
That's my worry but hopefully, we won't have issues.


-- 
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: commits-unsubscr...@airflow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to