Skip to content

squid: mds: QuiesceDbRequest: update the internal encoding of ops#57945

Merged
joscollin merged 1 commit intosquidfrom
wip-lusov-qdb-exclude-or-cancel-squid
Jun 14, 2024
Merged

squid: mds: QuiesceDbRequest: update the internal encoding of ops#57945
joscollin merged 1 commit intosquidfrom
wip-lusov-qdb-exclude-or-cancel-squid

Conversation

@leonid-s-usov
Copy link
Contributor

@leonid-s-usov leonid-s-usov commented Jun 9, 2024

Backport: Excluding the last root from a set will automatically mark it as QS_CANCELED. Hence, it makes more sense if exclude and cancel share the same op code, rather than exclude and release.

Fixes: https://tracker.ceph.com/issues/66400
Signed-off-by: Leonid Usov leonid.usov@ibm.com
Original-issue: https://tracker.ceph.com/issues/66383
Original-PR: #57912
(cherry picked from commit dad5249)

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

Excluding the last root from a set will automatically mark it as QS_CANCELED.
Hence, it makes more sense if `exclude` and `cancel` share the same op code,
rather than `exclude` and `release`.

Fixes: https://tracker.ceph.com/issues/66400
Signed-off-by: Leonid Usov <leonid.usov@ibm.com>
Fixes: https://tracker.ceph.com/issues/66383
(cherry picked from commit dad5249)
@leonid-s-usov leonid-s-usov requested a review from batrick June 9, 2024 15:30
@github-actions github-actions bot added cephfs Ceph File System tests labels Jun 9, 2024
@github-actions github-actions bot added this to the squid milestone Jun 9, 2024
@joscollin
Copy link
Member

Tested in https://tracker.ceph.com/issues/66423

@joscollin joscollin merged commit 7d4b013 into squid Jun 14, 2024
@joscollin joscollin deleted the wip-lusov-qdb-exclude-or-cancel-squid branch June 14, 2024 11:00
joscollin pushed a commit to joscollin/ceph that referenced this pull request Jun 27, 2024
* refs/pull/57945/head:
	squid: mds: QuiesceDbRequest: update the internal encoding of ops
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cephfs Ceph File System tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants