]>
git.apps.os.sepia.ceph.com Git - ceph.git/log
avanthakkar [Fri, 6 Oct 2023 13:40:48 +0000 (19:10 +0530)]
mgr/dashboard: add path validation for pseudo path of nfs export
Fixes: https://tracker.ceph.com/issues/63124
Signed-off-by: avanthakkar <avanjohn@gmail.com>
(cherry picked from commit
adeea099a57d34a63b483899a40c22b7bfb8c999 )
Yuri Weinstein [Wed, 22 May 2024 14:39:42 +0000 (07:39 -0700)]
Merge pull request #56478 from sseshasa/wip-65150-squid
squid: common/LogEntry: Add log level to str helper for fmt::formatter<LogEntry>
Reviewed-by: Ronen Friedman <rfriedma@redhat.com>
Yuri Weinstein [Wed, 22 May 2024 14:38:55 +0000 (07:38 -0700)]
Merge pull request #56477 from sseshasa/wip-65151-squid
squid: qa: Add benign cluster warning from ec-inconsistent-hinfo test to ignorelist
Reviewed-by: Laura Flores <lflores@redhat.com>
Matan Breizman [Wed, 22 May 2024 11:29:11 +0000 (14:29 +0300)]
Merge pull request #57514 from Matan-B/wip-56610-squid
squid: crimson/osd/replicated_recovery_backend: Fix recovery obc usage
Reviewed-by: Samuel Just <sjust@redhat.com>
Matan Breizman [Wed, 22 May 2024 11:28:22 +0000 (14:28 +0300)]
Merge pull request #57511 from Matan-B/wip-56844-squid
squid: crimson/common/tri_mutex: make locking/promotion atomic if possible
Reviewed-by: Samuel Just <sjust@redhat.com>
Matan Breizman [Wed, 22 May 2024 11:28:00 +0000 (14:28 +0300)]
Merge pull request #57510 from Matan-B/wip-56606-squid
squid: crimson/osd/ops_executer: fix snap overlap range error
Reviewed-by: Samuel Just <sjust@redhat.com>
Matan Breizman [Wed, 22 May 2024 11:27:40 +0000 (14:27 +0300)]
Merge pull request #57507 from Matan-B/wip-56848-squid
squid: crimson/osd/recovery_backends: discard outdated recovery ops
Reviewed-by: Samuel Just <sjust@redhat.com>
Matan Breizman [Wed, 22 May 2024 11:26:15 +0000 (14:26 +0300)]
Merge pull request #57505 from Matan-B/wip-56277-squid
squid: crimson: Add support for pool compression
Reviewed-by: Samuel Just <sjust@redhat.com>
Matan Breizman [Wed, 22 May 2024 11:25:46 +0000 (14:25 +0300)]
Merge pull request #57502 from Matan-B/wip-56511-squid
squid: qa/suites/crimson-rados/thrash: enable chance_down
Reviewed-by: Samuel Just <sjust@redhat.com>
Zac Dover [Wed, 22 May 2024 06:04:53 +0000 (16:04 +1000)]
Merge pull request #57559 from ceph/wip-lusov-await-eperm-squid
squid: mds/quiesce: db: quiesce-await should EPERM if a set is past QS_QUIESCED
Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
Matan Breizman [Tue, 21 May 2024 12:37:28 +0000 (15:37 +0300)]
Merge pull request #57513 from Matan-B/wip-57129-squid
squid: crimson/os/seastore/transaction_manager: fix write pipeline phase leak
Reviewed-by: Samuel Just <sjust@redhat.com>
Matan Breizman [Tue, 21 May 2024 12:28:12 +0000 (15:28 +0300)]
Merge pull request #57509 from Matan-B/wip-56383-squid
squid: crimson/os/seastore/btree: clean up `FixedKVLeafNode::get_logical_child()`
Reviewed-by: Samuel Just <sjust@redhat.com>
Matan Breizman [Tue, 21 May 2024 12:27:48 +0000 (15:27 +0300)]
Merge pull request #57508 from Matan-B/wip-56875-squid
squid: crimson/osd/osd_meta: load incremental osdmap from "inc_osdmap.XXX"
Reviewed-by: Samuel Just <sjust@redhat.com>
Matan Breizman [Tue, 21 May 2024 12:26:28 +0000 (15:26 +0300)]
Merge pull request #57506 from Matan-B/wip-56353-squid
squid: crimson/os/seastore: avoid new allocation when overwriting data in RBM for performance
Reviewed-by: Samuel Just <sjust@redhat.com>
Matan Breizman [Tue, 21 May 2024 12:25:24 +0000 (15:25 +0300)]
Merge pull request #57504 from Matan-B/wip-56611-squid
squid: crimson/osd/replicated_recovery_backend: prepare_pull use pg_info
Reviewed-by: Samuel Just <sjust@redhat.com>
Matan Breizman [Tue, 21 May 2024 12:24:57 +0000 (15:24 +0300)]
Merge pull request #57503 from Matan-B/wip-56912-squid
squid: crimson/common/operation: fix and move exit() after entering the next phase
Reviewed-by: Samuel Just <sjust@redhat.com>
Matan Breizman [Tue, 21 May 2024 12:24:21 +0000 (15:24 +0300)]
Merge pull request #57501 from Matan-B/wip-56775-squid
squid: crimson/osd: implement basic reactor-utilization stats report to log
Reviewed-by: Samuel Just <sjust@redhat.com>
Matan Breizman [Tue, 21 May 2024 12:23:44 +0000 (15:23 +0300)]
Merge pull request #57500 from Matan-B/wip-56627-squid
squid: crimson/os/seastore: alloc mapping with refcount when rewriting logical extents
Reviewed-by: Samuel Just <sjust@redhat.com>
Matan Breizman [Tue, 21 May 2024 12:23:11 +0000 (15:23 +0300)]
Merge pull request #57499 from Matan-B/wip-55847-squid
squid: crimson: convert some of client_request to use coroutines
Reviewed-by: Samuel Just <sjust@redhat.com>
Neha Ojha [Mon, 20 May 2024 21:35:42 +0000 (17:35 -0400)]
Merge pull request #57413 from neha-ojha/wip-squid-rc
squid: src/ceph_release: dev -> rc
Reviewed-by: Laura Flores <lflores@redhat.com>
Adam King [Mon, 20 May 2024 17:50:37 +0000 (13:50 -0400)]
Merge pull request #57532 from rhcs-dashboard/wip-66092-squid
squid: mgr/k8sevents: update V1Events to CoreV1Events
Reviewed-by: Laura Flores <lflores@ibm.com>
Neha Ojha [Mon, 20 May 2024 16:53:01 +0000 (16:53 +0000)]
doc/dev/release-checklists.rst: squid milestone added
Signed-off-by: Neha Ojha <nojha@redhat.com>
Neha Ojha [Fri, 10 May 2024 20:59:25 +0000 (20:59 +0000)]
.github/milestone.yml: add squid
Signed-off-by: Neha Ojha <nojha@redhat.com>
(cherry picked from commit
bca45c7df2c26b105401b299161f553ffe4e32c1 )
Leonid Usov [Fri, 26 Apr 2024 00:20:42 +0000 (03:20 +0300)]
mds/quiesce: db: quiesce-await should EPERM if a set is past QS_QUIESCED
Fixes: https://tracker.ceph.com/issues/65669
Signed-off-by: Leonid Usov <leonid.usov@ibm.com>
(cherry picked from commit
fd88b52d6fb11c206b3a06e8533c4551c902f173 )
Fixes: https://tracker.ceph.com/issues/66034
Zac Dover [Mon, 20 May 2024 06:12:48 +0000 (16:12 +1000)]
Merge pull request #57547 from zdover23/wip-doc-2024-05-19-backport-57542-to-squid
doc/cephfs: Squid and later - subvolume quiesce
Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
Zac Dover [Mon, 20 May 2024 06:11:52 +0000 (16:11 +1000)]
Merge pull request #57540 from zdover23/wip-doc-2024-05-18-backport-57534-to-squid
squid: doc/cephfs: edit fs-volumes.rst (2 of x)
Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
Zac Dover [Sun, 19 May 2024 00:00:29 +0000 (10:00 +1000)]
doc/cephfs: Squid and later - subvolume quiesce
Add a note to the "Subvolume quiesce" section that says that the
information in the section applies only to the Squid and later releases
of Ceph. This is included here so that I don't overwrite the Reef and
Quincy documentation with irrelevant information, and so that I don't
overwrite the Squid information with blank space where the "Subvolume
quiesce" section should be.
Signed-off-by: Zac Dover <zac.dover@proton.me>
(cherry picked from commit
be63ca6d037a954caacb3b9737332e886a1f762f )
Zac Dover [Fri, 17 May 2024 10:46:28 +0000 (20:46 +1000)]
doc/cephfs: edit fs-volumes.rst (2 of x)
Edit doc/cephfs/fs-volumes to the section "Cloning Snapshots" (but not
including the section "Cloning Snapshots".
Follows https://github.com/ceph/ceph/pull/57415
Signed-off-by: Zac Dover <zac.dover@proton.me>
(cherry picked from commit
0a3981a011d5e768ab8a5782884283b6650af28a )
Casey Bodley [Fri, 17 May 2024 16:09:06 +0000 (17:09 +0100)]
Merge pull request #57520 from cbodley/wip-66069-squid
squid: cmake: disable WITH_QATLIB/ZIP on non-x86
Reviewed-by: Ken Dreyer <kdreyer@ibm.com>
Nizamudeen A [Fri, 3 May 2024 08:56:19 +0000 (14:26 +0530)]
mgr/k8sevents: update V1Events to CoreV1Events
centos9 only provides kubernetes 26.1.0 as base dep and hence the
k8sevents code needs to be updated accordingly. the api changes happened
in kuberenetes while 19.0.0 was released
Fixes: https://tracker.ceph.com/issues/65627
Fixes: https://tracker.ceph.com/issues/64981
Signed-off-by: Nizamudeen A <nia@redhat.com>
(cherry picked from commit
6af964719217d720e6c2fd1ba2a607f6255d2604 )
Ken Dreyer [Tue, 14 May 2024 18:53:51 +0000 (14:53 -0400)]
cmake: disable WITH_QATLIB/ZIP on non-x86
This feature is only relevant to x86 hosts.
Signed-off-by: Ken Dreyer <kdreyer@ibm.com>
Fixes: https://tracker.ceph.com/issues/66016
Signed-off-by: Ken Dreyer <kdreyer@ibm.com>
(cherry picked from commit
487cd2fddbab784269af9f48206a130e63f1eca3 )
Matan Breizman [Mon, 22 Apr 2024 08:29:22 +0000 (08:29 +0000)]
crimson/osd/replicated_recovery_backend: don't resolve_oid on recovery
Signed-off-by: Matan Breizman <mbreizma@redhat.com>
(cherry picked from commit
48be64b686dee418595210733dbbe6411402adc8 )
Matan Breizman [Mon, 8 Apr 2024 07:52:20 +0000 (07:52 +0000)]
crimson/osd/object_context_loader: SnapTrim to not resolve_oid
SnapTrimObjSubEvent::remove_or_update partially resolves the to be
trimmed clone taking into account in_removed_snaps_queue.
The general resolve_oid is not suitable for this scenario.
Specifically the following check:
```
if (std::find(
citer->second.begin(),
citer->second.end(),
oid.snap) == citer->second.end()) {
logger().debug("{} {} does not contain {} -- DNE",
__func__, ss.clone_snaps, oid.snap);
return std::nullopt;
}
```
because of earlier snap_map_modify call.
Example:
```
INFO 2024-04-07 13:44:01,118 [shard 0:main] osd - SnapTrimObjSubEvent(coid=2:
e8855410 :::folio011816418-576:8 snapid=8): 2:
e8855410 :::folio011816418-576:8 snaps [8, 7] -> {7}
DEBUG 2024-04-07 13:44:01,118 [shard 0:main] osd - snap_map_modify: soid 2:
e8855410 :::folio011816418-576:8, snaps {7}
...
This case will fail:
INFO 2024-04-07 13:44:04,139 [shard 0:main] osd - SnapTrimObjSubEvent(coid=2:
e8855410 :::folio011816418-576:8 snapid=7): 2:
e8855410 :::folio011816418-576:8 snaps [7] -> {} ... deleting
DEBUG 2024-04-07 13:44:04,139 [shard 0:main] osd - snap_map_remove: soid 2:
e8855410 :::folio011816418-576:8
```
Signed-off-by: Matan Breizman <mbreizma@redhat.com>
(cherry picked from commit
263b2ae77ac1df8a939b82101eca09f1d8d4089a )
Matan Breizman [Sun, 7 Apr 2024 12:25:09 +0000 (12:25 +0000)]
crimson/osd/object_context_loader: explicit with_head_obc call
No change in behavior, improved readability
Signed-off-by: Matan Breizman <mbreizma@redhat.com>
(cherry picked from commit
09537e174f20722626da35c584eb6026de79b8a6 )
Matan Breizman [Sun, 7 Apr 2024 12:23:56 +0000 (12:23 +0000)]
crimson/osd/object_context_loader: Simplify with_obc
No change in behavior
Signed-off-by: Matan Breizman <mbreizma@redhat.com>
(cherry picked from commit
47af391c56e241e612efb656811d587cd309c82f )
Matan Breizman [Wed, 3 Apr 2024 08:50:55 +0000 (08:50 +0000)]
crimson/osd/object_context_loader: cleanup with_clone_obc_direct
ObjectContextLoader interface provides two variants:
* with_obc:
// Use this variant by default
// If oid is a clone object, the clone obc *and* it's
// matching head obc will be locked and can be used in func.
* with_clone_obc_only:
// Use this variant in the case where the head object
// obc is already locked and only the clone obc is needed.
// Avoid nesting with_head_obc() calls by using with_clone_obc()
// with an already locked head.
with_clone_obc_direct variant is equal to with_obc on a clone obc
since both the head and the clone obcs will be locked and can be used.
Signed-off-by: Matan Breizman <mbreizma@redhat.com>
(cherry picked from commit
9a27c95b958a6f5b6c52087e8d05899dc465befe )
Matan Breizman [Mon, 1 Apr 2024 08:20:14 +0000 (08:20 +0000)]
crimson/osd/object_context_loader: add comment to with_head_obc
Signed-off-by: Matan Breizman <mbreizma@redhat.com>
(cherry picked from commit
7e208d97e83e780bb939391df674c231a5f2cad8 )
Matan Breizman [Sun, 7 Apr 2024 09:38:06 +0000 (09:38 +0000)]
crimson/osd/object_context_loader: fix with_clone_obc on resolve_oid case
Resolve_oid on a clone object may actually return the head:
```
// Because oid.snap > ss.seq, we are trying to read from a snapshot
// taken after the most recent write to this object. Read from head.
```
In this case, with_clone_obc should apply `func` same as with_head_obc would have.
Note: previously, with_clone_obc_only was called on the resolved head object.
While it didn't cause any errors, using the head_obc as clone is wrong.
Signed-off-by: Matan Breizman <mbreizma@redhat.com>
(cherry picked from commit
d8aad5576f67c0e255b42c5d26334c87ba705931 )
Matan Breizman [Wed, 3 Apr 2024 08:56:56 +0000 (08:56 +0000)]
crimson/osd/object_context_loader: clones to support ssc
Previously, only the head obc had ssc reference. Let clone obc
also reference it's head ssc.
Fixes: https://tracker.ceph.com/issues/65203
Fixes: https://tracker.ceph.com/issues/65201
Signed-off-by: Matan Breizman <mbreizma@redhat.com>
(cherry picked from commit
5de3da0edaeb3b415cfc9359efb06cd0d7fb58d0 )
Xuehan Xu [Wed, 24 Apr 2024 09:00:53 +0000 (17:00 +0800)]
crimson/os/seastore/transaction_manager: fix write pipeline phase leak
At present, if a transaction gets interrupted right after it enters
WritePipeline::ReserveProjectedUsage and before any later continuations
get executed, WritePipeline::ReserveProjectedUsage will be locked
forever.
Signed-off-by: Xuehan Xu <xuxuehan@qianxin.com>
(cherry picked from commit
6a6f340c09a8dfe4565e298db11a30345ef7f82f )
Yingxin Cheng [Mon, 15 Apr 2024 02:03:35 +0000 (10:03 +0800)]
crimson/common/tri_mutex: make promotion atomic with func
Specifically, make promotion atomic with load-obc to fix
assert(readers/writers == 1) failures.
Fixes: https://tracker.ceph.com/issues/65451
Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
(cherry picked from commit
676947e1f73a9d7f900df5a6854c283c4eef24a9 )
Yingxin Cheng [Fri, 19 Apr 2024 06:28:33 +0000 (14:28 +0800)]
crimson/common/tri_mutex: use seastar::now()
Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
(cherry picked from commit
26b96f93bb1922e1f430ae0480aafa8c7ee0d38a )
Yingxin Cheng [Mon, 15 Apr 2024 02:02:48 +0000 (10:02 +0800)]
crimson/common/tri_mutex: improve comments
Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
(cherry picked from commit
8eb9f0380f4328c672cbde9d3e1971b730695d73 )
Yingxin Cheng [Thu, 21 Mar 2024 01:43:34 +0000 (09:43 +0800)]
crimson/common/tri_mutex: drop the unused greedy param
Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
(cherry picked from commit
9cdb67fa19ea7d912a5f3e7a14092a205dee80ac )
junxiang Mu [Mon, 1 Apr 2024 07:00:14 +0000 (03:00 -0400)]
crimson/osd/ops_executer: fix snap overlap range error
Fixes: https://tracker.ceph.com/issues/65113
Signed-off-by: junxiang Mu <1948535941@qq.com>
(cherry picked from commit
7eca779627a90dc80f54957cc49b25b4c965044d )
Xuehan Xu [Fri, 22 Mar 2024 07:22:46 +0000 (15:22 +0800)]
crimson/os/seastore/btree: clean up `FixedKVLeafNode::get_logical_child()`
Signed-off-by: Xuehan Xu <xuxuehan@qianxin.com>
(cherry picked from commit
6e4f52a8ff93484d156f1d9095608a55e9c46aed )
Xuehan Xu [Sun, 14 Apr 2024 07:20:31 +0000 (15:20 +0800)]
crimson/osd/osd_meta: load incremental osdmap from "inc_osdmap.XXX"
Fixes: https://tracker.ceph.com/issues/65474
Signed-off-by: Xuehan Xu <xxhdx1985126@gmail.com>
(cherry picked from commit
69271aef7002d0a76b43e8d1fc0667355119823e )
Xuehan Xu [Fri, 12 Apr 2024 06:01:16 +0000 (14:01 +0800)]
crimson/osd/recovery_backends: discard outdated recovery ops
Fixes: https://tracker.ceph.com/issues/65453
Signed-off-by: Xuehan Xu <xuxuehan@qianxin.com>
(cherry picked from commit
234f41c33917e921b53c091af7b3fcbf0f141b4a )
myoungwon oh [Fri, 19 Apr 2024 04:57:52 +0000 (04:57 +0000)]
crimson/seastore: add a TODO comment regarding is_data_stable()
Signed-off-by: Myoungwon Oh <myoungwon.oh@samsung.com>
(cherry picked from commit
5df6ffc79b415def0b7984096413e9562e83399a )
myoungwon oh [Thu, 18 Apr 2024 11:30:45 +0000 (11:30 +0000)]
common/options: correct explanation regarding delta_based_overwrite
Signed-off-by: Myoungwon Oh <myoungwon.oh@samsung.com>
(cherry picked from commit
387c0c89992b5a28a7c5bba9f720a951aee1673b )
myoungwon oh [Tue, 9 Apr 2024 06:31:12 +0000 (06:31 +0000)]
test/crimson/seastore: rename set/unset_overwrite_threshold() to enable/disable_delta_based_overwrite()
Signed-off-by: Myoungwon Oh <myoungwon.oh@samsung.com>
(cherry picked from commit
24c0c198a5dd57070f210dc69dd83481df73fa15 )
myoungwon oh [Thu, 21 Mar 2024 02:06:24 +0000 (02:06 +0000)]
crimson/os/seastore: avoid new allocation when overwriting data in RBM for performance
In 4K random write test, after seastore is filled up by 4MB extents,
current implementation performs deep copy in duplicate_for_write(), resulting in
significant performance degradation by 80%.
Therefore, this commit changes the deep copy behavior for bufferptr during the overwrite
situation to shallow copy, leaving the original data untouched.
Signed-off-by: Myoungwon Oh <myoungwon.oh@samsung.com>
Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
(cherry picked from commit
76b87855b4a6fb60a1fae59792aeebff3e8762d3 )
Aishwarya Mathuria [Wed, 31 Jan 2024 09:20:44 +0000 (09:20 +0000)]
crimson: Add support for pool compression
1. Send pool options to Bluestore which include compression options as well.
2. Add pool related stats to MPGStats so that they all compression details are available in 'ceph df detail' command.
Fixes: https://tracker.ceph.com/issues/59242
Signed-off-by: Aishwarya Mathuria <amathuri@redhat.com>
(cherry picked from commit
ba4f62c49ecee26d98100bb5cdb15ecfe212f0be )
Matan Breizman [Mon, 1 Apr 2024 08:58:48 +0000 (08:58 +0000)]
crimson/osd/replicated_recovery_backend: prepare_pull to use pg_info
Don't use peer's info on prepare_pull.
Fixes: https://tracker.ceph.com/issues/65200
Signed-off-by: Matan Breizman <mbreizma@redhat.com>
(cherry picked from commit
7586abfad239d433049fb714d6fb8f2530a1b9c6 )
Yingxin Cheng [Tue, 16 Apr 2024 07:53:47 +0000 (15:53 +0800)]
crimson/common/operation: fix and move exit() after entering the next phase
If exit/unlock the barrier before entering the next phase, it is
possible for the next request to exit the barrier at the same time, and
enters the next phase first, causing reorder issues.
Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
(cherry picked from commit
078768ff961abad12cf3e7f19190a6f986fc5fce )
Matan Breizman [Wed, 27 Mar 2024 08:37:40 +0000 (08:37 +0000)]
qa/suites/crimson-rados/thrash: enable chance_down
As the thrash tests were introduced, some options were disabled
until the tests are stabilized.
Re-enable chance_down option (default is 0.4) to detect bugs on restart.
Since it will probably take few iterations before thrash and recovery tests ('default.yaml')
will pass successfully, add anoter 'simple.yaml' which should remain stable.
Fixes: https://tracker.ceph.com/issues/65130
Signed-off-by: Matan Breizman <mbreizma@redhat.com>
(cherry picked from commit
372751789be1725ed06fd077d029295c8cab4c35 )
Yingxin Cheng [Tue, 9 Apr 2024 03:06:36 +0000 (11:06 +0800)]
crimson/osd: implement basic reactor-utilization stats report to log
Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
(cherry picked from commit
99909d45a282fb9dc89c6e8c98f4b866e67b09cb )
Zhang Song [Wed, 3 Apr 2024 09:21:43 +0000 (17:21 +0800)]
crimson/os/seastore: alloc mapping with refcount when rewriting logical extents
Signed-off-by: Zhang Song <zhangsong02@qianxin.com>
(cherry picked from commit
9269e68eacc825ccbe023e325c405bc0636d6b58 )
Zhang Song [Wed, 3 Apr 2024 09:10:31 +0000 (17:10 +0800)]
crimson/os/seastore/btree_lba_manager: update_refcount returns the refcount of intermediate mapping
Signed-off-by: Zhang Song <zhangsong02@qianxin.com>
(cherry picked from commit
513bb4f3f21b739ea3f751680716b0fbcda9c3e1 )
Zhang Song [Wed, 3 Apr 2024 09:07:52 +0000 (17:07 +0800)]
crimson/os/seastore/btree_lba_manager: cleanup methods that return std::pair
Signed-off-by: Zhang Song <zhangsong02@qianxin.com>
(cherry picked from commit
36f96ca000ab8839b7649c32f6d53ea88f9d9700 )
Zhang Song [Wed, 3 Apr 2024 08:27:50 +0000 (16:27 +0800)]
crimson/os/seastore: introduce extent_ref_count_t
Signed-off-by: Zhang Song <zhangsong02@qianxin.com>
(cherry picked from commit
bbee55a105ffb31d01c9a024325b8eb06d8f56e7 )
Zhang Song [Wed, 3 Apr 2024 08:02:22 +0000 (16:02 +0800)]
crimson/os/seastore: remove unused return value of RecordScanner::scan_valid_records
Signed-off-by: Zhang Song <zhangsong02@qianxin.com>
(cherry picked from commit
1a914f64070299ab5712ce389096052709b72622 )
Zhang Song [Wed, 3 Apr 2024 08:01:59 +0000 (16:01 +0800)]
crimson/common/interruptible_future: add discard_result
Signed-off-by: Zhang Song <zhangsong02@qianxin.com>
(cherry picked from commit
870441bb0c02992923ac6e0b6373b6d3770b470b )
Samuel Just [Fri, 9 Feb 2024 19:49:13 +0000 (11:49 -0800)]
crimson/.../client_request: work around gcc bz98401
See included comment for details.
Signed-off-by: Samuel Just <sjust@redhat.com>
(cherry picked from commit
1388173764408d047f2fbd5cf7b33dcf42971a03 )
Samuel Just [Fri, 9 Feb 2024 02:45:43 +0000 (18:45 -0800)]
crimson/.../client_request: work around gcc bz101244 and bz102217
See included comment for details.
Signed-off-by: Samuel Just <sjust@redhat.com>
(cherry picked from commit
90dcbfa0fa9d8a46f8075a7e6f2329652f0a312f )
Samuel Just [Wed, 7 Feb 2024 02:05:42 +0000 (02:05 +0000)]
crimson/.../client_request: convert ClientRequest::do_process to coroutine
Signed-off-by: Samuel Just <sjust@redhat.com>
(cherry picked from commit
30237472bea485cb57f5c14031fb545aed97da15 )
Samuel Just [Wed, 7 Feb 2024 01:19:37 +0000 (17:19 -0800)]
crimson/.../client_request: convert process_op to coroutine
Signed-off-by: Samuel Just <sjust@redhat.com>
(cherry picked from commit
a4905fb3e2f2400656f0b436275e9fa745f14cc1 )
Samuel Just [Wed, 7 Feb 2024 01:19:13 +0000 (17:19 -0800)]
crimson/.../client_request: make reply_op_error return interruptible_future<>
co_await can't implicitely convert it.
Signed-off-by: Samuel Just <sjust@redhat.com>
(cherry picked from commit
ecf5795b6666f122fa3bf2771e83b066e611ade9 )
Samuel Just [Wed, 7 Feb 2024 00:32:38 +0000 (00:32 +0000)]
crimson/.../client_request: convert ClientRequest::process_pg_op to coroutine
Signed-off-by: Samuel Just <sjust@redhat.com>
(cherry picked from commit
0de3613072659cfdcb41ea7b1df5865425774b3b )
Samuel Just [Mon, 5 Feb 2024 20:46:19 +0000 (20:46 +0000)]
crimson/.../client_request: convert with_pg_process_interruptible coroutine
Signed-off-by: Samuel Just <sjust@redhat.com>
(cherry picked from commit
904a9937b6560f9a1f936f7f79badc56d8949dd1 )
Samuel Just [Mon, 5 Feb 2024 00:04:49 +0000 (16:04 -0800)]
crimson/.../client_request: factor out with_pg_interruptible
Signed-off-by: Samuel Just <sjust@redhat.com>
(cherry picked from commit
b521cc3d822c39084f72c4f22f621928a184d731 )
Samuel Just [Tue, 6 Feb 2024 04:46:57 +0000 (20:46 -0800)]
crimson/.../client_request: don't pass Ref<PG> by reference
If we only need a reference to the PG, pass a PG&. Passing Ref<PG>&
makes it easy to inadvertently std::move() the passed value from
a caller.
Signed-off-by: Samuel Just <sjust@redhat.com>
(cherry picked from commit
4164b08887c2b2c7a9df0cd80a681bb61be28ee6 )
Samuel Just [Sun, 4 Feb 2024 22:29:38 +0000 (14:29 -0800)]
crimson/.../client_request: rename with_pg_int to with_pg_process
Signed-off-by: Samuel Just <sjust@redhat.com>
(cherry picked from commit
13042dde1134983f074f5bd4cf4c912d47d0193c )
Samuel Just [Sun, 4 Feb 2024 22:28:46 +0000 (14:28 -0800)]
crimson/.../client_request.cc: move message decode check to with_pg
We only need to do this once, no need to recheck on requeue.
Signed-off-by: Samuel Just <sjust@redhat.com>
(cherry picked from commit
10a9a11f5d535044f081964dc6b79813aa9bb624 )
Matan Breizman [Thu, 16 May 2024 11:36:50 +0000 (14:36 +0300)]
Merge pull request #57316 from Matan-B/wip-56027-squid
squid: doc/dev/crimson: add Code Walkthroughs
Reviewed-by: Samuel Just <sjust@redhat.com>
Matan Breizman [Thu, 16 May 2024 11:36:13 +0000 (14:36 +0300)]
Merge pull request #57484 from Matan-B/wip-56534-squid
squid: crimson: introduce INTERNAL_PG_LOCAL_NS, use for snapmapper
Reviewed-by: Samuel Just <sjust@redhat.com>
Matan Breizman [Thu, 16 May 2024 07:38:03 +0000 (10:38 +0300)]
Merge pull request #57483 from Matan-B/wip-56378-squid
squid: crimson/osd/osd_operations/snaptrim_event: update PG's stats
Reviewed-by: Samuel Just <sjust@redhat.com>
Matan Breizman [Thu, 16 May 2024 07:37:37 +0000 (10:37 +0300)]
Merge pull request #57482 from Matan-B/wip-56246-squid
squid: crimson/os/seastore/btree: check for reserved ptrs when determining children stability
Reviewed-by: Samuel Just <sjust@redhat.com>
Matan Breizman [Thu, 16 May 2024 07:37:01 +0000 (10:37 +0300)]
Merge pull request #57481 from Matan-B/wip-55947-squid
squid: crimson/os/seastore: handle OP_COLL_MOVE_RENAME
Reviewed-by: Samuel Just <sjust@redhat.com>
Matan Breizman [Thu, 16 May 2024 07:36:31 +0000 (10:36 +0300)]
Merge pull request #57480 from Matan-B/wip-55708-squid
squid: crimson/osd/main: Enable multicore cluster messenger
Reviewed-by: Samuel Just <sjust@redhat.com>
Matan Breizman [Thu, 16 May 2024 07:36:00 +0000 (10:36 +0300)]
Merge pull request #57270 from Matan-B/wip-55632-squid
squid: debian/control: add new dependencies protobuf for crismon
Reviewed-by: Samuel Just <sjust@redhat.com>
Matan Breizman [Wed, 15 May 2024 16:19:55 +0000 (19:19 +0300)]
Merge pull request #57287 from Matan-B/wip-55751-squid
squid: test/crimson/test_socket: complete write-side errors in test_unexpected_down()
Reviewed-by: Samuel Just <sjust@redhat.com>
Matan Breizman [Wed, 8 May 2024 09:52:19 +0000 (12:52 +0300)]
common/hobject: make hobject_t formatter methods const
this is an alternative to
6021129f0c5c1c133436e471741c73f6e315c5dd
Since https://github.com/ceph/ceph/pull/55583 is not backported to S.
Signed-off-by: Matan Breizman <mbreizma@redhat.com>
Samuel Just [Tue, 26 Mar 2024 06:17:10 +0000 (23:17 -0700)]
crimson/.../scrub_events: use PGBackend::list_objects in ScrubScan::run
Fixes: https://tracker.ceph.com/issues/64975
Signed-off-by: Samuel Just <sjust@redhat.com>
(cherry picked from commit
e69883ad2a05afab9d9ab02b098ad2418a4eec8a )
Samuel Just [Tue, 26 Mar 2024 04:20:48 +0000 (21:20 -0700)]
crimson/.../scrub_events: convert ScrubScan::run to coroutine
Signed-off-by: Samuel Just <sjust@redhat.com>
(cherry picked from commit
0d042f9d18fb191c3c8d6dbbf7c5bae54b8ed0fb )
Samuel Just [Wed, 27 Mar 2024 01:43:06 +0000 (01:43 +0000)]
crimson/.../scrub_events: use PGBackend::list_objects in ScrubFindRange::run
Signed-off-by: Samuel Just <sjust@redhat.com>
(cherry picked from commit
9f39124f33e878e321b330e58e97efc0a1cf1242 )
Samuel Just [Wed, 27 Mar 2024 01:39:17 +0000 (01:39 +0000)]
crimson/.../scrub_events: convert ScrubFindRange::run to a coroutine
Signed-off-by: Samuel Just <sjust@redhat.com>
(cherry picked from commit
5b0416ae100443f43361e1308e2d28e2763ba725 )
Samuel Just [Tue, 26 Mar 2024 05:44:37 +0000 (22:44 -0700)]
crimson: move snapmapper object into INTERNAL_PG_LOCAL_NS
Signed-off-by: Samuel Just <sjust@redhat.com>
(cherry picked from commit
1b7e3c4fb773a6e0a16145114013a07aa85edcc5 )
Samuel Just [Tue, 26 Mar 2024 05:36:07 +0000 (22:36 -0700)]
crimson/osd/pg_backend: introduce INTERNAL_PG_LOCAL_NS, skip in PGBackend::list_objects
Signed-off-by: Samuel Just <sjust@redhat.com>
(cherry picked from commit
f32b37a599592d35ea948e66e7cfdfef7113b8ac )
Samuel Just [Tue, 26 Mar 2024 05:31:08 +0000 (05:31 +0000)]
crimson/osd/pg_backend: convert PGBackend::list_objects to a coroutine
Signed-off-by: Samuel Just <sjust@redhat.com>
(cherry picked from commit
4f056a4c1e9f01e4b04c96e85ebcfa29e6469adc )
Samuel Just [Tue, 26 Mar 2024 05:13:40 +0000 (22:13 -0700)]
crimson/osd/pg_backend: generalize PGBackend::list_objects, add overloads
Signed-off-by: Samuel Just <sjust@redhat.com>
(cherry picked from commit
22a628534307d35e3fa90fb36a98f81cbf7cb7f6 )
junxiang Mu [Fri, 22 Mar 2024 02:44:49 +0000 (22:44 -0400)]
crimson/osd/osd_operations/snaptrim_event: update PG's stats
Fixes: https://tracker.ceph.com/issues/63307
Signed-off-by: junxiang Mu <1948535941@qq.com>
(cherry picked from commit
4a180f559a3fff1615664898603fc832cfd8d72c )
Xuehan Xu [Thu, 21 Mar 2024 01:54:42 +0000 (09:54 +0800)]
crimson/os/seastore/btree: always check the stability of extents within
the current transaction's view
Signed-off-by: Xuehan Xu <xuxuehan@qianxin.com>
(cherry picked from commit
7dcd74ac098201d2ea6f9d1c723f9944ade6bf33 )
Xuehan Xu [Mon, 18 Mar 2024 09:24:02 +0000 (17:24 +0800)]
crimson/os/seastore/cached_extent: add comments to elaborate why
MUTATION_PENDING and under-io extents are "stable"
Signed-off-by: Xuehan Xu <xuxuehan@qianxin.com>
(cherry picked from commit
b1c59ca077597e14a053d47d4961b12ba7a2a1b0 )
Xuehan Xu [Mon, 18 Mar 2024 02:54:24 +0000 (10:54 +0800)]
crimson/os/seastore/btree: check for reserved ptrs when determining
children stability
Fixes: https://tracker.ceph.com/issues/64957
Signed-off-by: Xuehan Xu <xuxuehan@qianxin.com>
(cherry picked from commit
7aa972def9f53fd9ed2f708aa87ce83362765cbf )
Xuehan Xu [Thu, 7 Mar 2024 09:06:13 +0000 (17:06 +0800)]
test/crimson/seastore: add the test case for renaming objects
Signed-off-by: Xuehan Xu <xuxuehan@qianxin.com>
(cherry picked from commit
2520927590ac48783cb35feb570285ec066964a4 )
Xuehan Xu [Tue, 5 Mar 2024 07:58:05 +0000 (15:58 +0800)]
crimson/os/seastore: handle OP_COLL_MOVE_RENAME
Signed-off-by: Xuehan Xu <xuxuehan@qianxin.com>
(cherry picked from commit
70245c5b207eca6349920d1877ea237ec687f93d )
Matan Breizman [Thu, 7 Mar 2024 09:49:34 +0000 (09:49 +0000)]
crimson/osd/osd: handle_scrub_message on any core
After enabling multicore cluster messenger, there is no
guarantee which core will handle the request.
See: OSDSingletonState::send_to_osd()
Signed-off-by: Matan Breizman <mbreizma@redhat.com>
(cherry picked from commit
cbad3e4517e91476b19a8766f98240c81df40a36 )
Matan Breizman [Thu, 22 Feb 2024 09:35:14 +0000 (09:35 +0000)]
crimson/osd/main: Enable multicore cluster messenger
Taken from:
f78e99c
Co-authored-by: Yingxin Cheng <yingxin.cheng@intel.com>
Co-authored-by: Chunmei Liu <chunmei.liu@intel.com>
Co-authored-by: Xinyu Huang <xinyu.huang@intel.com>
Signed-off-by: Matan Breizman <mbreizma@redhat.com>
(cherry picked from commit
ab239ad6cc5eb1cb4ca71d5b409bc4001124342f )
Matan Breizman [Wed, 15 May 2024 14:36:26 +0000 (17:36 +0300)]
Merge pull request #57347 from Matan-B/wip-56375-squid
squid: crimson/.../scrub_events: fix progress lifetime in deep_scan_object
Reviewed-by: Samuel Just <sjust@redhat.com>