]>
git.apps.os.sepia.ceph.com Git - ceph.git/log
Zac Dover [Wed, 30 Nov 2022 23:01:17 +0000 (09:01 +1000)]
doc/cephadm: improve airgapping procedure grammar
Improve the grammar of the airgapping procedure.
Signed-off-by: Zac Dover <zac.dover@gmail.com>
Casey Bodley [Wed, 30 Nov 2022 16:55:03 +0000 (11:55 -0500)]
Merge pull request #48908 from kalebskeithley/ceph-zipper-utils
rgw: refactor selected files for better above- vs below-the-line
Reviewed-by: Daniel Gryniewicz <dang@redhat.com>
zdover23 [Wed, 30 Nov 2022 04:52:34 +0000 (14:52 +1000)]
Merge pull request #49147 from zdover23/wip-doc-2022-11-30-cephadm-install-airgap-formatting
doc/cephadm: format airgap install procedure
Reviewed-by: Cole Mitchell <cole.mitchell@gmail.com>
Zac Dover [Wed, 30 Nov 2022 04:31:17 +0000 (14:31 +1000)]
doc/cephadm: format airgap install procedure
Format the procedure that explains how to install an airgapped cluster.
Signed-off-by: Zac Dover <zac.dover@gmail.com>
Nizamudeen A [Wed, 30 Nov 2022 04:26:33 +0000 (09:56 +0530)]
Merge pull request #49060 from rhcs-dashboard/accept-cors-dashboard
mgr/dashboard: allow cross origin when the url is set
Reviewed-by: Avan Thakkar <athakkar@redhat.com>
Reviewed-by: Pere Diaz Bou <pdiazbou@redhat.com>
zdover23 [Wed, 30 Nov 2022 04:16:06 +0000 (14:16 +1000)]
Merge pull request #49143 from zdover23/wip-doc-2022-11-30-44346-update
doc/cephadm: add airgapped install procedure
Reviewed-by: Cole Mitchell <cole.mitchell@gmail.com>
Zac Dover [Wed, 30 Nov 2022 03:56:52 +0000 (13:56 +1000)]
doc/cephadm: add airgapped install procedure
Add a procedure describing an installation with an airgapped registry.
This commit ingests work done in https://github.com/ceph/ceph/pull/44346
that was abandoned for lo these past eleven months. The PR connected
with this commit supersedes that PR.
Signed-off-by: Zac Dover <zac.dover@gmail.com>
Yingxin [Wed, 30 Nov 2022 01:23:46 +0000 (09:23 +0800)]
Merge pull request #47969 from myoungwon/wip-rbm-integration-step1
crimson/os/seastore: add RBM OOL path
Reviewed-by: Yingxin Cheng <yingxin.cheng@intel.com>
Reviewed-by: Samuel Just <sjust@redhat.com>
Casey Bodley [Tue, 29 Nov 2022 21:28:01 +0000 (16:28 -0500)]
Merge pull request #49131 from cbodley/wip-rgw-log-identity-type
rgw: initialize rgw_log_entry::identity_type
Reviewed-by: Matt Benjamin <mbenjamin@redhat.com>
Kaleb S. KEITHLEY [Wed, 16 Nov 2022 12:36:47 +0000 (07:36 -0500)]
rgw: refactor selected files for better above- vs below-the-line
Based on https://github.com/ceph/ceph/pull/48272, separate selected methods
into new files for above-the-line vs below-the-line linkage. This is more of
the work to prepare for eventual merging of the loadable module implementation.
"Utility" functions that don't reference the Store are moved, e.g. from
rgw_zone.cc, into the new rgw_zone_utils.cc. Methods in the new *_utils.cc
files are above-the-line.
Signed-off-by: Kaleb S. KEITHLEY <kkeithle@redhat.com>
Casey Bodley [Tue, 29 Nov 2022 18:30:04 +0000 (13:30 -0500)]
rgw: default-initialize delete_multi_obj_op_meta
Signed-off-by: Casey Bodley <cbodley@redhat.com>
Casey Bodley [Tue, 29 Nov 2022 15:11:03 +0000 (10:11 -0500)]
rgw: initialize rgw_log_entry::identity_type
uninitialized identity_type field was causing unit test failures from
check-generated.sh:
**** rgw_log_entry test 2 dump_json check failed ****
ceph-dencoder type rgw_log_entry select_test 2 dump_json > /tmp/typ-AW7AEODB3
ceph-dencoder type rgw_log_entry select_test 2 encode decode dump_json > /tmp/typ-0NX4cmLHR
21c21
< "identity_type":
3599295712
---
> "identity_type":
2072560864
Fixes: https://tracker.ceph.com/issues/58115
Signed-off-by: Casey Bodley <cbodley@redhat.com>
zdover23 [Tue, 29 Nov 2022 17:44:43 +0000 (03:44 +1000)]
Merge pull request #49136 from zdover23/wip-doc-2022-11-30-glossary-dashboard-module
doc/glossary: add "Dashboard Module"
Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
Zac Dover [Tue, 29 Nov 2022 17:26:04 +0000 (03:26 +1000)]
doc/glossary: add "Dashboard Module"
Define "Dashboard Module" in doc/glossary.rst.
Signed-off-by: Zac Dover <zac.dover@gmail.com>
zdover23 [Tue, 29 Nov 2022 17:06:20 +0000 (03:06 +1000)]
Merge pull request #49133 from zdover23/wip-doc-2022-11-30-glossary-rados-cluster
doc/glossary: add "RADOS Cluster"
Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
Zac Dover [Tue, 29 Nov 2022 16:52:31 +0000 (02:52 +1000)]
doc/glossary: add "RADOS Cluster"
Define "RADOS Cluster" in the glossary.
Signed-off-by: Zac Dover <zac.dover@gmail.com>
Matan [Tue, 29 Nov 2022 15:35:15 +0000 (17:35 +0200)]
Merge pull request #48682 from Matan-B/wip-matanb-obc-loader
crimson/osd: Introduce ObjectContextLoader
Reviewed-by: Samuel Just <sjust@redhat.com>
Reviewed-by: chunmei-liu <chunmei.liu@intel.com>
Matan [Tue, 29 Nov 2022 15:31:04 +0000 (17:31 +0200)]
Merge pull request #48831 from rzarzynski/wip-crimson-list_snaps-fix-errmsg
crimson/osd: fix error msg in OpsExecuter::do_list_snaps()
Reviewed-by: Samuel Just <sjust@redhat.com>
Matan [Tue, 29 Nov 2022 15:29:11 +0000 (17:29 +0200)]
Merge pull request #48527 from rzarzynski/wip-crimson-simpler-cloning
crimson/osd: simplify cloning
Reviewed-by: Samuel Just <sjust@redhat.com>
zdover23 [Tue, 29 Nov 2022 14:05:20 +0000 (00:05 +1000)]
Merge pull request #49119 from zdover23/wip-doc-2022-11-29-rados-ops-change-mon-elections-prompts
doc/rados: add prompts to change-mon-elections.rst
Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
zdover23 [Tue, 29 Nov 2022 13:59:08 +0000 (23:59 +1000)]
Merge pull request #49120 from zdover23/wip-doc-2022-11-29-rados-ops-control-prompts
doc/rados: add prompts to control.rst
Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
zdover23 [Tue, 29 Nov 2022 13:56:23 +0000 (23:56 +1000)]
Merge pull request #49118 from zdover23/wip-doc-2022-11-29-rados-cache-tiering-prompts
doc/rados: add prompts to cache-tiering.rst
Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
zdover23 [Tue, 29 Nov 2022 13:51:16 +0000 (23:51 +1000)]
Merge pull request #49114 from zdover23/wip-doc-2022-11-29-rados-bluestore-migration-prompts
doc/rados: add prompts to bluestore-migration.rst
Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
Zac Dover [Tue, 29 Nov 2022 13:30:02 +0000 (23:30 +1000)]
doc/rados: add prompts to control.rst
Add unselectable prompts to doc/rados/operations/control.rst.
https://tracker.ceph.com/issues/57108
Signed-off-by: Zac Dover <zac.dover@gmail.com>
Zac Dover [Tue, 29 Nov 2022 13:10:59 +0000 (23:10 +1000)]
doc/rados: add prompts to change-mon-elections.rst
Add unselectable prompts to
doc/rados/operations/change-mon-elections.rst.
https://tracker.ceph.com/issues/57108
Signed-off-by: Zac Dover <zac.dover@gmail.com>
Nizamudeen A [Tue, 29 Nov 2022 12:59:39 +0000 (18:29 +0530)]
Merge pull request #48525 from rhcs-dashboard/fix-PgImbalance-alert
mgr/dashboard: fix CephPGImbalance alert
Reviewed-by: Avan Thakkar <athakkar@redhat.com>
Reviewed-by: Ernesto Puerta <epuertat@redhat.com>
Reviewed-by: Nizamudeen A <nia@redhat.com>
Reviewed-by: Tatjana Dehler <tdehler@suse.com>
Nizamudeen A [Tue, 29 Nov 2022 12:59:01 +0000 (18:29 +0530)]
Merge pull request #48526 from rhcs-dashboard/fix-cephPoolGrowth-alert
mgr/dashboard: Fix CephPoolGrowthWarning alert
Reviewed-by: Pegonzal <NOT@FOUND>
Reviewed-by: Ernesto Puerta <epuertat@redhat.com>
Reviewed-by: Nizamudeen A <nia@redhat.com>
Zac Dover [Tue, 29 Nov 2022 12:28:36 +0000 (22:28 +1000)]
doc/rados: add prompts to cache-tiering.rst
Add unselectable prompts to doc/rados/operations/cache-tiering.rst.
https://tracker.ceph.com/issues/57108
Signed-off-by: Zac Dover <zac.dover@gmail.com>
Zac Dover [Tue, 29 Nov 2022 11:08:10 +0000 (21:08 +1000)]
doc/rados: add prompts to bluestore-migration.rst
Add unselectable prompts to
doc/rados/operations/bluestore-migration.rst.
https://tracker.ceph.com/issues/57108
Signed-off-by: Zac Dover <zac.dover@gmail.com>
zdover23 [Tue, 29 Nov 2022 07:43:48 +0000 (17:43 +1000)]
Merge pull request #49107 from zdover23/wip-doc-2022-11-29-rados-balancer-prompts
doc/rados: add prompts to balancer.rst
Reviewed-by: Cole Mitchell <cole.mitchell@gmail.com>
Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
Zac Dover [Tue, 29 Nov 2022 05:41:11 +0000 (15:41 +1000)]
doc/rados: add prompts to balancer.rst
Add unselectable prompts to doc/rados/operations/balancer.rst.
https://tracker.ceph.com/issues/57108
Signed-off-by: Zac Dover <zac.dover@gmail.com>
Venky Shankar [Tue, 29 Nov 2022 04:07:04 +0000 (09:37 +0530)]
Merge pull request #49015 from batrick/meta-doc
doc/cephfs: correct names for fs/meta
Reviewed-by: Venky Shankar <vshankar@redhat.com>
Reviewed-by: Kotresh HR <khiremat@redhat.com>
Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
Yuri Weinstein [Mon, 28 Nov 2022 18:38:12 +0000 (10:38 -0800)]
Merge pull request #48490 from aclamk/wip-aclamk-fix-deferred
[bluestore] Improve deferred write decision
Reviewed-by: Mark Nelson <mnelson@redhat.com>
Reviewed-by: Igor Fedotov <ifedotov@suse.com>
Nizamudeen A [Fri, 25 Nov 2022 14:53:31 +0000 (20:23 +0530)]
mgr/dashboard: allow cross origin when the url is set
Allow CORS when the cross_origin_url is set in the config opt.
you have to update the cross_origin_url setting with the url of the
requesting entity.
The request needs to have the header `Access-Control-Allow-Origin`
with the origin URL
The url can be set using this command
`ceph config set mgr mgr/dashboard/cross_origin_url
http://localhost:4200`
multiple urls can be set as
`ceph config set mgr mgr/dashboard/cross_origin_url
http://localhost:4200,http://localhost:4201`
If multiple url is provided in the configuration option, then whatever
url is there in the Access-Control-Allow-Origin request header will be
allowed for CORS
Once the URL is set you have to restart the dashboard module to restart
the cherrypy server with the new CORS policies
Fixes: https://tracker.ceph.com/issues/58086
Signed-off-by: Nizamudeen A <nia@redhat.com>
Samuel Just [Mon, 28 Nov 2022 16:43:12 +0000 (08:43 -0800)]
Merge pull request #49053 from tchaikov/wip-crimson-fmt-v9
crimson: various fmt related changes and fixes
Reviewed-by: Yingxin Cheng <yingxin.cheng@intel.com>
Mark Nelson [Mon, 28 Nov 2022 16:32:16 +0000 (10:32 -0600)]
Merge pull request #47221 from markhpc/wip-faster-rocksdb
[WIP] kv/RocksDBStore: Improved RocksDB Settings and Tombstone behavior
Yuval Lifshitz [Mon, 28 Nov 2022 16:01:32 +0000 (18:01 +0200)]
Merge pull request #48875 from yuvalif/wip-yuval-fix-58014
rgw/notifications: sending metadata in COPY and CompleteMultipartUpload
Ilya Dryomov [Mon, 28 Nov 2022 09:29:00 +0000 (10:29 +0100)]
Merge pull request #49082 from zdover23/wip-doc-2022-11-28-rbd-isci-monitoring-prompts
doc/rbd: add prompts to iscsi-monitoring.rst
Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
Reviewed-by: Ilya Dryomov <idryomov@gmail.com>
Yuval Lifshitz [Mon, 28 Nov 2022 06:01:06 +0000 (08:01 +0200)]
Merge pull request #48598 from yuvalif/wip-yuval-fix-57899
rgw/admin/notifications: support admin operations on topics with tenants
zdover23 [Mon, 28 Nov 2022 00:16:58 +0000 (10:16 +1000)]
Merge pull request #49079 from zdover23/wip-doc-2022-11-28-glossary-ceph-system-remove
doc/glossary: s/Ceph System/Ceph Cluster/
Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
Zac Dover [Mon, 28 Nov 2022 00:11:04 +0000 (10:11 +1000)]
doc/rbd: add prompts to iscsi-monitoring.rst
Add unselectable prompts to doc/rbd/iscsi-monitoring.rst.
Signed-off-by: Zac Dover <zac.dover@gmail.com>
Zac Dover [Sun, 27 Nov 2022 23:45:25 +0000 (09:45 +1000)]
doc/glossary: s/Ceph System/Ceph Cluster/
Remove "Ceph System" and replace it with "Ceph Cluster".
Signed-off-by: Zac Dover <zac.dover@gmail.com>
zdover23 [Sun, 27 Nov 2022 23:35:17 +0000 (09:35 +1000)]
Merge pull request #49074 from zdover23/wip-doc-2022-11-27-dev-guide-basic-workflow-local-working-copy
doc/dev: improve Basic Workflow wording
Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
Zac Dover [Sun, 27 Nov 2022 08:20:51 +0000 (18:20 +1000)]
doc/dev: improve Basic Workflow wording
Make several semantic and syntactic improvements to basic-workflow.rst.
Signed-off-by: Zac Dover <zac.dover@gmail.com>
Ilya Dryomov [Sun, 27 Nov 2022 13:10:31 +0000 (14:10 +0100)]
Merge pull request #48680 from pkalever/snap-id
rbd: add --snap-id option to "rbd device map" to allow mapping arbitrary snapshots
Reviewed-by: Ilya Dryomov <idryomov@gmail.com>
zdover23 [Sun, 27 Nov 2022 04:06:18 +0000 (14:06 +1000)]
Merge pull request #49066 from zdover23/wip-doc-2022-11-26-glossary-ceph-system-remove
doc/glossary: remove "Ceph System"
Reviewed-by: Cole Mitchell <cole.mitchell@gmail.com>
Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
zdover23 [Sat, 26 Nov 2022 22:00:28 +0000 (08:00 +1000)]
Merge pull request #49065 from zdover23/wip-doc-2022-11-26-rados-ops-add-or-rm-osds-prompts
doc/rados: add prompts to add-or-remove-osds
Reviewed-by: Cole Mitchell <cole.mitchell@gmail.com>
Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
zdover23 [Sat, 26 Nov 2022 20:59:55 +0000 (06:59 +1000)]
Merge pull request #49045 from zdover23/wip-doc-2022-11-25-rbd-iscsi-initiator
doc/rbd: update iSCSI gateway info
Reviewed-by: Cole Mitchell <cole.mitchell@gmail.com>
Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
Reviewed-by: Ilya Dryomov <idryomov@redhat.com>
Zac Dover [Thu, 24 Nov 2022 18:08:13 +0000 (04:08 +1000)]
doc/rbd: update iSCSI gateway info
Update iSCSI gateway info in the RBD documentation.
https://tracker.ceph.com/issues/58080
Signed-off-by: Zac Dover <zac.dover@gmail.com>
Zac Dover [Sat, 26 Nov 2022 13:16:18 +0000 (23:16 +1000)]
doc/glossary: remove "Ceph System"
"Ceph System" appears only in definitions where "Ceph Cluster" should
be, so I'm removing the headword. A future commit will scrub the
glossary of the phrase "Ceph System".
Signed-off-by: Zac Dover <zac.dover@gmail.com>
Ilya Dryomov [Sat, 26 Nov 2022 12:27:33 +0000 (13:27 +0100)]
qa/workunits/rbd-nbd: clear DEV after detach tests
Otherwise we attempt to unmap it in cleanup(), needlessly.
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
Zac Dover [Sat, 26 Nov 2022 07:44:22 +0000 (17:44 +1000)]
doc/rados: add prompts to add-or-remove-osds
Add unselectable prompts to add-or-rm-osds.rst.
Signed-off-by: Zac Dover <zac.dover@gmail.com>
zdover23 [Sat, 26 Nov 2022 00:16:42 +0000 (10:16 +1000)]
Merge pull request #49062 from zdover23/wip-doc-2022-11-26-glossary-ceph-manager
doc/glossary: link to "Ceph Manager"
Reviewed-by: Cole Mitchell <cole.mitchell@gmail.com>
Zac Dover [Sat, 26 Nov 2022 00:06:34 +0000 (10:06 +1000)]
doc/glossary: link to "Ceph Manager"
Add link to "Ceph Manager" material.
Signed-off-by: Zac Dover <zac.dover@gmail.com>
Soumya Koduri [Fri, 25 Nov 2022 15:13:19 +0000 (20:43 +0530)]
Merge pull request #48761 from soumyakoduri/wip-skoduri-cloudtier-fixes
rgw/cloud-transition: Fix issues with MCG endpoint
Reviewed-by: Casey Bodley <cbodley@redhat.com>
Kefu Chai [Fri, 25 Nov 2022 09:07:29 +0000 (17:07 +0800)]
crimson/osd: correct the fmt::formatter<WatchTimeoutRequest>
it was introduced by
c9569f8eb027b9e08c8eeb59ecb37378ef66e5c0
Signed-off-by: Kefu Chai <tchaikov@gmail.com>
Kefu Chai [Fri, 25 Nov 2022 09:04:48 +0000 (17:04 +0800)]
crimson: print exceptions using e.what()
see also
e6223872c58e492d3570335fd7f05bb52641837c
Signed-off-by: Kefu Chai <tchaikov@gmail.com>
Kefu Chai [Fri, 25 Nov 2022 09:00:11 +0000 (17:00 +0800)]
crimson/os: specialize fmt::formatter<> for StagedIterator
StagedIterator is a nested class of stage_t, which is in turn a template
class. it would be impossible to partial specialize fmt::formatter<> for
a nested class of a template class. to workaround this, we specialize
fmt::formatter<> for a type which has `do_format_to()` method, and
define this method for StagedIterator.
since seastar::logger is the only user of the operator<<() operator,
the original operator<<() operator is replaced with this new helper
method.
Signed-off-by: Kefu Chai <tchaikov@gmail.com>
Ilya Dryomov [Fri, 25 Nov 2022 10:48:42 +0000 (11:48 +0100)]
Merge pull request #49036 from idryomov/wip-52932
mgr/rbd_support: avoid wedging the task queue if pool is removed
Reviewed-by: Mykola Golub <mgolub@suse.com>
Ilya Dryomov [Fri, 25 Nov 2022 10:47:38 +0000 (11:47 +0100)]
Merge pull request #49042 from idryomov/wip-rbd-support-flake8
mgr/rbd_support: clean up and enable flake8
Reviewed-by: Mykola Golub <mgolub@suse.com>
Kefu Chai [Fri, 25 Nov 2022 08:58:17 +0000 (16:58 +0800)]
crimson/os: include fmt/os.h for printing std::error_code
this is required by {fmt} v9 and up.
Signed-off-by: Kefu Chai <tchaikov@gmail.com>
Kefu Chai [Fri, 25 Nov 2022 08:57:19 +0000 (16:57 +0800)]
crimson: specialize fmt::formatter<> for formatting using {fmt}
so the specialized types can be formatted using {fmt} v9.
Signed-off-by: Kefu Chai <tchaikov@gmail.com>
zdover23 [Fri, 25 Nov 2022 07:12:05 +0000 (17:12 +1000)]
Merge pull request #49046 from zdover23/wip-doc-2022-11-25-rados-ops-erasure-code-profiles
doc/rados: cleanup "erasure code profiles"
Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
Zac Dover [Thu, 24 Nov 2022 18:41:12 +0000 (04:41 +1000)]
doc/rados: cleanup "erasure code profiles"
Improve the grammar of "Erasure Code Profiles".
Signed-off-by: Zac Dover <zac.dover@gmail.com>
Venky Shankar [Thu, 24 Nov 2022 17:25:17 +0000 (22:55 +0530)]
Merge pull request #48927 from neesingh-rh/fix_58028
cephfs-top: Fix unresposive sorting in some cases
Reviewed-by: Jos Collin <jcollin@redhat.com>
Ilya Dryomov [Thu, 24 Nov 2022 15:12:36 +0000 (16:12 +0100)]
Merge pull request #48542 from CongMinYin/fix-inconsistent-pwl-status
librbd/cache/pwl: fix clean vs bytes_dirty cache state inconsistency
Reviewed-by: Ilya Dryomov <idryomov@gmail.com>
Neeraj Pratap Singh [Tue, 22 Nov 2022 06:36:48 +0000 (12:06 +0530)]
cephfs-top: Fix unresposive sorting in some cases
Sorting is not working properly in case of change
in the number of clients or after removing and
creating the same filesystem again.
Fixes: https://tracker.ceph.com/issues/58028
Signed-off-by: Neeraj Pratap Singh <neesingh@redhat.com>
Venky Shankar [Thu, 24 Nov 2022 13:36:17 +0000 (19:06 +0530)]
Merge pull request #49026 from neesingh-rh/fix_58031
cephfs-top: sorting excepts when the filesystems are removed and created
Reviewed-by: Venky Shankar <vshankar@redhat.com>
Reviewed-by: Jos Collin <jcollin@redhat.com>
Ilya Dryomov [Thu, 24 Nov 2022 12:37:44 +0000 (13:37 +0100)]
mgr/rbd_support: clean up and enable flake8
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
Kefu Chai [Thu, 24 Nov 2022 12:34:47 +0000 (20:34 +0800)]
Merge pull request #49029 from Matan-B/wip-matanb-EIO-flag
crimson/osd: Support Pool EIO flag
Reviewed-by: Chunmei Liu <chunmei.liu@intel.com>
Reviewed-by: Kefu Chai <tchaikov@gmail.com>
Kefu Chai [Thu, 24 Nov 2022 09:12:51 +0000 (17:12 +0800)]
Merge pull request #48932 from aisakaki/wip-crimson-loop
crimson/common: re-implement do_for_each
Reviewed-by: Chunmei Liu <chunmei.liu@intel.com>
Reviewed-by: Xuehan Xu <xxhdx1985126@gmail.com>
Reviewed-by: Yingxin Cheng <yingxin.cheng@intel.com>
Reviewed-by: Kefu Chai <tchaikov@gmail.com>
Kefu Chai [Thu, 24 Nov 2022 08:29:26 +0000 (16:29 +0800)]
Merge pull request #49040 from tchaikov/wip-cephadm-flake8
cephadm: pin flake8 to 5.0.4
Reviewed-by: Redouane Kachach <rkachach@redhat.com>
Kefu Chai [Thu, 24 Nov 2022 06:01:13 +0000 (14:01 +0800)]
pybind/mgr: use annotation style without comment
* use annotation style without comment
* add noqa to import statement to prevent flake8 from complaining
those which import typing modules
to silence the warnings like:
```
flake8 run-test: commands[0] | flake8 --config=tox.ini alerts balancer cephadm cli_api crash devicehealth diskprediction_local hello iostat localpool nfs orchestrator prometheus selftest
cephadm/schedule.py:5:1: F401 'typing.Callable' imported but unused
cephadm/schedule.py:8:1: F401 'ceph.deployment.service_spec.ServiceSpec' imported but unused
cephadm/tests/fixtures.py:17:1: F401 'orchestrator.OrchResult' imported but unused
orchestrator/module.py:4:1: F401 'typing.Set' imported but unused
orchestrator/module.py:17:1: F401 'ceph.deployment.inventory.Device' imported but unused
prometheus/module.py:17:1: F401 'typing.DefaultDict' imported but unused
6 F401 'typing.Callable' imported but unused
ERROR: InvocationError for command /home/jenkins-build/build/workspace/ceph-pull-requests/src/pybind/mgr/.tox/flake8/bin/flake8 --config=tox.ini alerts
```
Signed-off-by: Kefu Chai <tchaikov@gmail.com>
Kefu Chai [Thu, 24 Nov 2022 05:46:27 +0000 (13:46 +0800)]
qa/tasks/cephfs: annotate variable without comment
to silence the warning like
```
flake8 run-test: commands[0] | flake8 --select=F,E9 --exclude=venv,.tox
./tasks/cephfs/test_full.py:6:5: F401 'typing.Optional' imported but unused
```
since we've migrated to python3.6 and up, there is no need to be
compatible with Python 2 anymore.
Signed-off-by: Kefu Chai <tchaikov@gmail.com>
Kefu Chai [Thu, 24 Nov 2022 05:18:10 +0000 (13:18 +0800)]
cephadm: pin flake8 to 5.0.4
to workaround https://github.com/zheller/flake8-quotes/issues/110
flake8 v5.0.4 is the last known-to-work flake8 before flake8 v6.0.0.
Fixes: https://tracker.ceph.com/issues/58069
Signed-off-by: Kefu Chai <tchaikov@gmail.com>
Neeraj Pratap Singh [Wed, 23 Nov 2022 12:01:33 +0000 (17:31 +0530)]
cephfs-top: sorting excepts when the filesystems are removed and created
Fixes: https://tracker.ceph.com/issues/58031
Signed-off-by: Neeraj Pratap Singh <neesingh@redhat.com>
Kefu Chai [Wed, 23 Nov 2022 23:45:45 +0000 (07:45 +0800)]
Merge pull request #49022 from cyx1231st/wip-seastar-msgr-fix-yielded-closing
crimson/net: fix use-after-free during yielded closing
Reviewed-by: Kefu Chai <tchaikov@gmail.com>
Ilya Dryomov [Wed, 23 Nov 2022 17:10:03 +0000 (18:10 +0100)]
mgr/rbd_support: avoid wedging the task queue if pool is removed
rados.ObjectNotFound exception handler was referencing ioctx variable
which is assigned only if the pool exists and rados.open_ioctx() call
succeeds. This lead to a fatal error
mgr[rbd_support] Failed to locate pool mypool
mgr[rbd_support] execute_task: [errno 2] error opening pool 'b'mypool''
mgr[rbd_support] Fatal runtime error: local variable 'ioctx' referenced before assignment
and wedged the task queue. No other commands were processed until
ceph-mgr daemon restart.
Fixes: https://tracker.ceph.com/issues/52932
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
zdover23 [Wed, 23 Nov 2022 19:52:30 +0000 (05:52 +1000)]
Merge pull request #49037 from zdover23/wip-doc-2022-11-24-container-install-consume-44310
doc/install: update "Official Releases" sources
Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
Zac Dover [Wed, 23 Nov 2022 19:26:48 +0000 (05:26 +1000)]
doc/install: update "Official Releases" sources
Remove "Docker Hub" from the list of locations from which Ceph can be
acquired.
This commit represents a second attempt to get
https://github.com/ceph/ceph/pull/49034 past the Jenkins CI tests, and
that PR was itself an attempt to incorporate the change in
https://github.com/ceph/ceph/pull/44310, which could not be merged
because the commit message associated with that PR did not begin with
the string "doc", which meant that the "Signed-off-by" check failed.
Signed-off-by: Zac Dover <zac.dover@gmail.com>
Matan Breizman [Wed, 23 Nov 2022 17:18:10 +0000 (17:18 +0000)]
test/librados/aio_cxx: use fmt formatting
Signed-off-by: Matan Breizman <mbreizma@redhat.com>
Matan Breizman [Wed, 23 Nov 2022 13:25:52 +0000 (13:25 +0000)]
crimson/osd/osd_operations/client_request: Support FLAG_EIO
See
5ac9f523ea263eeb747e2543a667a25de67d07b8
Signed-off-by: Matan Breizman <mbreizma@redhat.com>
zdover23 [Wed, 23 Nov 2022 15:16:20 +0000 (01:16 +1000)]
Merge pull request #49025 from zdover23/wip-doc-2022-11-23-glossary-ceph-client
doc/glossary: fix "Ceph Client"
Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
zdover23 [Wed, 23 Nov 2022 15:13:26 +0000 (01:13 +1000)]
Merge pull request #49027 from zdover23/wip-doc-2022-11-23-glossary-ceph-object-store
doc/glossary: add "Ceph Object Store"
Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
Kefu Chai [Wed, 23 Nov 2022 14:44:02 +0000 (22:44 +0800)]
Merge pull request #48859 from yanghonggang/yhg-code-cleanup
osd/ECBackend: remove unused variables
Reviewed-by: Omri Zeneva <ozeneva@redhat.com>
Reviewed-by: Kefu Chai <tchaikov@gmail.com>
Kefu Chai [Wed, 23 Nov 2022 14:11:38 +0000 (22:11 +0800)]
Merge pull request #49010 from tchaikov/more-than-future
crimson/common: use concept for type constraints
Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
Anthony D'Atri [Wed, 23 Nov 2022 13:31:58 +0000 (08:31 -0500)]
Merge pull request #49005 from CHOKYUJIN/main
doc/dev: fix a typo in deduplication.rst
Matan Breizman [Wed, 23 Nov 2022 13:22:59 +0000 (13:22 +0000)]
test/librados/aio_cxx: Simple PoolEIOFlag is added
Useful for debugging.
Signed-off-by: Matan Breizman <mbreizma@redhat.com>
Matan Breizman [Wed, 23 Nov 2022 13:20:01 +0000 (13:20 +0000)]
test/librados/aio_cxx: Fix PoolEIOFlag test
* wait_for_complete is added before getting the return value.
* for loop iteration limit is added to avoid infinite run.
Signed-off-by: Matan Breizman <mbreizma@redhat.com>
Zac Dover [Wed, 23 Nov 2022 13:09:47 +0000 (23:09 +1000)]
doc/glossary: add "Ceph Object Store"
Add a definition for "Ceph Object Store".
Signed-off-by: Zac Dover <zac.dover@gmail.com>
Kefu Chai [Tue, 22 Nov 2022 12:22:54 +0000 (20:22 +0800)]
crimson/common: use concept for type constraints
instead of using SFINAE, use concept for specifying the type constraints
for better readability.
Signed-off-by: Kefu Chai <tchaikov@gmail.com>
Yuval Lifshitz [Wed, 23 Nov 2022 11:03:20 +0000 (13:03 +0200)]
Merge pull request #48896 from yuvalif/wip-yuval-fix-50076
rgw/kafka: route librdkafka log messages to rgw log
Yuval Lifshitz [Wed, 23 Nov 2022 11:02:29 +0000 (13:02 +0200)]
Merge pull request #48877 from yuvalif/wip-yuval-fix-58020
rgw/notifications: send mtime in complete multipart upload event
Yuval Lifshitz [Wed, 23 Nov 2022 11:00:00 +0000 (13:00 +0200)]
Merge pull request #48758 from yuvalif/wip-yuval-kafka-tests
rgw/notification/test: add support for testing kafka security
Zac Dover [Wed, 23 Nov 2022 08:16:47 +0000 (18:16 +1000)]
doc/glossary: fix "Ceph Client"
Add detail to and clean the syntactics and semantics of "Ceph Client".
Signed-off-by: Zac Dover <zac.dover@gmail.com>
Kyujin Cho [Tue, 22 Nov 2022 10:49:06 +0000 (10:49 +0000)]
doc/dev: fix a typo in deduplication.rst
Signed-off-by: Kyujin Cho <bori19960@snu.ac.kr>
Yingxin Cheng [Wed, 23 Nov 2022 05:46:47 +0000 (13:46 +0800)]
crimson/net: fix use-after-free during yielded closing
Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
Xinyu Huang [Fri, 18 Nov 2022 02:43:42 +0000 (02:43 +0000)]
crimson/common: re-implement do_for_each
The current implementation of crimson::do_for_each might meet
stack overflow when future is available but seastar::need_preempt
is true. This new implementation mirror to the seastar::do_for_each
with crimson errorator mechanism will solve this problem.
Fixes: https://tracker.ceph.com/issues/58005.
Signed-off-by: Xinyu Huang <xinyu.huang@intel.com>
Kefu Chai [Wed, 23 Nov 2022 00:37:43 +0000 (08:37 +0800)]
Merge pull request #49011 from tchaikov/wip-crimson-fmt-v9
crimson: various fmt related changes and fixes.
Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
zdover23 [Tue, 22 Nov 2022 18:33:11 +0000 (04:33 +1000)]
Merge pull request #49017 from zdover23/wip-doc-2022-11-23-glossary-cluster-map
doc/glossary: s/comprising/consisting of/
Reviewed-by: Cole Mitchell <cole.mitchell@gmail.com>