Skip to content
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

[dagster-airlift][dag] in-airflow dag proxying behavior #25160

Merged
merged 2 commits into from
Oct 14, 2024

Conversation

dpeng817
Copy link
Contributor

@dpeng817 dpeng817 commented Oct 9, 2024

Summary & Motivation

Adds everything we need to get dag-level proxying working airflow side. Namely a new operator type and handling within proxying_to_dagster
Implementation here more or less matches what was in #25013.

How I Tested These Changes

Added a kitchen sink test for the behavior.

Changelog

NOCHANGELOG

Copy link
Contributor Author

dpeng817 commented Oct 9, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @dpeng817 and the rest of your teammates on Graphite Graphite

@dpeng817 dpeng817 changed the title [dagster-airlift] operator to proxy dag [dagster-airlift] in-airflow dag proxying behavior Oct 9, 2024
@dpeng817 dpeng817 requested a review from schrockn October 9, 2024 17:50
@dpeng817 dpeng817 marked this pull request as ready for review October 9, 2024 17:50
@dpeng817 dpeng817 changed the title [dagster-airlift] in-airflow dag proxying behavior [dagster-airlift][dag] in-airflow dag proxying behavior Oct 9, 2024
@dpeng817 dpeng817 force-pushed the dpeng817/proxied_state_parsing branch from d58ce1f to 9bbeafe Compare October 9, 2024 22:10
@dpeng817 dpeng817 force-pushed the dpeng817/in_airflow_dag_override branch from 1aecb3f to ce3351b Compare October 9, 2024 22:10
@dpeng817 dpeng817 force-pushed the dpeng817/proxied_state_parsing branch from 9bbeafe to 3e90231 Compare October 10, 2024 16:18
@dpeng817 dpeng817 force-pushed the dpeng817/in_airflow_dag_override branch from ce3351b to fec7df8 Compare October 10, 2024 16:18
@dpeng817 dpeng817 force-pushed the dpeng817/proxied_state_parsing branch from 3e90231 to e5b6ca6 Compare October 10, 2024 16:25
@dpeng817 dpeng817 force-pushed the dpeng817/in_airflow_dag_override branch from fec7df8 to 8137f6b Compare October 10, 2024 16:26
@dpeng817 dpeng817 force-pushed the dpeng817/proxied_state_parsing branch from e5b6ca6 to d1e7936 Compare October 10, 2024 16:34
@dpeng817 dpeng817 force-pushed the dpeng817/in_airflow_dag_override branch from 8137f6b to 9a9f7f4 Compare October 10, 2024 16:34
@dpeng817 dpeng817 force-pushed the dpeng817/proxied_state_parsing branch from d1e7936 to 9838517 Compare October 10, 2024 16:40
@dpeng817 dpeng817 force-pushed the dpeng817/in_airflow_dag_override branch from 9a9f7f4 to 6f585b9 Compare October 10, 2024 16:40
@dpeng817 dpeng817 force-pushed the dpeng817/proxied_state_parsing branch from 9838517 to abb83f5 Compare October 10, 2024 16:45
@dpeng817 dpeng817 force-pushed the dpeng817/in_airflow_dag_override branch from 6f585b9 to 9de4d84 Compare October 10, 2024 16:45
@dpeng817 dpeng817 force-pushed the dpeng817/proxied_state_parsing branch from abb83f5 to 6de4054 Compare October 10, 2024 17:10
@dpeng817 dpeng817 force-pushed the dpeng817/in_airflow_dag_override branch from 9de4d84 to 859e43a Compare October 10, 2024 17:10
@dpeng817 dpeng817 force-pushed the dpeng817/proxied_state_parsing branch from 6de4054 to 0f32078 Compare October 11, 2024 23:07
@dpeng817 dpeng817 force-pushed the dpeng817/in_airflow_dag_override branch from 859e43a to 1d65852 Compare October 11, 2024 23:07
@dpeng817 dpeng817 deleted the dpeng817/in_airflow_dag_override branch October 14, 2024 20:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants