Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

pacific: mds: report clients laggy due laggy OSDs only after checking any OSD is laggy #54120

Merged
merged 3 commits into from Nov 15, 2023
Merged

pacific: mds: report clients laggy due laggy OSDs only after checking any OSD is laggy #54120

merged 3 commits into from Nov 15, 2023

Conversation

dparmar18
Copy link
Contributor

Backport tracker: https://tracker.ceph.com/issues/63269
Parent tracker: https://tracker.ceph.com/issues/63105
Backport of #53839

Contribution Guidelines

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

@dparmar18
mds: report clients laggy due laggy OSDs only after checking any OSD …
69602db 
…is laggy

Fixes: https://tracker.ceph.com/issues/63105
Signed-off-by: Dhairya Parmar <dparmar@redhat.com>
(cherry picked from commit 8a5677f)
@dparmar18
mds: erase clients getting evicted from laggy_clients
4f2c462 
Fixes: https://tracker.ceph.com/issues/63105
Signed-off-by: Dhairya Parmar <dparmar@redhat.com>
(cherry picked from commit 754b602)
@dparmar18
qa: enhance test cases
1be4ff9 
Fixes: https://tracker.ceph.com/issues/63105
Signed-off-by: Dhairya Parmar <dparmar@redhat.com>
(cherry picked from commit 9005451)
@github-actions github-actions bot added cephfs Ceph File System tests labels Oct 20, 2023
@github-actions github-actions bot added this to the pacific milestone Oct 20, 2023
@dparmar18 dparmar18 requested a review from a team October 20, 2023 10:59
@batrick batrick modified the milestones: pacific, v16.2.15 Nov 8, 2023
rishabh-d-dave
rishabh-d-dave approved these changes Nov 15, 2023
View reviewed changes
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.

FS approved. QA run was successful - https://tracker.ceph.com/projects/cephfs/wiki/Pacific#15-Nov-2023

@yuriw yuriw merged commit b6297d5 into ceph:pacific Nov 15, 2023
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rishabh-d-dave rishabh-d-dave rishabh-d-dave approved these changes

Assignees
No one assigned
Labels
cephfs Ceph File System needs-qa tests wip-yuri7-testing
Projects
None yet
Milestone
v16.2.15
5 participants
@dparmar18 @rishabh-d-dave @vshankar @batrick @yuriw