Skip to content

squid: mds/cache: don't assume non-auth xlocks to be remote locks#57202

Merged
batrick merged 1 commit intosquidfrom
wip-lusov-bp-squid-xlock
May 22, 2024
Merged

squid: mds/cache: don't assume non-auth xlocks to be remote locks#57202
batrick merged 1 commit intosquidfrom
wip-lusov-bp-squid-xlock

Conversation

@leonid-s-usov
Copy link
Contributor

Fixes-Backport: https://tracker.ceph.com/issues/65710
Original-PR: #57020
Original-Issue: https://tracker.ceph.com/issues/65606

Checklist

  • Tracker (select at least one)
    • References tracker ticket
  • Component impact
    • No impact that needs to be tracked
  • Documentation (select at least one)
    • No doc update is appropriate
  • Tests (select at least one)
    • No tests
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
  • jenkins test rook e2e

@github-actions github-actions bot added the cephfs Ceph File System label May 1, 2024
@leonid-s-usov leonid-s-usov requested review from a team and batrick May 1, 2024 13:25
A few places in the code assumed that non-auth xlocks
must be remote, which prevented a proper drop lock procedure
when those locks turned out to be locallocks.

Fixes: https://tracker.ceph.com/issues/65710
Original-Issue: https://tracker.ceph.com/issues/65606
Original-PR: #57020
Signed-off-by: Leonid Usov <leonid.usov@ibm.com>
(cherry picked from commit 3aa055d)
@leonid-s-usov leonid-s-usov force-pushed the wip-lusov-bp-squid-xlock branch from 780328f to 1f0641f Compare May 1, 2024 13:28
@leonid-s-usov
Copy link
Contributor Author

jenkins test api

@batrick batrick added this to the v19.1.0 milestone May 17, 2024
@batrick
Copy link
Member

batrick commented May 17, 2024

This PR is under test in https://tracker.ceph.com/issues/66101.

@batrick batrick modified the milestones: v19.1.0, v19.1.1 May 17, 2024
Copy link
Member

@batrick batrick left a comment

Choose a reason for hiding this comment

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

@batrick batrick merged commit bdb5716 into squid May 22, 2024
@batrick batrick deleted the wip-lusov-bp-squid-xlock branch May 22, 2024 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants