Skip to content

DAOS-18995 control: Incorrect value printed for percentage tier-ratio#18333

Open
tanabarr wants to merge 1 commit into
masterfrom
tanabarr/control-fix-tierratio-print
Open

DAOS-18995 control: Incorrect value printed for percentage tier-ratio#18333
tanabarr wants to merge 1 commit into
masterfrom
tanabarr/control-fix-tierratio-print

Conversation

@tanabarr
Copy link
Copy Markdown
Contributor

@tanabarr tanabarr commented May 22, 2026

Correct percentage value printed.

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).

Correct percentage value printed.

Signed-off-by: Tom Nabarro <thomas.nabarro@hpe.com>
@tanabarr tanabarr requested review from a team as code owners May 22, 2026 16:58
@tanabarr tanabarr self-assigned this May 22, 2026
@github-actions
Copy link
Copy Markdown

Ticket title is 'Incorrect value printed for percentage tier-ratio'
Status is 'In Review'
https://daosio.atlassian.net/browse/DAOS-18995

Copy link
Copy Markdown
Contributor

@kjacque kjacque left a comment

Choose a reason for hiding this comment

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

Good catch. Also nice pre-emptive improvements to the terminology.

@daosbuild3
Copy link
Copy Markdown
Collaborator

@daosbuild3
Copy link
Copy Markdown
Collaborator

Test stage Functional Hardware Medium MD on SSD completed with status UNSTABLE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos//view/change-requests/job/PR-18333/4/testReport/

@tanabarr tanabarr requested a review from knard38 May 25, 2026 09:08
@tanabarr tanabarr added the forced-landing The PR has known failures or has intentionally reduced testing, but should still be landed. label May 25, 2026
@tanabarr tanabarr requested a review from a team May 25, 2026 09:12
@tanabarr
Copy link
Copy Markdown
Contributor Author

known CI failure in test_dfuse_daos_build_wb, all others passed

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.

4 participants