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 Elasticsearch example DAGs to new design #22445

Closed
1 task done
potiuk opened this issue Mar 22, 2022 · 6 comments · Fixed by #22811
Closed
1 task done

Migrate Elasticsearch example DAGs to new design #22445

potiuk opened this issue Mar 22, 2022 · 6 comments · Fixed by #22811
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 Elasticsearch provider. It is created to track progress of their migration.

List of paths to example DAGs:

  • airflow/providers/elasticsearch/example_dags/example_elasticsearch_query.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 2, 2022

@potiuk I will pick this up and try if i could fix it

@Bowrna
Copy link
Contributor

Bowrna commented Apr 14, 2022

@potiuk Could this be updated in the project too? https://github.com/apache/airflow/projects/15

@potiuk
Copy link
Member Author

potiuk commented Apr 14, 2022

I did

@potiuk
Copy link
Member Author

potiuk commented Apr 14, 2022

Ah ... automation was not set in the project - normally when issue is closed, it goes to Done automatically but it was disabled :)? Re-enabled it back

@potiuk potiuk reopened this Jun 3, 2022
@Bowrna
Copy link
Contributor

Bowrna commented Jun 3, 2022

@potiuk Do i have to fix anything more to get this issue closed?

@potiuk
Copy link
Member Author

potiuk commented Jun 3, 2022

It was automated checks - indeed it seems that the original test is not removed yet :)

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