Skip to content

AIP-84 Migrate public endpoint Clear Task Instances to FastAPI#44220

Merged
pierrejeambrun merged 5 commits intoapache:mainfrom
omkar-foss:aip-84/tis/post-clear-tis
Nov 22, 2024
Merged

AIP-84 Migrate public endpoint Clear Task Instances to FastAPI#44220
pierrejeambrun merged 5 commits intoapache:mainfrom
omkar-foss:aip-84/tis/post-clear-tis

Conversation

@omkar-foss
Copy link
Collaborator

@omkar-foss omkar-foss commented Nov 20, 2024

closes: #43751
related: #42370

This migrates the Clear Task Instances API from api_connexion to api_fastapi.

@omkar-foss omkar-foss self-assigned this Nov 20, 2024
@boring-cyborg boring-cyborg bot added area:API Airflow's REST/HTTP API area:UI Related to UI/UX. For Frontend Developers. labels Nov 20, 2024
@omkar-foss omkar-foss added legacy api Whether legacy API changes should be allowed in PR and removed area:API Airflow's REST/HTTP API area:UI Related to UI/UX. For Frontend Developers. labels Nov 20, 2024
@omkar-foss omkar-foss closed this Nov 20, 2024
@omkar-foss omkar-foss reopened this Nov 20, 2024
@omkar-foss omkar-foss marked this pull request as ready for review November 20, 2024 17:49
@omkar-foss
Copy link
Collaborator Author

omkar-foss commented Nov 20, 2024

Resolving conflicts from recent PR merges into main.

@omkar-foss omkar-foss force-pushed the aip-84/tis/post-clear-tis branch from c54811d to f670a0a Compare November 20, 2024 19:36
@omkar-foss
Copy link
Collaborator Author

omkar-foss commented Nov 20, 2024

Resolving conflicts from recent PR merges into main.

This is done, PR synced with main and all conflicts resolved ✅

@omkar-foss
Copy link
Collaborator Author

omkar-foss commented Nov 20, 2024

All checks passed, PR ready to review ✅

pierrejeambrun
pierrejeambrun reviewed Nov 21, 2024
View reviewed changes
Copy link
Member

@pierrejeambrun pierrejeambrun left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Looking good!

A few suggestions (only #44220 (comment) requires change), we can merge after :)

@omkar-foss omkar-foss force-pushed the aip-84/tis/post-clear-tis branch from 8cd608c to b50bd96 Compare November 21, 2024 11:34
@omkar-foss
Copy link
Collaborator Author

omkar-foss commented Nov 21, 2024 •
edited
Loading

Looking good!

A few suggestions (only #44220 (comment) requires change), we can merge after :)

Thanks @pierrejeambrun, I've made the changes as per your suggestions and resolved the corresponding conversations. Also rebased with main. Please have a look, thanks! 😁

pierrejeambrun
pierrejeambrun approved these changes Nov 21, 2024
View reviewed changes
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.

One extra nit, can be merged after that.

bbovenzi
bbovenzi reviewed Nov 21, 2024
View reviewed changes
@omkar-foss omkar-foss force-pushed the aip-84/tis/post-clear-tis branch from b50bd96 to 2eb2daf Compare November 21, 2024 20:04
@omkar-foss
Copy link
Collaborator Author

omkar-foss commented Nov 21, 2024

@pierrejeambrun @bbovenzi Thank you for your reviews! I've updated the PR as per your suggestions and also rebased with main. Please check it out when possible, thanks 😁

@omkar-foss omkar-foss force-pushed the aip-84/tis/post-clear-tis branch from 8a6ea5e to dd6f05a Compare November 21, 2024 22:25
@pierrejeambrun pierrejeambrun force-pushed the aip-84/tis/post-clear-tis branch from dd6f05a to 83e9891 Compare November 22, 2024 11:09
@pierrejeambrun pierrejeambrun changed the title Migrate public endpoint Clear Task Instances to FastAPI AIP-84 Migrate public endpoint Clear Task Instances to FastAPI Nov 22, 2024
@pierrejeambrun pierrejeambrun added the AIP-84 Modern Rest API label Nov 22, 2024
@pierrejeambrun pierrejeambrun added this to the Airflow 3.0.0 milestone Nov 22, 2024
@pierrejeambrun
Copy link
Member

pierrejeambrun commented Nov 22, 2024

I just rebased the branch, to solve additional conflicts introduced by recent merge.

Should be good to merge.

@omkar-foss omkar-foss force-pushed the aip-84/tis/post-clear-tis branch from 83e9891 to a6bba42 Compare November 22, 2024 13:28
@omkar-foss
Copy link
Collaborator Author

omkar-foss commented Nov 22, 2024

1 test failing after rebase, rebased again. It should pass now.

@pierrejeambrun pierrejeambrun merged commit 00fd540 into apache:main Nov 22, 2024
got686-yandex pushed a commit to got686-yandex/airflow that referenced this pull request Jan 30, 2025
@omkar-foss @got686-yandex
AIP-84 Migrate public endpoint Clear Task Instances to FastAPI (apach…
12efca0 
…e#44220)

* Migrate public endpoint Clear Task Instances to FastAPI

* Remove unused alias choice, route trailing slash

* Make changes as per feedback from Pierre

* Add total_entries to response

* Remove logical_date from clear tis response
@omkar-foss omkar-foss deleted the aip-84/tis/post-clear-tis branch April 1, 2025 06:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bbovenzi bbovenzi bbovenzi left review comments

@pierrejeambrun pierrejeambrun pierrejeambrun approved these changes

@ryanahamilton ryanahamilton Awaiting requested review from ryanahamilton

@jscheffl jscheffl Awaiting requested review from jscheffl

@ephraimbuddy ephraimbuddy Awaiting requested review from ephraimbuddy

Assignees

@omkar-foss omkar-foss

Labels

AIP-84 Modern Rest API legacy api Whether legacy API changes should be allowed in PR

Projects

None yet

Milestone

Airflow 3.0.0

Development

Successfully merging this pull request may close these issues.

AIP-84 Post Clear Task Instances

3 participants

@omkar-foss @pierrejeambrun @bbovenzi