Skip to content

YAML Dag file parser - #74084

Merged
uranusjr merged 39 commits into
apache:mainfrom
astronomer:yaml-importer
Oct 12, 2026
Merged

uranusjr merged 39 commits into
apache:mainfrom
astronomer:yaml-importer

Conversation

@uranusjr

@uranusjr uranusjr commented Oct 2, 2026

Copy link
Copy Markdown
Member

Ground work for the AIP-85 YAML Dag importer.

This establishes a schema implemented in Pydantic models, which are also used to generate a JSON Schema file to describe the format. The parser can parse the basic structure, but not perform argument validation.

This establishes a schema implemented in Pydantic models, which are also
used to generate a JSON Schema file to describe the format. The parser
can parse the basic structure, but not perform argument validation.
@uranusjr

uranusjr commented Oct 5, 2026 •

Copy link
Copy Markdown
Member Author

I need to replace the compatibility_date field with $schema to match discussion in devlist, but overall the parser should be ready for review.

Edit: Done and pushed. Marking PR as ready.

@uranusjr
uranusjr marked this pull request as ready for review October 5, 2026 08:21
@uranusjr uranusjr added this to the Airflow 3.4.0 milestone Oct 5, 2026

@pierrejeambrun pierrejeambrun left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall LGTM and the direction.

A couple of minor questions/suggestions

Looking forward to a consensus on the mailing list before approval.

@jason810496 jason810496 left a comment •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. Not-urgent, just an early comment for the follow-up in my mind.

@uranusjr
uranusjr requested review from ashb and kaxil October 11, 2026 10:25
@uranusjr
uranusjr force-pushed the yaml-importer branch 2 times, most recently from 1a8b86d to 9a28ddd Compare October 11, 2026 10:34
@uranusjr
uranusjr merged commit 3376425 into apache:main Oct 12, 2026
127 checks passed
@uranusjr
uranusjr deleted the yaml-importer branch October 12, 2026 10:54
@github-project-automation github-project-automation Bot moved this from In Progress to Done in AIP-85 Dag Importer Oct 12, 2026
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.

6 participants