Skip to content

mds/purgequeue: add l_pq_executed_ops counter#54390

Merged
rishabh-d-dave merged 1 commit intoceph:mainfrom
shminjs:feat-purgequeue-executed-ops
Jan 16, 2024
Merged

mds/purgequeue: add l_pq_executed_ops counter#54390
rishabh-d-dave merged 1 commit intoceph:mainfrom
shminjs:feat-purgequeue-executed-ops

Conversation

@shminjs
Copy link

@shminjs shminjs commented Nov 7, 2023

We need to know cluster deleting pressure better. And only pq_executed
is not enough, because file size is different.

Fixes: https://tracker.ceph.com/issues/63468
Signed-off-by: shimin shimin@kuaishou.com

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
  • jenkins retest this please
  • jenkins test classic perf
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test dashboard cephadm
  • jenkins test api
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox
  • jenkins test windows

@github-actions github-actions bot added the cephfs Ceph File System label Nov 7, 2023
We need to know cluster deleting pressure better. And only pq_executed
 is not enough, because file size is different.

Fixes: https://tracker.ceph.com/issues/63468
Signed-off-by: shimin <shimin@kuaishou.com>
@shminjs shminjs force-pushed the feat-purgequeue-executed-ops branch from 12459b4 to bdae273 Compare November 7, 2023 13:18
@shminjs
Copy link
Author

shminjs commented Nov 8, 2023

jenkins retest this please

@shminjs
Copy link
Author

shminjs commented Nov 8, 2023

jenkins test make check arm64

@vshankar vshankar requested a review from a team November 8, 2023 14:15
Copy link
Contributor

@kotreshhr kotreshhr left a comment

Choose a reason for hiding this comment

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

lgtm

@vshankar
Copy link
Contributor

@mchangir please pick this up for QA run.

Copy link
Contributor

@rishabh-d-dave rishabh-d-dave left a comment

Choose a reason for hiding this comment

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

@rishabh-d-dave rishabh-d-dave merged commit 2c5acea into ceph:main Jan 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cephfs Ceph File System wip-rishabh-testing Rishabh's testing label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants