Skip to content

Conversation

@phender
Copy link
Contributor

@phender phender commented Feb 6, 2026

Remove system_ram_reserved setting now the test is running on HW.

Skip-unit-tests: true
Skip-fault-injection-test: true

Steps for the author:

  • Commit message follows the guidelines.
  • Appropriate Features or Test-tag pragmas were used.
  • Appropriate Functional Test Stages were run.
  • At least two positive code reviews including at least one code owner from each category referenced in the PR.
  • Testing is complete. If necessary, forced-landing label added and a reason added in a comment.

After all prior steps are complete:

  • Gatekeeper requested (daos-gatekeeper added as a reviewer).

Remove system_ram_reserved setting now the test is running on HW.

Skip-unit-tests: true
Skip-fault-injection-test: true

Signed-off-by: Phil Henderson <phillip.henderson@hpe.com>
@phender phender marked this pull request as ready for review February 6, 2026 17:38
@phender phender requested review from a team as code owners February 6, 2026 17:38
@github-actions
Copy link

github-actions bot commented Feb 6, 2026

Ticket title is 'rebuild/basic.py:RbldBasic.test_rebuild_basic - Available memory (RAM) insufficient for configured 243 GiB ram-disk size'
Status is 'In Review'
https://daosio.atlassian.net/browse/DAOS-18572

Copy link
Contributor

@daltonbohning daltonbohning left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Should this test also be using storage: auto now?

@phender phender requested a review from a team February 12, 2026 14:30
@phender phender added the forced-landing The PR has known failures or has intentionally reduced testing, but should still be landed. label Feb 12, 2026
@phender phender merged commit f010597 into master Feb 12, 2026
33 checks passed
@phender phender deleted the hendersp/DAOS-18572 branch February 12, 2026 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

forced-landing The PR has known failures or has intentionally reduced testing, but should still be landed.

Development

Successfully merging this pull request may close these issues.

3 participants