Commit Graph

129659 Commits

Author SHA1 Message Date
Laura Flores
f2b75166a2
Merge pull request #43388 from e337/wip-doc-50499 2022-02-18 18:24:02 -06:00
Samuel Just
cee3cae9b2
Merge pull request #44933 from liu-chunmei/seastore-support-sparse-read
crimson/seastore: support sparse_read

Reviewed-by: Samuel Just <sjust@redhat.com>
2022-02-17 14:40:27 -08:00
Neha Ojha
e3463e286d
Merge pull request #44955 from aclamk/wip-vselector-check-fix
os/bluestore/bluefs: Fix improper vselector tracking in _flush_special()

Reviewed-by: Igor Fedotov <igor.fedotov@croit.io>
2022-02-17 10:26:10 -08:00
chunmei-liu
bc1e1116f5 crimson/seastore: put ObjectDataHandler instance into do_with
make ObjectDataHandler instance alive before sub functions are resolved.

Signed-off-by: chunmei-liu <chunmei.liu@intel.com>
2022-02-17 09:18:55 -08:00
chunmei-liu
f4e20eec8e crimson/seastore:: add fiemap and sparse read unit test
Signed-off-by: chunmei-liu <chunmei.liu@intel.com>
2022-02-17 09:18:49 -08:00
chunmei-liu
2baabdaf12 crimson/seastore: support sparse_read
add fiemap and readv functions.

Signed-off-by: chunmei-liu <chunmei.liu@intel.com>
2022-02-17 07:57:57 -08:00
zdover23
d11d2e72da
Merge pull request #45057 from zdover23/wip-doc-2022-02-17-os-recs-nautilus-and-luminous-table-removal
doc/start: remove L and N release-related tables

Reviewed-by: Josh Durgin <jdurgin@redhat.com>
2022-02-17 20:19:35 +10:00
Samuel Just
74b7fe8264
Merge pull request #45049 from liu-chunmei/crimson-clang-warning-cleanup
crimson: eliminate clang build warning in crimson

Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
Reviewed-by: Ronen Friedman <rfriedma@redhat.com>
2022-02-16 12:07:56 -08:00
chunmei-liu
3d7d6dbd05 crimson: eliminate clang build warning in crimson
Signed-off-by: chunmei-liu <chunmei.liu@intel.com>
2022-02-16 08:20:35 -08:00
Zac Dover
cc6d3871ec doc/start: remove L and N release-related tables
This PR removes the tables for Luminous and Nautilus,
in order to keep the documentation as current as possible.
This is one a series of PRs intended to update the OS
Recommendations page. The scope of this PR is confined to
the removal of these two tables. Other PRs will update the
information elsewhere on the OS Recommendations page.

Signed-off-by: Zac Dover <zac.dover@gmail.com>
2022-02-17 01:21:44 +10:00
Ilya Dryomov
88eb23e585
Merge pull request #45045 from idryomov/wip-qemu-task-rbd-package
qa/tasks/qemu: make sure block-rbd.so is installed

Reviewed-by: Deepika Upadhyay <dupadhya@redhat.com>
2022-02-16 14:15:24 +01:00
Ilya Dryomov
8f0fd0af3d qa/suites/rbd: make sure block-rbd.so is installed
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
2022-02-16 12:20:44 +01:00
Ilya Dryomov
e1e9ee61af
Merge pull request #45021 from idryomov/wip-groupsnapns-operator-less
cls/rbd: GroupSnapshotNamespace comparator violates ordering rules

Reviewed-by: Mykola Golub <mgolub@suse.com>
Reviewed-by: Sunny Kumar <sunkumar@redhat.com>
2022-02-16 11:32:10 +01:00
Avan
fd5f5f3aa5
Merge pull request #45020 from rhcs-dashboard/snmp-priv-protocol-fix
mgr/dashboard: change privacy protocol field from required to optional
2022-02-16 15:22:26 +05:30
Igor Fedotov
2e2513decc
Merge pull request #44659 from ifed01/wip-ifed-alloc-replay-boind-size
test/alloc_replay_test: be able to cap allocator space with capacity …

Reviewed-by: Adam Kupczyk <akupczyk@redhat.com>
2022-02-16 12:50:08 +03:00
Adam Kupczyk
3813416e6a os/bluestore/bluefs: Fix vselector
Fix bluefs volume selector in device_migrate_to_existing.
Fix bluefs volume selector in _rewrite_log_and_layout_sync_LNF_LD.

Fixes: https://tracker.ceph.com/issues/54248

