Skip to content
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.

Sign up for GitHub

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

Jump to bottom

AIP-84 Migrate the public endpoint DAG Details to FastAPI #42453

Closed
2 tasks done
omkar-foss opened this issue Sep 24, 2024 · 1 comment · Fixed by #42631
Closed
2 tasks done

AIP-84 Migrate the public endpoint DAG Details to FastAPI #42453

omkar-foss opened this issue Sep 24, 2024 · 1 comment · Fixed by #42631
Assignees
@omkar-foss
Labels
AIP-84 Modern Rest API area:API Airflow's REST/HTTP API kind:feature Feature Requests

Comments

@omkar-foss
Copy link
Collaborator

Description

Currently the DAG Details public endpoint is at api_connexion/endpoints/dag_endpoint.py under get_dag_details. We need to migrate it to the api_fastapi/views/public/dags.py under a get_dag_details or similar.

Features and functionality of the endpoint will remain unchanged.

Use case/motivation

AIP-84 cWiki Link: https://cwiki.apache.org/confluence/display/AIRFLOW/AIP-84+UI+REST+API

Related issues

Related to: #42370

Are you willing to submit a PR?

  • Yes I am willing to submit a PR!

Code of Conduct
@omkar-foss omkar-foss added kind:feature Feature Requests needs-triage label for new issues that we didn't triage yet labels Sep 24, 2024
@dosubot dosubot bot added the area:API Airflow's REST/HTTP API label Sep 24, 2024
@omkar-foss
Copy link
Collaborator Author

I've started working on this, will raise a PR in the next few days (cc: @pierrejeambrun).

@pierrejeambrun pierrejeambrun mentioned this issue Sep 25, 2024
Open
1 task
@pierrejeambrun pierrejeambrun added the AIP-84 Modern Rest API label Sep 25, 2024
@pierrejeambrun pierrejeambrun moved this to In Progress in AIP-84 MODERN REST API Sep 25, 2024
@nathadfield nathadfield removed the needs-triage label for new issues that we didn't triage yet label Sep 25, 2024
@omkar-foss omkar-foss mentioned this issue Oct 1, 2024
Merged
@github-project-automation github-project-automation bot moved this from In Progress to Done in AIP-84 MODERN REST API Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@omkar-foss omkar-foss

Labels
AIP-84 Modern Rest API area:API Airflow's REST/HTTP API kind:feature Feature Requests
Projects
AIP-84 MODERN REST API
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

AIP-84 Migrate the public endpoint DAG Details to FastAPI omkar-foss/airflow
3 participants
@nathadfield @pierrejeambrun @omkar-foss