Skip to content

AIP-84 Get Task Instance Tries#44301

Merged
pierrejeambrun merged 7 commits intoapache:mainfrom
kandharvishnu:AIP-84_get_task_instance_tries
Nov 25, 2024
Merged

AIP-84 Get Task Instance Tries#44301
pierrejeambrun merged 7 commits intoapache:mainfrom
kandharvishnu:AIP-84_get_task_instance_tries

Conversation

@kandharvishnu
Copy link
Contributor

related to: #42370


@boring-cyborg boring-cyborg bot added area:API Airflow's REST/HTTP API area:UI Related to UI/UX. For Frontend Developers. labels Nov 23, 2024
@rawwar rawwar added the legacy api Whether legacy API changes should be allowed in PR label Nov 23, 2024
Copy link
Member

@pierrejeambrun pierrejeambrun left a comment

Choose a reason for hiding this comment

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

Nice.

Just a few suggestions.

Copy link
Member

@pierrejeambrun pierrejeambrun left a comment

Choose a reason for hiding this comment

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

Ready to merge, thanks.

@pierrejeambrun pierrejeambrun force-pushed the AIP-84_get_task_instance_tries branch from 3c86a62 to 66cf9e1 Compare November 25, 2024 16:20
@pierrejeambrun
Copy link
Member

Rebasing again, need latest main fix for the CI.

@pierrejeambrun pierrejeambrun merged commit 19e97da into apache:main Nov 25, 2024
got686-yandex pushed a commit to got686-yandex/airflow that referenced this pull request Jan 30, 2025
* adding get_task_instance_tries code

* renaming variables

* dummy change

* changing test case

* remove print statement

* checking response in test cases

* rename variables

---------

Co-authored-by: kandharvishnuu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:API Airflow's REST/HTTP API area:UI Related to UI/UX. For Frontend Developers. legacy api Whether legacy API changes should be allowed in PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants