Vamsi-klu opened a new pull request, #67818:
URL: https://github.com/apache/airflow/pull/67818

   Fixes #55734
   
   ## What changed
   
   The dashboard Pool Summary now aggregates `deferred_slots` into the usage 
bar only for pools where `include_deferred` is enabled.
   
   I also added focused regression coverage for:
   - individual pool cards, where non-consuming deferred tasks remain secondary 
information below the bar
   - dashboard aggregation, where deferred slots from non-counting pools do not 
inflate total usage
   - mixed pools, where only counting pools contribute deferred usage to the 
dashboard bar
   
   ## Why
   
   A pool with open slots and a deferred task could make the dashboard summary 
look like the deferred task consumed an additional slot even when that pool was 
configured not to count deferred tasks.
   
   ## Impact
   
   The dashboard now matches pool-slot semantics:
   - `include_deferred=false`: deferred tasks are visible as secondary 
information but do not increase displayed bar usage
   - `include_deferred=true`: deferred tasks are included in the usage bar
   
   ## How this solves the problem
   
   The dashboard aggregate now skips `deferred_slots` for pools that do not 
include deferred tasks in slot accounting. This keeps the summary from 
displaying cases like `128` open plus `1` deferred as inflated usage while 
preserving the existing individual pool-card behavior.
   
   ## Validation
   
   - `corepack [email protected] --config.verify-deps-before-run=false test PoolBar 
PoolSummary`
   - `corepack [email protected] --config.verify-deps-before-run=false lint`
   - `corepack [email protected] --config.verify-deps-before-run=false build`
   - `prek run --from-ref upstream/main --stage pre-commit`
   - `prek run --from-ref upstream/main --stage manual`
   - Rendered smoke check with a static local UI and mock API:
     - dashboard showed `255` open and `1` deferred for one non-counting 
deferred pool plus one counting deferred pool
     - `/pools` showed only the counting pool linking its deferred bar segment
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes — Codex (GPT-5)
   
   Generated-by: Codex (GPT-5) following [the 
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)


-- 
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