kaxil opened a new pull request, #73009:
URL: https://github.com/apache/airflow/pull/73009

   Stacked on #72909, which adds the Dag Bundles list. This adds the page 
behind each row.
   
   `GET /api/v2/dagBundles/{bundle_name}` and `GET 
/api/v2/dagBundles/{bundle_name}/files`, plus a detail page showing a bundle's 
files with the Dags each one defines, when it was last parsed, how long that 
took, and its import-error count.
   
   The list answers "has my commit landed". It does not answer "what in it is 
broken", and there is one case where nothing else in the UI answers it either: 
a file that fails before defining a Dag. There is no `DagModel` row for it, so 
no per-Dag view mentions it, and the bundle's import-error count says only that 
something somewhere is wrong.
   
   So the file list has two sources. Files holding a Dag the caller may read 
are aggregated from `dag`, which is also the only record of parse time and 
duration Airflow keeps. Files that recorded an import error without registering 
a Dag are added separately, on the same admin-by-default terms as `GET 
/importErrors`, since they have no Dag to authorize on. Both routes 404 rather 
than 403 for a bundle that exists but holds no readable Dag.
   
   Screenshots and the fuller rationale are being added to this description.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to