Skip to content

qa/rados: Fix problem with recompression failing osd bench testing#63429

Merged
ljflores merged 1 commit intoceph:mainfrom
aclamk:aclamk-qa-no-bench-for-write-v2-compression
May 30, 2025
Merged

qa/rados: Fix problem with recompression failing osd bench testing#63429
ljflores merged 1 commit intoceph:mainfrom
aclamk:aclamk-qa-no-bench-for-write-v2-compression

Conversation

@aclamk
Copy link
Contributor

@aclamk aclamk commented May 22, 2025

The change refactors structure of objectstore_debug/bluestore.

  1. compr$ dir is not longer independant dimension.
  2. compr$ dir is multiplied into write$/(v1+v2+random)
  3. write$/v1 gets original compr settings
  4. write$/(v2+random) get modified compr with
    osd_mclock_skip_benchmark=true

The rationale is to disable bench when both v2 and compression are enabled.

Fixes: https://tracker.ceph.com/issues/71168
Note: Not really fixes, it just disables detecting symptoms.
Real solution needs more sophisticated approach.

Tracking tracker: https://tracker.ceph.com/issues/71419

Contribution Guidelines

  • To sign and title your commits, please refer to Submitting Patches to Ceph.

  • If you are submitting a fix for a stable branch (e.g. "quincy"), please refer to Submitting Patches to Ceph - Backports for the proper workflow.

  • When filling out the below checklist, you may click boxes directly in the GitHub web UI. When entering or editing the entire PR message in the GitHub web UI editor, you may also select a checklist item by adding an x between the brackets: [x]. Spaces and capitalization matter when checking off items this way.

Checklist

  • Tracker (select at least one)
    • References tracker ticket
    • Very recent bug; references commit where it was introduced
    • New feature (ticket optional)
    • Doc update (no ticket needed)
    • Code cleanup (no ticket needed)
  • Component impact
    • Affects Dashboard, opened tracker ticket
    • Affects Orchestrator, opened tracker ticket
    • No impact that needs to be tracked
  • Documentation (select at least one)
    • Updates relevant documentation
    • No doc update is appropriate
  • Tests (select at least one)
Show available Jenkins commands

@github-actions github-actions bot added the tests label May 22, 2025
@aclamk aclamk requested a review from sseshasa May 22, 2025 11:09
The change refactors structure of objectstore_debug/bluestore.
1) compr$ dir is not longer independant dimension.
2) compr$ dir is multiplied into write$/(v1+v2+random)
3) write$/v1 gets original compr settings
4) write$/(v2+random) get modified compr with
   osd_mclock_skip_benchmark=true

The rationale is to disable bench when both v2 and compression are enabled.

Fixes: https://tracker.ceph.com/issues/71168
Note: Not really fixes, it just disables detecting symptoms.
      Real solution needs more sophisticated approach.

Signed-off-by: Adam Kupczyk <akupczyk@ibm.com>
Copy link
Contributor

@sseshasa sseshasa left a comment

Choose a reason for hiding this comment

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

LGTM.

@aclamk aclamk force-pushed the aclamk-qa-no-bench-for-write-v2-compression branch from faad88f to c3c9e5e Compare May 22, 2025 11:33
@aclamk aclamk changed the title qa/rados: Fix problem with recompression failing osd bench tesing qa/rados: Fix problem with recompression failing osd bench testing May 27, 2025
@ljflores
Copy link
Member

I scheduled some teuthology tests here: https://pulpito.ceph.com/lflores-2025-05-27_20:19:07-rados-main-distro-default-gibba/

I used this command to include all tests from the rados suite with some reference to compression:

./teuthology/virtualenv/bin/teuthology-suite -v -m gibba -c main -S d0dcdce06e17703cb2bcd1e6afcbad7e0830cde4 --suite-branch aclamk-qa-no-bench-for-write-v2-compression --suite-repo https://github.com/aclamk/ceph.git --subset 111/120000 -s rados --filter-all "compr$"

@ljflores
Copy link
Member

I scheduled some teuthology tests here: https://pulpito.ceph.com/lflores-2025-05-27_20:19:07-rados-main-distro-default-gibba/

I used this command to include all tests from the rados suite with some reference to compression:

./teuthology/virtualenv/bin/teuthology-suite -v -m gibba -c main -S d0dcdce06e17703cb2bcd1e6afcbad7e0830cde4 --suite-branch aclamk-qa-no-bench-for-write-v2-compression --suite-repo https://github.com/aclamk/ceph.git --subset 111/120000 -s rados --filter-all "compr$"

Rados-approved. Failures, unrelated, tracked by:

@ljflores ljflores merged commit bb1109a into ceph:main May 30, 2025
22 of 24 checks passed
@aclamk
Copy link
Contributor Author

aclamk commented Jun 3, 2025

Also verified by https://tracker.ceph.com/issues/71503.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants