-
Notifications
You must be signed in to change notification settings - Fork 14.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
AIP-84 Migrate public endpoints from api_connexion to the new rest api #42370
Comments
Trimming down May I pick this up? If yes, please assign this issue to me. Thanks! |
Hello @omkar-foss,
This issue focuses on the public endpoints, those are located under the In anycase, just mention the endpoint you are working on, so people do not do the same ones, and I will update the description to track that :). I think we should do one at the time. |
Hi @pierrejeambrun, thank you for explaining it to me, much appreciate it. I'll start migrating the public endpoints, beginning with migration of the DAG Details API ( Since this is a meta issue (missed that previously!), I'll create the issue/PR corresponding to the endpoint(s) that I'll be working on and link to this issue (like usual). P.S: |
Thanks for creating the ticket @pierrejeambrun! Let me take connection endpoints |
Great, thanks @bugraoz93 |
Thanks for assigning! Let me save you from regular updates :) I created issues for all the endpoints in connections. |
Great, thanks @bugraoz93. I just updated the meta issue with those new additions. Let me know if you plan to work on all of them so I can assign you, or just some of them so I can label the others with |
I am planning to work on all of them @pierrejeambrun. if you can assign them to me, that would be great! Thanks :) |
Hey folks! After #42631 (open for review), I can pick up these two next in order:
With these 2 done, I suppose we should have the |
@omkar-foss Assigned. Thanks for picking these issues up! |
Hi @pierrejeambrun, could you please assign these TI-related endpoints to me in above issue description: GET:
POST: PUT: |
Hey omkar, as you can see in I'm already assigned to the Thanks for creating the issue, we can split the work on that matter, i'll assign some of them to you maybe all the |
You're assigned, issue updated, let me know if that shounds good for you :). |
Yes sure, that works. Thank you :) |
Body
#42366 is taking care of migrating private endpoints.
This issue focus on migrating all the public ones. Ideally they shouldn't change too much and be feature rich compare to airflow 2.x.
An exemple PR can be found here #42196
Tasks (total ~ 81 endpoints)
Backfill (6 endpoints) @dstandish
list_backfills
AIP-84 Migrate backfill API to fast api #43496 @dstandishget_backfill
AIP-84 Migrate backfill API to fast api #43496 @dstandishpause_backfill
AIP-84 Migrate backfill API to fast api #43496 @dstandishunpause_backfill
AIP-84 Migrate backfill API to fast api #43496 @dstandishcancel_backfill
AIP-84 Migrate backfill API to fast api #43496 @dstandishcreate_backfill
AIP-84 Migrate backfill API to fast api #43496 @dstandishConfig (2 endpoints) @rawwar
Connection @bugraoz93 (6 endpoints)
api_connexion/endpoints/connection_endpoint.py
delete_connection
AIP-84 Migrate the public endpoint Delete Connection to FastAPI #42559 @bugraoz93api_connexion/endpoints/connection_endpoint.py
get_connection
AIP-84 Migrate the public endpoint Get Connection to FastAPI #42590 @rawwarapi_connexion/endpoints/connection_endpoint.py
get_connections
AIP-84 Migrate the public endpoint Get Connections to FastAPI #42591 @bugraoz93api_connexion/endpoints/connection_endpoint.py
patch_connection
AIP-84 Migrate the public endpoint Patch Connection to FastAPI #42592 @bugraoz93api_connexion/endpoints/connection_endpoint.py
post_connection
AIP-84 Migrate the public endpoint Post Connection to FastAPI #42593 @bugraoz93api_connexion/endpoints/connection_endpoint.py
test_connection
AIP-84 Migrate the public endpoint Test Connection to FastAPI #42594 @bugraoz93DAG (5 endpoints)
api_connexion/endpoints/dag_endpoint.py
get_dag_details
AIP-84 Migrate the public endpoint DAG Details to FastAPI #42453 @omkar-fossapi_connexion/endpoints/dag_endpoint.py
patch_dag
AIP-84 Migrate the public endpoint Update DAG to FastAPI #42468 @pierrejeambrunapi_connexion/endpoints/dag_endpoint.py
patch_dags
AIP-84 Migrate the public endpoint Batch Update DAG to FastAPI #42544 @pierrejeambrunapi_connexion/endpoints/dag_endpoint.py
get_dag
AIP-84 Migrate the public endpoint Get DAG to FastAPI #42652 @omkar-fossapi_connexion/endpoints/dag_endpoint.py
delete_dag
AIP-84 Migrate the public endpoint Delete DAG to FastAPI #42650 @omkar-fossDag Parsing (1 endpoint) @prabhusneha
Dag Runs @rawwar (9 endpoints)
api_connexion/endpoints/dag_run_endpoint.py
AIP-84 Migrate Dag Run related public endpoint to FastAPI #42701 @rawwarDag Source (1 endpoint)
Dag Stats (1 endpoint)
Dag Warnings (1 endpoint) @rawwar
Dataset / Assets (10 endpoints) @amoghrajesh
get_assets
AIP-84 Migrating GET ASSETS Legacy API to fastAPI #43783 @amoghrajeshget_asset
AIP 84: Migrate GET one ASSET legacy API to fast API #43825 @amoghrajeshget_upstream_asset_events
AIP-84: Migrating GET Dataset events for DAG runs api to fastAPI #43874 @amoghrajeshget_asset_events
AIP 84: Migrate GET ASSET EVENTS legacy API to fast API #43881 @vatsrahul1001create_asset_event
: AIP 84: Migrate POST ASSET EVENT legacy API to fast API. #43984 @vatsrahul1001get_asset_queued_events
: AIP-84: Migrating GET queued asset events for assets to fastAPI #44048 @vatsrahul1001delete_asset_queued_events
AIP-84: Migrating DELETE queued asset events for assets to fastAPI #44052 @vatsrahul1001get_dag_asset_queued_events
AIP-84: Migrating GET queued asset events for DAG to fastAPI #44124 @amoghrajeshdelete_dag_asset_queued_events
AIP-84: Migrating DELETE queued asset events for DAG to fastAPI #44129 @amoghrajeshdelete_dag_asset_queued_event
AIP-84: Migrating DELETE a queued asset events for DAG to fastAPI #44130 @amoghrajeshget_dag_asset_queued_event
AIP-84: Migrating GET one queued asset events for DAG to fastAPI #44128 @amoghrajeshEvent Logs (2 endpoints)
Extra Link (1 endpoint) @prabhusneha
Monitor (2 endpoints)
Import Error (2 endpoints) @jason810496
AIP-84 Migrate public endpoint to list import errors to FastAPI #43595 @jason810496
Log (1 endpoint) @prabhusneha
Plugin (1 endpoint)
Pool (5 endpoints) @pierrejeambrun
Provider (1 endpoint)
Task (2 endpoints) @omkar-foss
Task Instance (15 endpoints) @pierrejeambrun
get_task_instance_try_details
AIP-84 Get Task Instance Try Details #43675 @kandharvishnuget_mapped_task_instance_try_details
@kandharvishnuget_task_instance_tries
@kandharvishnuget_mapped_task_instance_tries
@kandharvishnuVariables (5 endpoints)
XCom (2 endpoints)
Committer
The text was updated successfully, but these errors were encountered: