Skip to content

Fetching dynamic target branches for execution API prek hook#63985

Merged
amoghrajesh merged 5 commits intoapache:mainfrom
amoghrajesh:execution-api-prek-hook-fix-2
Mar 21, 2026
Merged

Fetching dynamic target branches for execution API prek hook#63985
amoghrajesh merged 5 commits intoapache:mainfrom
amoghrajesh:execution-api-prek-hook-fix-2

Conversation

@amoghrajesh
Copy link
Contributor


Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

This is in ref to this comment: #63968 (comment)


  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

@boring-cyborg boring-cyborg bot added area:dev-tools backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch labels Mar 20, 2026
@amoghrajesh amoghrajesh added full tests needed We need to run full set of tests for this PR to merge and removed backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch labels Mar 20, 2026
@amoghrajesh amoghrajesh reopened this Mar 20, 2026
@amoghrajesh
Copy link
Contributor Author

Unrelated failure.

@amoghrajesh
Copy link
Contributor Author

cc: @jason810496 @vatsrahul1001 would appreciate review / approval here, imp one for 3.2 beta

@jason810496
Copy link
Member

The current dynamic resolution approach should be robust enough, thanks.

@amoghrajesh amoghrajesh reopened this Mar 21, 2026
@amoghrajesh
Copy link
Contributor Author

Now it works as I expect it to: https://github.com/apache/airflow/actions/runs/23374378365/job/68004324173?pr=63985, rebasing and merging this.

The failures are unrelated and are failing on main too

@amoghrajesh amoghrajesh merged commit fc3b015 into apache:main Mar 21, 2026
140 of 144 checks passed
fat-catTW pushed a commit to fat-catTW/airflow that referenced this pull request Mar 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-tools full tests needed We need to run full set of tests for this PR to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants