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

Update dags reserialize command to work with DAG bundle #45507

Merged
merged 6 commits into from
Jan 14, 2025

Conversation

ephraimbuddy
Copy link
Contributor

This PR also changed the --subdir arg to --bundle-name.

Closes: #45336

@ephraimbuddy ephraimbuddy force-pushed the update-dag-reserialize branch from a0885c7 to d1d318b Compare January 9, 2025 17:41
airflow/cli/cli_config.py Outdated Show resolved Hide resolved
airflow/cli/commands/remote_commands/dag_command.py Outdated Show resolved Hide resolved
airflow/cli/commands/remote_commands/dag_command.py Outdated Show resolved Hide resolved
airflow/cli/commands/remote_commands/dag_command.py Outdated Show resolved Hide resolved
airflow/cli/commands/remote_commands/dag_command.py Outdated Show resolved Hide resolved
airflow/cli/commands/remote_commands/dag_command.py Outdated Show resolved Hide resolved
tests/cli/commands/remote_commands/test_dag_command.py Outdated Show resolved Hide resolved
@ephraimbuddy ephraimbuddy self-assigned this Jan 13, 2025
@ephraimbuddy ephraimbuddy force-pushed the update-dag-reserialize branch from d1d318b to 1b5f556 Compare January 13, 2025 14:01
@ephraimbuddy ephraimbuddy force-pushed the update-dag-reserialize branch from 72fc76e to 45aa606 Compare January 14, 2025 07:59
@ephraimbuddy ephraimbuddy merged commit f007929 into apache:main Jan 14, 2025
45 checks passed
@ephraimbuddy ephraimbuddy deleted the update-dag-reserialize branch January 14, 2025 12:28
HariGS-DB pushed a commit to HariGS-DB/airflow that referenced this pull request Jan 16, 2025
* Update `dags reserialize` command to work with DAG bundle

This PR also changed the `--subdir` arg to --bundle-name.

* fixup! Update `dags reserialize` command to work with DAG bundle

* Apply suggestions from code review

* Apply suggestions from code review

Co-authored-by: Jed Cunningham <[email protected]>

* Update tests/cli/commands/remote_commands/test_dag_command.py

Co-authored-by: Jed Cunningham <[email protected]>

* fixup! Update tests/cli/commands/remote_commands/test_dag_command.py

---------

Co-authored-by: Jed Cunningham <[email protected]>
dauinh pushed a commit to dauinh/airflow that referenced this pull request Jan 24, 2025
* Update `dags reserialize` command to work with DAG bundle

This PR also changed the `--subdir` arg to --bundle-name.

* fixup! Update `dags reserialize` command to work with DAG bundle

* Apply suggestions from code review

* Apply suggestions from code review

Co-authored-by: Jed Cunningham <[email protected]>

* Update tests/cli/commands/remote_commands/test_dag_command.py

Co-authored-by: Jed Cunningham <[email protected]>

* fixup! Update tests/cli/commands/remote_commands/test_dag_command.py

---------

Co-authored-by: Jed Cunningham <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

Make dags reserialize bundle aware
2 participants