Change the Link implementation in Google Provider
This PR will change the Link implementation in Google Provider to make it clean.
The PR consists of the following:
- Move the not standardized
persistmethod to the base class. - Define
extra_links_paramsmethod in the operator level to pass static arguments (project_id,location, etc) to the link
I was also planning to get rid of all
persistcalls with only the operator properties but I realized that the operator instance (operator: BaseOperator) in theget_linkmethod is different in Airflow 2 and 3. Hence, for Airflow 3, on the operator side we don't need to useLink.persist(context=context)but for Airflow 2 we need to use it. After fully migrating to Airflow 3 we may get rid of all usage ofpersistwithcontext=context
^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.
Hi @potiuk ! Can you please check this one? A big update for the Google provider. Thanks :)
Ineed huge change but easy to review as most of it is the same. Currently Conflicts need to be resolved but I already reviewed it and it's good to go as long as CI agrees.
@potiuk, I think the CI is finally green :) and also thank you for the review 🙇🏼
:tada: