Skip to content

DAOS-17946 doc: update space_rb default in comment#17644

Open
daltonbohning wants to merge 1 commit intomasterfrom
dbohning/daos-17946-doc
Open

DAOS-17946 doc: update space_rb default in comment#17644
daltonbohning wants to merge 1 commit intomasterfrom
dbohning/daos-17946-doc

Conversation

@daltonbohning
Copy link
Contributor

Per #17032, default space_rb is now 5%.

Doc-only: 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).

Per #17032, default space_rb is now 5%.

Doc-only: true

Signed-off-by: Dalton Bohning <dalton.bohning@hpe.com>
@daltonbohning daltonbohning self-assigned this Mar 4, 2026
@daltonbohning daltonbohning marked this pull request as ready for review March 4, 2026 14:35
@daltonbohning daltonbohning requested review from a team as code owners March 4, 2026 14:35
/**
* Reserve space ratio: amount of space to be reserved on each target
* for rebuild purpose. default = 0%.
* for rebuild purpose. default = 5%.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I wonder if we should actually just remove the default in this comment so it doesn't get out of sync. And most other props in this file don't mention default values either.

@github-actions
Copy link

github-actions bot commented Mar 4, 2026

Ticket title is 'Document better Defaults for DAOS (RF3, cell size, etc)'
Status is 'Resolved'
Labels: '2.8pp,scrubbed_2.8'
https://daosio.atlassian.net/browse/DAOS-17946

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant