Skip to content

Conversation

@kacpermuda
Copy link
Contributor

@kacpermuda kacpermuda commented May 26, 2025

This PR is only modifying system tests for OpenLineage.

Adding possibility to validate OpenLineage events during system tests against specific Airflow version expected value. There are some differences in OL events between Airflow 2 and 3, and there may be more in the future so this is useful. Also, this facilitates running OL system tests for multiple versions of Airflow.

This will be done automatically as long as the OL expected files follow the naming template. We will try to locate the most specific expected event file available for the running Airflow version, following this precedence order:

  1. A file named {dag_id}__af{major_version}_{minor_version}.json (e.g., openlineage_basic_dag__af2_10.json for Airflow 2.10.x)
  2. A file named {dag_id}__af{major_version}.json (e.g., openlineage_basic_dag__af3.json for any Airflow 3.x version)
  3. A generic file named {dag_id}.json without version suffix (e.g., openlineage_basic_dag.json for any Airflow version)

In this PR I also introduced many new DAGs that test specific, different parts of OL events.


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@kacpermuda kacpermuda requested a review from mobuchowski as a code owner May 26, 2025 16:13
@kacpermuda kacpermuda force-pushed the ol-system-tests-af-version branch 3 times, most recently from 64ad297 to 4cb30dc Compare May 29, 2025 15:46
@kacpermuda kacpermuda marked this pull request as draft May 29, 2025 15:47
@kacpermuda kacpermuda force-pushed the ol-system-tests-af-version branch 4 times, most recently from 02baacf to f4403a5 Compare May 30, 2025 06:13
@kacpermuda kacpermuda marked this pull request as ready for review May 30, 2025 06:27
@kacpermuda kacpermuda force-pushed the ol-system-tests-af-version branch from f4403a5 to 7a6cdf9 Compare May 30, 2025 06:57
@mobuchowski mobuchowski merged commit 5e8dee8 into apache:main Jun 3, 2025
69 checks passed
@kacpermuda kacpermuda deleted the ol-system-tests-af-version branch June 3, 2025 09:08
potiuk added a commit to potiuk/airflow that referenced this pull request Jun 3, 2025
potiuk added a commit that referenced this pull request Jun 3, 2025
sanederchik pushed a commit to sanederchik/airflow that referenced this pull request Jun 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants