AIP-84 Migrate DagWarning public endpoint to FastAPI #42748

Activity

1. rawwar

   added

   kind:featureFeature RequestsFeature Requests

   needs-triagelabel for new issues that we didn't triage yetlabel for new issues that we didn't triage yet

   on Oct 4, 2024
2. dosubot

   added

   area:APIAirflow's REST/HTTP APIAirflow's REST/HTTP API

   on Oct 4, 2024
3. rawwar commented on Oct 9, 2024

   

   rawwar

   on Oct 9, 2024

   ContributorAuthor

   More actions

   While I was working on this issue, I noticed, legacy API 's list dag warning endpoint's Responses schema shows "import_errors" . I think, this should have been DagWarnings.

   https://airflow.apache.org/docs/apache-airflow/stable/stable-rest-api-ref.html#operation/get_dag_warnings

   @pierrejeambrun , do you think this is a bug?
4. pierrejeambrun commented on Oct 9, 2024

   

   pierrejeambrun

   on Oct 9, 2024

   Member

   More actions

   Yes this is a mistake in the legacy API. The actual response has dag_warnings:

   

   PR here:
   #42858

   👍React with 👍1Reacted by Kalyan R
5. pierrejeambrun

   assigned

   rawwar

   on Oct 9, 2024
6. pierrejeambrun

   added

   AIP-84Modern Rest APIModern Rest API

   and removed

   needs-triagelabel for new issues that we didn't triage yetlabel for new issues that we didn't triage yet

   on Oct 9, 2024
7. pierrejeambrun

   closed this as completedin #42749on Nov 4, 2024