Signed-off-by: Adam Kupczyk <akupczyk@redhat.com>
2022-02-15 23:31:53 +01:00
Adam King
9d0d7a489e
Merge pull request #44997 from pcfe/pcfe-add-cos9-instructions
doc/cephadm: Add CentOS Stream install instructions

Reviewed-by: Adam King <adking@redhat.com>
2022-02-15 14:09:31 -05:00
Patrick C. F. Ernzer
7f243262f9 doc/cephadm: Add CentOS Stream install instructions
Signed-off-by: Patrick C. F. Ernzer <pcfe@pcfe.net>.
2022-02-15 17:28:07 +01:00
Casey Bodley
29c9d21d26
Merge pull request #44988 from caisan/FixRGWLogStringError
Fix rgw data sync log info error

Reviewed-by: Yehuda Sadeh <ysadehwe@redhat.com>
2022-02-15 09:29:12 -05:00
Avan Thakkar
2866db1eac mgr/dashboard: add validation for snmp v3 engine id
Fixes: https://tracker.ceph.com/issues/54270
Signed-off-by: Avan Thakkar <athakkar@redhat.com>
2022-02-15 19:42:57 +05:30
Ilya Dryomov
525ff61cfc qa/tasks/qemu: make sure block-rbd.so is installed
Fixes: https://tracker.ceph.com/issues/54286
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
2022-02-15 14:57:51 +01:00
David Galloway
bcb67715df
Merge pull request #45032 from ceph/revert-45018
Revert "admin/doc-requirements: pin breathe to 4.32.0"
2022-02-15 08:55:31 -05:00
Ernesto Puerta
c28fb3165b
Merge pull request #44951 from votdev/issue_54215_expand_cluster
mgr/dashboard: "Please expand your cluster first" shouldn't be shown if cluster is already meaningfully running

Reviewed-by: Ernesto Puerta <epuertat@redhat.com>
Reviewed-by: Nizamudeen A <nia@redhat.com>
Reviewed-by: Tatjana Dehler <tdehler@suse.com>
Reviewed-by: Volker Theile <vtheile@suse.com>
2022-02-15 11:58:49 +01:00
Venky Shankar
1a762893e3
Merge pull request #43588 from lxbsz/fsize_support
mds: add truncate size handling support for fscrypt

Reviewed-by: Venky Shankar <vshankar@redhat.com>
2022-02-15 14:48:34 +05:30
Ilya Dryomov
a3e632a322
Merge pull request #45011 from tchaikov/librados/cleanup
include/rados/librados.h: drop duplicated function declaration

Reviewed-by: Ilya Dryomov <idryomov@gmail.com>
2022-02-15 00:16:14 +01:00
David Galloway
df2d7b29ec admin/doc-requirements.txt: Not breathe=4.33
See https://github.com/michaeljones/breathe/issues/803

Signed-off-by: David Galloway <dgallowa@redhat.com>
2022-02-14 15:11:58 -05:00
David Galloway
f459ada4be Revert "admin/doc-requirements: pin breathe to 4.32.0"
This reverts commit d020d07be9.

Fixed upstream https://github.com/michaeljones/breathe/issues/803#issuecomment-1039341857

Signed-off-by: David Galloway <dgalloway@redhat.com>
2022-02-14 14:38:26 -05:00
Adam King
2f2061810a
Merge pull request #44779 from rkachach/fix_issue_53010
mgr/cephadm: Adding logic to cleanup several ceph dirs (rm-cluster)

Reviewed-by: Adam King <adking@redhat.com>
2022-02-14 12:35:15 -05:00
Adam King
f98f4de03c
Merge pull request #45014 from guits/fix-cephadm-ingress
cephadm/ingress: make frontend stats bind on localhost

Reviewed-by: Adam King <adking@redhat.com>
2022-02-14 12:29:36 -05:00
Yuri Weinstein
f6fff9ba6c
Merge pull request #44794 from rzarzynski/wip-common-fix-numa-warning
common: fix compilation warnings in numa.cc

Reviewed-by: Ronen Friedman <rfriedma@redhat.com>
2022-02-14 09:06:28 -08:00
Yuri Weinstein
ddeec8d88a
Merge pull request #44781 from ljflores/wip-basic-channel-additions
mgr/telemetry: add `basic_pool_usage` and `basic_usage_by_class` collections to the telemetry module

