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

Migrate Docker example DAGs to new design #22444

Closed
3 tasks done
potiuk opened this issue Mar 22, 2022 · 4 comments · Fixed by #23167
Closed
3 tasks done

Migrate Docker example DAGs to new design #22444

potiuk opened this issue Mar 22, 2022 · 4 comments · Fixed by #23167
Assignees
Labels
AIP-47 AIP-47 New Design of System Tests kind:meta High-level information important to the community

Comments

@potiuk
Copy link
Member

potiuk commented Mar 22, 2022

There is a new design of system tests that was introduced by the AIP-47.

All current example dags need to be migrated and converted into system tests, so they can be run in the CI process automatically before releases.

This is an aggregated issue for all example DAGs related to Docker provider. It is created to track progress of their migration.

List of paths to example DAGs:

  • airflow/providers/docker/example_dags/example_docker_swarm.py
  • airflow/providers/docker/example_dags/example_docker.py
  • airflow/providers/docker/example_dags/example_docker_copy_data.py
@potiuk potiuk added the AIP-47 AIP-47 New Design of System Tests label Mar 22, 2022
@potiuk potiuk added the kind:meta High-level information important to the community label Mar 22, 2022
@Bowrna
Copy link
Contributor

Bowrna commented Apr 14, 2022

@potiuk Could I try this task?

@potiuk
Copy link
Member Author

potiuk commented Apr 14, 2022

Sure :)

@Bowrna
Copy link
Contributor

Bowrna commented Apr 22, 2022

@potiuk There is an example tutorial in this example_dags folder for docker airflow/providers/docker/example_dags/tutorial_taskflow_api_etl_docker_virtualenv.py
Do I have to migrate this example too?

@potiuk
Copy link
Member Author

potiuk commented Apr 22, 2022

yeah.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AIP-47 AIP-47 New Design of System Tests kind:meta High-level information important to the community
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants