You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "bobu-putheeckal (via GitHub)" <gi...@apache.org> on 2026/05/24 14:11:42 UTC
[PR] Fix deferred slots in pool summary chart [airflow]
bobu-putheeckal opened a new pull request, #67414:
URL: https://github.com/apache/airflow/pull/67414
## Summary
- exclude deferred slots from the dashboard pool summary chart when a pool is configured with `include_deferred: false`
- keep deferred slots visible in the aggregate chart when they do consume pool slots
- add focused PoolSummary regression coverage
Fixes #55734
## Tests
- `npm exec --yes pnpm@10.25.0 -- install --frozen-lockfile`
- `npm exec --yes pnpm@10.25.0 -- test src/pages/Dashboard/PoolSummary/PoolSummary.test.tsx`
- `npm exec --yes pnpm@10.25.0 -- exec eslint src/pages/Dashboard/PoolSummary/PoolSummary.tsx src/pages/Dashboard/PoolSummary/PoolSummary.test.tsx`
- `npm exec --yes pnpm@10.25.0 -- exec tsc -p tsconfig.app.json --noEmit`
## Screenshots
Not included. This is a dashboard aggregation fix covered by a component test; I did not run a full Airflow backend with representative deferred task data locally.
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Fix deferred slots in pool summary chart [airflow]
Posted by "bobu-putheeckal (via GitHub)" <gi...@apache.org>.
bobu-putheeckal commented on PR #67414:
URL: https://github.com/apache/airflow/pull/67414#issuecomment-4529800331
Additional detail for review: the visual effect is limited to the Dashboard Pool Summary aggregate bar. For pools with `include_deferred=false`, `deferred_slots` are excluded from the aggregate total because those deferred tasks do not consume pool slots; pools where deferred tasks do consume slots still include them. The component test covers both cases.
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Fix deferred slots in pool summary chart [airflow]
Posted by "kaxil (via GitHub)" <gi...@apache.org>.
kaxil commented on PR #67414:
URL: https://github.com/apache/airflow/pull/67414#issuecomment-4529871861
PR template hasn't been followed on any of your PRs. As such I am closing all your PRs since I see little to no evidence of actual testing and mass PRs to Airflow & other repos.
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Fix deferred slots in pool summary chart [airflow]
Posted by "kaxil (via GitHub)" <gi...@apache.org>.
kaxil closed pull request #67414: Fix deferred slots in pool summary chart
URL: https://github.com/apache/airflow/pull/67414
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org