Reviewed-by: Yaarit Hatuka <yaarit@redhat.com>
2022-02-14 09:06:00 -08:00
Yuri Weinstein
d81cc0cd76
Merge pull request #44746 from ljflores/wip-telemetry-down-osd-fix
mgr/telemetry: collect what we can from histograms, mempools, and heap stats

Reviewed-by: Yaarit Hatuka <yaarit@redhat.com>
2022-02-14 09:04:33 -08:00
Yuri Weinstein
adb2e71aec
Merge pull request #44670 from rzarzynski/wip-common-dout-gcc12-ftbfs
common: fix FTBFS due to dout & need_dynamic on GCC-12

Reviewed-by: Kefu Chai <kchai@redhat.com>
2022-02-14 09:03:51 -08:00
Yuri Weinstein
0de42e61e4
Merge pull request #44983 from cbodley/wip-52673
rgw: remove rgw_rados_pool_pg_num_min and its use on pool creation

Reviewed-by: Neha Ojha <nojha@redhat.com>
2022-02-14 09:02:51 -08:00
Adam Kupczyk
4bc0f61d23 os/bluestore/bluefs: Fix improper vselector tracking in _flush_special()
Moves vselector size tracking outside _flush_special().
Function _compact_log_async...() updated sizes twice.
Problem could not be solved by making second modification of size just update,
as it will possibly disrupt vselector consistency check (_vselector_check()).
Feature to track vselector consistency relies on the fact that either log.lock or nodes.lock
are taken when the check is performed. Which is not true for _compact_log_async...().

Now _flush_special does not update vselector sizes by itself but leaves the update to
the caller.

Fixes: https://tracker.ceph.com/issues/54248

Signed-off-by: Adam Kupczyk <akupczyk@redhat.com>
2022-02-14 16:59:03 +01:00
Adam King
c1b48ce400
Merge pull request #44917 from guits/fix_rook_smoke_nvme_loop
qa/orch: use hwe kernel in rook/smoke tests (ubuntu 20.04)

Reviewed-by: Neha Ojha nojha@redhat.com
Reviewed-by: Yuri Weinstein yweinste@redhat.com
2022-02-14 10:55:36 -05:00
David Galloway
cd63cb6dd4
Merge pull request #45018 from tchaikov/wip-doc-breathe
admin/doc-requirements: pin breathe to 4.32.0
2022-02-14 10:44:34 -05:00
Avan Thakkar
2d8f2b8195 mgr/dashboard: change privacy protocol field from required to optional
Fixes: https://tracker.ceph.com/issues/54270
Signed-off-by: Avan Thakkar <athakkar@redhat.com>

Privacy protocol field shouldn't be a required field.
2022-02-14 20:01:27 +05:30
Guillaume Abrioux
ff482da6cb cephadm/ingress: make frontend stat bind on localhost
The current configuration of keepalived makes it do
a curl on localhost:9999 in order to check the endpoint is alive.
Given the endpoint only binds on the vip addr, that doesn't work.

Fixes: https://tracker.ceph.com/issues/53807

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
2022-02-14 15:27:09 +01:00
Adam King
884dc76683
Merge pull request #44965 from adk3798/test_cli_timeout
qa/tasks/cephadm_cases: increase timeouts in test_cli.py

Reviewed-by: Michael Fritch mfritch@suse.com
2022-02-14 08:24:32 -05:00
Ilya Dryomov
830e72ab9d cls/rbd: GroupSnapshotNamespace comparator violates ordering rules
For

  GroupSnapshotNamespace a(1, "group-1", "snap-2");
  GroupSnapshotNamespace b(1, "group-2", "snap-1");

both a < b and b < a evaluate to true.  This violates STL strict weak
ordering requirements which is a problem because GroupSnapshotNamespace
is used as a key in std::map (ictx->snap_ids at least), etc.

Fixes: https://tracker.ceph.com/issues/49792
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
2022-02-14 13:31:54 +01:00
Kefu Chai
d020d07be9 admin/doc-requirements: pin breathe to 4.32.0
in breathe v4.33, it includes following commit

2498a43723

which specfies the app config value of "graphviz_dot". this annoys
sphinx:

