OpenLineage AirflowRunFacet.json wrong type for tags

[MaartenHubrechts]

Apache Airflow Provider(s)

openlineage

Versions of Apache Airflow Providers

apache-airflow-providers-openlineage 1.12.2

Apache Airflow version

2.9.3

Operating System

Linux

Deployment

Official Apache Airflow Helm Chart

Deployment details

No response

What happened

The AirflowRunFacet.json currently isn't usable to validate OpenLineage logs that contain the AirflowRunFacet:
https://github.com/apache/airflow/blob/main/providers/src/airflow/providers/openlineage/facets/AirflowRunFacet.json#L175
This is due to the type of the tags element (on lines 175-177):

"tags": {
  "type": "string"
}

The logs that get produced by the openlineage provider contain lists of strings as tags. When one tag is given to a DAG this is still a list that contains 1 string, when multiple tags are given to a DAG the list contains all these strings.

What you think should happen instead

The tags field in AirflowRunFacet.json should be changed as follows:

"tags": {
  "type": "array",
  "items": {
    "type": "string"
  }
}

Which validates that it is indeed an array of strings.

How to reproduce

Validate any given openlineage log that contains the airflow run facet with the current AirflowRunFacet.json and the validation will fail.

Anything else

No response

Are you willing to submit PR?

• Yes I am willing to submit a PR!

Code of Conduct

• I agree to follow this project's Code of Conduct

[boring-cyborg]

Thanks for opening your first issue here! Be sure to follow the issue template! If you are willing to raise PR to address this issue please do so, no need to wait for approval.

[potiuk]

cc: @mobuchowski @kacpermuda

[mobuchowski]

@MaartenHubrechts We've moved to that before and unfortunately had to revert because it broke users workflow: #41786

However, we can add this as a separate list with that format. The information will be duplicated, but we'd be able to drop it... in Airflow 3 probably.

[MaartenHubrechts]

@mobuchowski Thanks for the reply! It would be nice to have it as a separate list, if possible.

[kacpermuda]

Done in #51303