-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SDK - Using Airflow ops in Pipelines #1483
SDK - Using Airflow ops in Pipelines #1483
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR. This is really great that KFP users can directly use Airflow operators in their pipeline.
ad984bb
to
a5b0644
Compare
/test kubeflow-pipeline-sample-test |
@Ark-kun is this only for bash operator? Other operators being supported as well? Also can this be extended to take an overall Airflow DAG, and execute as one of the 'steps' in KFP, assuming an Airflow deployment exists and KFP is made aware of it? |
@paveldournov you have a viewpoint on the above question? cc @vicaire @jlewi |
This feature supports all Airflow operators.
It could. How do you envision the API for AirFlow DAGs?
This might be unnecessary |
/test kubeflow-pipeline-e2e-test |
/cc @hongye-sun I've added documentation for this functionality. |
/lgtm |
Two modes
|
Airflow has removed most of the old images and tags. See https://issues.apache.org/jira/browse/AIRFLOW-5093 and https://cwiki.apache.org/confluence/display/AIRFLOW/AIP-10+Multi-layered+and+multi-stage+official+Airflow+CI+image#AIP-10Multi-layeredandmulti-stageofficialAirflowCIimage-ProposedsetupoftheDockerHubandTravisCI .
/lgtm |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: Ark-kun The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Hi, I’m an Airflow user, now i want to create components by using create_component_from_airflow_op(SimpleHttpOperator), but i do not konw how to define the airflow‘s conn such as msyql conn or http conn, can u give me an example? thx |
This CL adds support for converting arbitrary Airflow operators to Pipelines components so that it can be used as a pipeline task.
This change is