WARNING: while setting up extension breathe: node class 'graphviz' is already registered, its visitors will be overridden

Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/ceph/envs/44951/lib/python3.8/site-packages/sphinx/cmd/build.py", line 276, in build_main
    app = Sphinx(args.sourcedir, args.confdir, args.outputdir,
  File "/home/docs/checkouts/readthedocs.org/user_builds/ceph/envs/44951/lib/python3.8/site-packages/sphinx/application.py", line 245, in __init__
    self.setup_extension(extension)
  File "/home/docs/checkouts/readthedocs.org/user_builds/ceph/envs/44951/lib/python3.8/site-packages/sphinx/application.py", line 402, in setup_extension
    self.registry.load_extension(self, extname)
  File "/home/docs/checkouts/readthedocs.org/user_builds/ceph/envs/44951/lib/python3.8/site-packages/sphinx/registry.py", line 430, in load_extension
    metadata = setup(app)
  File "/home/docs/checkouts/readthedocs.org/user_builds/ceph/envs/44951/lib/python3.8/site-packages/breathe/__init__.py", line 14, in setup
    renderer_setup(app)
  File "/home/docs/checkouts/readthedocs.org/user_builds/ceph/envs/44951/lib/python3.8/site-packages/breathe/renderer/sphinxrenderer.py", line 2613, in setup
    app.add_config_value("graphviz_dot", "dot", "html")
  File "/home/docs/checkouts/readthedocs.org/user_builds/ceph/envs/44951/lib/python3.8/site-packages/sphinx/application.py", line 535, in add_config_value
    self.config.add(name, default, rebuild, types)
  File "/home/docs/checkouts/readthedocs.org/user_builds/ceph/envs/44951/lib/python3.8/site-packages/sphinx/config.py", line 282, in add
    raise ExtensionError(__('Config value %r already present') % name)
sphinx.errors.ExtensionError: Config value 'graphviz_dot' already present

Extension error:
Config value 'graphviz_dot' already present

this issue has been reported to upstream, see
https://github.com/michaeljones/breathe/issues/803

before it is fixed upstream, let's stick with 4.32.0
which is known to work.

Signed-off-by: Kefu Chai <tchaikov@gmail.com>
2022-02-14 18:31:39 +08:00
Volker Theile
48fff60b63 mgr/dashboard: "Please expand your cluster first" shouldn't be shown if cluster is already meaningfully running
This PR will assume that a cluster is already up and fully running. If this should not be the expected behaviour, deployment tools have to set 'INSTALLED' explicitly. Without this assumption it might happen that upgraded and fully running clusters, e.g. Octopus -> Pacific, will show the 'Expand Cluster' on first log in.

cephadm will take care that the bootstrap phase will write the necessary key to show the 'Expand cluster' page.

Fixes: https://tracker.ceph.com/issues/54215

Signed-off-by: Volker Theile <vtheile@suse.com>
2022-02-14 10:49:38 +01:00
Venky Shankar
2de993b969
Merge pull request #44971 from joscollin/wip-doc-fix-name-id
doc: mention MDS name and ID format in ceph-mds.rst

Reviewed-by: Venky Shankar <vshankar@redhat.com>
2022-02-14 12:19:43 +05:30
Venky Shankar
43b3a7c845
Merge pull request #39315 from shenhang/fetchkey
mds: fetch single dentry instead of complete dirfrag for lookup

Reviewed-by: Venky Shankar <vshankar@redhat.com>
2022-02-14 09:53:10 +05:30
Kefu Chai
3a1ef3fe36 include/rados/librados.h: drop duplicated function declaration
rados_mgr_command() has been declared in the very same header file
with exactly the same function signature, so there is no need to
repeat it.

in this change, the duplicated declaration of rados_mgr_command()
is dropped. the one with doxygen document is reserved.

Signed-off-by: Kefu Chai <tchaikov@gmail.com>
2022-02-14 12:19:51 +08:00
Kotresh HR
481093b7f6
Merge pull request #44308 from vshankar/wip-document-mds-locker
doc: document mds locking operation
2022-02-13 08:41:10 +05:30
Anthony D'Atri
e5b9173836
Merge pull request #44898 from taggelos/patch-1
doc/man/8: Fix spelling in user rename example
2022-02-12 02:52:16 -08:00
Yuri Weinstein
1ead2229ae
Merge pull request #44780 from sosodev/dump-blocked-ops-count
mds, osd: add dump_blocked_ops_count command

Reviewed-by: Neha Ojha <nojha@redhat.com>
2022-02-11 14:51:00 -08:00
Yuri Weinstein
66e651ce3c
Merge pull request #44752 from NitzanMordhai/wip-nitzan-osd-status
pybind/mgr: ceph osd status crash with ZeroDivisionError

Reviewed-by: Josh Durgin <jdurgin@redhat.com>
Reviewed-by: Neha Ojha <nojha@redhat.com>
Reviewed-by: Ronen Friedman <rfriedma@redhat.com>
2022-02-11 14:50:31 -08:00