[v3-3-test] Add the option to select bundle version parameter on dag run trigger endpoint (#61550)#69719
Draft
github-actions[bot] wants to merge 1 commit into
Draft
[v3-3-test] Add the option to select bundle version parameter on dag run trigger endpoint (#61550)#69719github-actions[bot] wants to merge 1 commit into
github-actions[bot] wants to merge 1 commit into
Conversation
…run trigger endpoint (#61550) * feat: adding the option to add bundle version parameter on dag run trigger endpoint * removing code comments and adding the new parameter to the schema * feat: adding the new parameter to schema and types files * fix: fixing format and missing field in class * fix: fixing the usage of bundle version in orm dagrun creation * feat: added migration file * feat: adding unit test for bundle version parameter dagrun endpoint * fix: fixing the unit test by removing unecessary asserts * fix: removing duplicate dag version check and raising exception accordingly * fix: adding a raise exception when passing incorrect bundle version * fix: verify exception message to raise accordingly * fix: wiring to bundle version and moved to the correct version target * fix: missed adding bundle_version field in 2 unit tests response assertion * fix: adding missing bundle version field in tests * fix: handling dag and dag's tasks correctly when passing bundle version * feat: added new airflow exception to handle no dag version has found * fix: assigning run dag to dag version when only using bundle version * fix: fixing assertion equal failing * fix: removed unecessary init files after prek hook added them * fix: adding support for all the dag-level callbacks * feat: adding versioning test for bundle version parameter * fix: rebasing branch * feat: adding the option to add bundle version parameter on dag run trigger endpoint * removing code comments and adding the new parameter to the schema * feat: adding the new parameter to schema and types files * fix: fixing format and missing field in class * fix: fixing the usage of bundle version in orm dagrun creation * feat: added migration file * feat: adding unit test for bundle version parameter dagrun endpoint * fix: fixing the unit test by removing unecessary asserts * fix: removing duplicate dag version check and raising exception accordingly * fix: adding a raise exception when passing incorrect bundle version * fix: verify exception message to raise accordingly * fix: wiring to bundle version and moved to the correct version target * fix: missed adding bundle_version field in 2 unit tests response assertion * fix: adding missing bundle version field in tests * fix: handling dag and dag's tasks correctly when passing bundle version * feat: added new airflow exception to handle no dag version has found * fix: assigning run dag to dag version when only using bundle version * fix: fixing assertion equal failing * fix: removed unecessary init files after prek hook added them * fix: adding support for all the dag-level callbacks * fix: prek didn't updated datamodels for airflowctl * fix: adding bundle version support also in asset route, fixing execution api versioning, and also getting latest version of dag with bundle version * fix: duplicate import and duplicate methods * fix: remove duplicate DagVersion lookup and unintended dag swap for non-explicit triggers * fix: reverting uv lock due to unnecessary changes * fix: fixing test to resolve ci errors * fix: restore uv.lock to upstream/main * fix: remove double code, moving to new CalVer file, raising correct exception and add more callbacks support in test * fix: instead of always using latest dag using the correct dag if bundle version is passed * fix: missing fields in schema.json * Regenerate supervisor schema snapshot after rebase onto main The schema.json was stale: it contained legacy top-level $defs (DagRun, DagRunState, DagRunType, JsonValue, and nested asset reference types) that the generator no longer emits after upstream main restructured them. bundle_version also needed to land at the correct offset in the generated output. * fix: consolidate bundle version resolution, fix double deserialization, drop bundle_version from Execution API * restored provider_dependencies.json.sha256sum * Fix bundle_version overwritten by sync_dag_to_db and stale supervisor schema snapshot * revert accidently changes to uv.lock * Move AddPartitionDateField to 2026-06-30 version to match partition_date introduction timeline * Remove dead AirflowBadRequest handler and unintended execution API version split * Derive run context from requested bundle version on trigger * Validate partition_key and allowed_run_types against the requested bundle version * Revert accidental uv.lock * Regenerate private UI OpenAPI spec * Removing unnecessary callback attributes * Remove unnecessary test due to remove bundle version from execution api --------- (cherry picked from commit 1ce3f17) Co-authored-by: Itay Adler <35665981+itayweb@users.noreply.github.com> Co-authored-by: Itay Adler <itayadler@MacBook-Pro-sl-Itay.local> Co-authored-by: Itay Adler <itayadler@Itays-Mac-mini.lan>
1 task
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
feat: adding the option to add bundle version parameter on dag run trigger endpoint
removing code comments and adding the new parameter to the schema
feat: adding the new parameter to schema and types files
fix: fixing format and missing field in class
fix: fixing the usage of bundle version in orm dagrun creation
feat: added migration file
feat: adding unit test for bundle version parameter dagrun endpoint
fix: fixing the unit test by removing unecessary asserts
fix: removing duplicate dag version check and raising exception accordingly
fix: adding a raise exception when passing incorrect bundle version
fix: verify exception message to raise accordingly
fix: wiring to bundle version and moved to the correct version target
fix: missed adding bundle_version field in 2 unit tests response assertion
fix: adding missing bundle version field in tests
fix: handling dag and dag's tasks correctly when passing bundle version
feat: added new airflow exception to handle no dag version has found
fix: assigning run dag to dag version when only using bundle version
fix: fixing assertion equal failing
fix: removed unecessary init files after prek hook added them
fix: adding support for all the dag-level callbacks
feat: adding versioning test for bundle version parameter
fix: rebasing branch
feat: adding the option to add bundle version parameter on dag run trigger endpoint
removing code comments and adding the new parameter to the schema
feat: adding the new parameter to schema and types files
fix: fixing format and missing field in class
fix: fixing the usage of bundle version in orm dagrun creation
feat: added migration file
feat: adding unit test for bundle version parameter dagrun endpoint
fix: fixing the unit test by removing unecessary asserts
fix: removing duplicate dag version check and raising exception accordingly
fix: adding a raise exception when passing incorrect bundle version
fix: verify exception message to raise accordingly
fix: wiring to bundle version and moved to the correct version target
fix: missed adding bundle_version field in 2 unit tests response assertion
fix: adding missing bundle version field in tests
fix: handling dag and dag's tasks correctly when passing bundle version
feat: added new airflow exception to handle no dag version has found
fix: assigning run dag to dag version when only using bundle version
fix: fixing assertion equal failing
fix: removed unecessary init files after prek hook added them
fix: adding support for all the dag-level callbacks
fix: prek didn't updated datamodels for airflowctl
fix: adding bundle version support also in asset route, fixing execution api versioning, and also getting latest version of dag with bundle version
fix: duplicate import and duplicate methods
fix: remove duplicate DagVersion lookup and unintended dag swap for non-explicit triggers
fix: reverting uv lock due to unnecessary changes
fix: fixing test to resolve ci errors
fix: restore uv.lock to upstream/main
fix: remove double code, moving to new CalVer file, raising correct exception and add more callbacks support in test
fix: instead of always using latest dag using the correct dag if bundle version is passed
fix: missing fields in schema.json
Regenerate supervisor schema snapshot after rebase onto main
The schema.json was stale: it contained legacy top-level $defs
(DagRun, DagRunState, DagRunType, JsonValue, and nested asset
reference types) that the generator no longer emits after upstream
main restructured them. bundle_version also needed to land at the
correct offset in the generated output.
fix: consolidate bundle version resolution, fix double deserialization, drop bundle_version from Execution API
restored provider_dependencies.json.sha256sum
Fix bundle_version overwritten by sync_dag_to_db and stale supervisor schema snapshot
revert accidently changes to uv.lock
Move AddPartitionDateField to 2026-06-30 version to match partition_date introduction timeline
Remove dead AirflowBadRequest handler and unintended execution API version split
Derive run context from requested bundle version on trigger
Validate partition_key and allowed_run_types against the requested bundle version
Revert accidental uv.lock
Regenerate private UI OpenAPI spec
Removing unnecessary callback attributes
Remove unnecessary test due to remove bundle version from execution api
(cherry picked from commit 1ce3f17)
Co-authored-by: Itay Adler 35665981+itayweb@users.noreply.github.com
Co-authored-by: Itay Adler itayadler@MacBook-Pro-sl-Itay.local
Co-authored-by: Itay Adler itayadler@Itays-Mac-mini.lan