Skip to content

DAOS-18520 test: test_aggregation_throttling - AssertionError: False is not true#17748

Open
dinghwah wants to merge 2 commits intomasterfrom
dinghwah/DAOS-18520-attest
Open

DAOS-18520 test: test_aggregation_throttling - AssertionError: False is not true#17748
dinghwah wants to merge 2 commits intomasterfrom
dinghwah/DAOS-18520-attest

Conversation

@dinghwah
Copy link
Contributor

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

Test-tag: test_snapshot_aggregation
Skip-unit-tests: true
Test-repeat: 10

Signed-off-by: Ding-Hwa Ho <ding-hwa.ho@hpe.com>
…is not true

Description: Update expected_perf_diff from 30 to 35

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

Signed-off-by: Ding-Hwa Ho <ding-hwa.ho@hpe.com>
@dinghwah dinghwah requested review from a team as code owners March 20, 2026 21:27
@github-actions
Copy link

Ticket title is 'aggregation/throttling.py:DaosAggregationThrottling.test_aggregation_throttling - AssertionError: False is not true'
Status is 'In Progress'
Labels: 'ci_master_weekly,weekly_test'
https://daosio.atlassian.net/browse/DAOS-18520

Comment on lines 75 to +76
# keep it at 30 %
expected_perf_diff = 30.0
expected_perf_diff = 35.0
Copy link
Contributor

Choose a reason for hiding this comment

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

The comment does not align with the number now. I would suggest just removing the comment since it is no longer relevant anyway.

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.

2 participants