Skip to content

Count backfill memory in the memory gate of the new runner - #2499

Open
sawenzel wants to merge 1 commit into
AliceO2Group:masterfrom
sawenzel:runner-backfill-mem-gate
Open

sawenzel wants to merge 1 commit into
AliceO2Group:masterfrom
sawenzel:runner-backfill-mem-gate

Conversation

@sawenzel

@sawenzel sawenzel commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

This fixes a problem in the memory admission of the new workflow runner and adds unit tests.

  • The default tier ignored memory booked by backfill tasks, so --mem-limit could be exceeded.
  • fits_default and mem_free_default now include the backfill memory.
  • The backfill memory factor now defaults to 1.0 and can be set with --backfill-mem-factor.
  • The simulator default is aligned with the runner.

This fixes a problem in the memory admission of the new workflow runner and adds unit tests.

- The default tier ignored memory booked by backfill tasks, so `--mem-limit` could be exceeded.
- `fits_default` and `mem_free_default` now include the backfill memory.
- The backfill memory factor now defaults to 1.0 and can be set with `--backfill-mem-factor`.
- The simulator default is aligned with the runner.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@sawenzel
sawenzel requested a review from jackal1-66 as a code owner October 8, 2026 12:11
@github-actions

github-actions Bot commented Oct 8, 2026

Copy link
Copy Markdown

REQUEST FOR PRODUCTION RELEASES:
To request your PR to be included in production software, please add the corresponding labels called "async-" to your PR. Add the labels directly (if you have the permissions) or add a comment of the form (note that labels are separated by a ",")

+async-label <label1>, <label2>, !<label3> ...

This will add <label1> and <label2> and removes <label3>.

The following labels are available
async-2023-pbpb-apass4
async-2023-pp-apass4
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0
async-2024-PbPb-apass1
async-2024-ppRef-apass1
async-2024-PbPb-apass2
async-2023-PbPb-apass5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant