-
Notifications
You must be signed in to change notification settings - Fork 14.2k
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! |
I am working on the following
|
Added min version for Pydantic in #42694 |
Thanks for your contribution and help on AIP-84, as mentioned above, I think @bugraoz93 is already working on connections endpoints, or is planning to do so. Feel free to choose any other endpoints that looks cool to you, under |
Assigning myself to variables |
I've created new issues for sections Dag Source (1 endpoint), Dag Stats (1 endpoint), Task (2 endpoints) and Version (1 endpoint). List as below:
Please assign these to me, I'll pick these up one by one in the next few days, right after #42650. Cheers. |
@omkar-foss assigned and meta issue updated, thanks :) |
Hi @pierrejeambrun I've added issues for XCom, please assign to me 🙂 thanks |
Done, and meta task is updated. Thanks for taking those ones @michaeljs-c! |
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)
Config (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)
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)
Event Logs (2 endpoints)
Extra Link (1 endpoint)
Monitor (2 endpoints)
Import Error (2 endpoints)
Log (1 endpoint)
Plugin (1 endpoint)
Pool (5 endpoints) @pierrejeambrun
Provider (1 endpoint)
Task (2 endpoints) @omkar-foss
Task Instance (15 endpoints)
Variables (5 endpoints)
XCom (2 endpoints)
Committer
The text was updated successfully, but these errors were encountered: