You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "pierrejeambrun (via GitHub)" <gi...@apache.org> on 2025/07/28 16:02:56 UTC
[PR] Fix poolbar counts [airflow]
pierrejeambrun opened a new pull request, #53825:
URL: https://github.com/apache/airflow/pull/53825
closes: https://github.com/apache/airflow/issues/52272
This just fixes the bars. Indeed as mentioned in the related issue we might want to reconsider how we build this because the total number of slots displayed can grow beyond the total pool slots. This is caused by the fact that we display in the same way state that take up a pool slots and those that do not. (QUEUE, RUNNING, OPEN) should always be included, (DEFERRED) should only be included if configured, and QUEUE / DEFERRED (non configured) should probably be displayed outside of that pool occupancy bar, to now provide confusion.
This is a bigger effort for the short I just fixed the bar, and will create an issue for the follow up.
### Before
<img width="1911" height="764" alt="Screenshot 2025-07-28 at 17 59 35" src="https://github.com/user-attachments/assets/0124a1bd-f7cc-4b1e-901b-311072762b3e" />
<img width="1909" height="647" alt="Screenshot 2025-07-28 at 17 59 42" src="https://github.com/user-attachments/assets/a90fd200-7db0-437a-9a64-69ef715a36f1" />
### After:
<img width="1881" height="556" alt="Screenshot 2025-07-28 at 17 59 08" src="https://github.com/user-attachments/assets/e875be04-b48c-48f7-8df1-86dd482fbe2c" />
<img width="1916" height="480" alt="Screenshot 2025-07-28 at 17 59 18" src="https://github.com/user-attachments/assets/59a84dcc-8d08-47ad-b770-7900dd0212b9" />
--
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 poolbar counts [airflow]
Posted by "pierrejeambrun (via GitHub)" <gi...@apache.org>.
pierrejeambrun commented on PR #53825:
URL: https://github.com/apache/airflow/pull/53825#issuecomment-3127979481
cc: @uranusjr just in case you have time to double check that this fixes it on your side.
--
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 poolbar counts [airflow]
Posted by "pierrejeambrun (via GitHub)" <gi...@apache.org>.
pierrejeambrun commented on PR #53825:
URL: https://github.com/apache/airflow/pull/53825#issuecomment-3131606607
No need to backport, `v3-0-test` branch does not have the problem.
<img width="1914" height="853" alt="Screenshot 2025-07-29 at 11 47 17" src="https://github.com/user-attachments/assets/c1d0888f-8c50-4ea5-b5c1-a749e7d11241" />
--
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 poolbar counts [airflow]
Posted by "pierrejeambrun (via GitHub)" <gi...@apache.org>.
pierrejeambrun merged PR #53825:
URL: https://github.com/apache/airflow/pull/53825
--
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 poolbar counts [airflow]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #53825:
URL: https://github.com/apache/airflow/pull/53825#issuecomment-3129225691
### Backport failed to create: v3-0-test. View the failure log <a href='https://github.com/apache/airflow/actions/runs/16578964453'> Run details </a>
<table>
<tr>
<th>Status</th>
<th>Branch</th>
<th>Result</th>
</tr>
<tr>
<td>❌</td>
<td>v3-0-test</td>
<td><a href="https://github.com/apache/airflow/commit/e83ca8e"><img src='https://img.shields.io/badge/Commit-e83ca8e-red' alt='Commit Link'></a></td>
</tr>
</table>
You can attempt to backport this manually by running:
```bash
cherry_picker e83ca8e v3-0-test
```
This should apply the commit to the v3-0-test branch and leave the commit in conflict state marking
the files that need manual conflict resolution.
After you have resolved the conflicts, you can continue the backport process by running:
```bash
cherry_picker --continue
```
--
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 poolbar counts [airflow]
Posted by "pierrejeambrun (via GitHub)" <gi...@apache.org>.
pierrejeambrun commented on PR #53825:
URL: https://github.com/apache/airflow/pull/53825#issuecomment-3127958171
Follow up issue:
https://github.com/apache/airflow/issues/53826
--
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