Commit Graph

136957 Commits

Author SHA1 Message Date
Venky Shankar
26cbd243e4 Merge PR #50177 into main
* refs/pull/50177/head:
	cephfs-top: navigate to home screen when no fs

Reviewed-by: Neeraj Pratap Singh <neesingh@redhat.com>
Reviewed-by: Rishabh Dave <ridave@redhat.com>
2023-03-09 17:51:17 +05:30
Nizamudeen A
dcbf81c6cc
Merge pull request #49953 from rhcs-dashboard/dashboard-rgw-multisite
mgr/dashboard: Dashboard RGW multisite configuration

Reviewed-by: aashish1412 <NOT@FOUND>
Reviewed-by: Avan Thakkar <athakkar@redhat.com>
Reviewed-by: Ernesto Puerta <epuertat@redhat.com>
Reviewed-by: Nizamudeen A <nia@redhat.com>
Reviewed-by: Pere Diaz Bou <pdiazbou@redhat.com>
2023-03-09 17:13:45 +05:30
Aashish Sharma
3d6e9ac22b mgr/dashboard: Dashboard RGW multisite configuration
Signed-off-by: Aashish Sharma <aasharma@redhat.com>
2023-03-09 14:48:56 +05:30
zdover23
2e8c20cca2
Merge pull request #50439 from anthonyeleven/anthonyeleven/caps
doc/rados/operations: Improve wording, capitalization, formatting

Reviewed-by: Zac Dover <zac.dover@proton.me>
2023-03-09 16:01:12 +10:00
Anthony D'Atri
5f2060c082 doc/rados/operations: Improve wording, capitalizatiopn, formatting
Signed-off-by: Anthony D'Atri <anthonyeleven@users.noreply.github.com>
2023-03-08 18:53:51 -05:00
Guillaume Abrioux
44e4dfd0d8
Merge pull request #50149 from fmount/fs_cleanup
Remove the filestore section from ceph-volume
2023-03-08 17:01:10 +01:00
Adam King
d3bdd7b17c
Merge pull request #50082 from adk3798/move-repos
mgr/cephadm: make upgrade respect use_repo_digest

Reviewed-by: Redouane Kachach <rkachach@redhat.com>
2023-03-08 10:54:01 -05:00
Adam King
88ef4e2aa1
Merge pull request #50318 from Svelar/extra_conf
cephadm: eliminate duplication of sections

Reviewed-by: Adam King <adking@redhat.com>
2023-03-08 10:52:39 -05:00
Adam King
9b37c8c88e
Merge pull request #50413 from adk3798/prometheus-cephadm-dep
mgr/prometheus: remove dependency on cephadm module

Reviewed-by: Redouane Kachach <rkachach@redhat.com>
2023-03-08 10:49:37 -05:00
Ronen Friedman
9c52fbd7ac
Merge pull request #50402 from ronen-fr/wip-rf-operator-log
osd: ECTransactions.cc: remove __func__ used inside a lambda

Reviewed-by: Samuel Just <sjust@redhat.com>
2023-03-08 14:52:55 +02:00
Anthony D'Atri
f021748e45
Merge pull request #50437 from Svelar/rados-doc
doc/rados: fix grammar mistake
2023-03-08 06:45:08 -05:00
Rongqi Sun
9d107ac3bb doc/rados: fix grammar mistake
Signed-off-by: Rongqi Sun <sunrongqi@huawei.com>
2023-03-08 15:33:59 +08:00
Venky Shankar
2e2e7b9186 Merge PR #49008 into main
* refs/pull/49008/head:
	doc/quota: accept human readable quota value documented
	qa/workunits/fs/quota.sh: use human readable format for ceph.quota.max_bytes
	qa: added three testcases
	mds: accept human readable values for quotas

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
Reviewed-by: Milind Changire <mchangir@redhat.com>
Reviewed-by: Kotresh Hiremath Ravishankar <khiremat@redhat.com>
Reviewed-by: Rishabh Dave <ridave@redhat.com>
Reviewed-by: Venky Shankar <vshankar@redhat.com>
Reviewed-by: Anthony D Atri <anthony.datri@gmail.com>
2023-03-08 12:36:21 +05:30
zdover23
3fa3cd08c5
Merge pull request #50430 from zdover23/wip-doc-2023-03-08-install-minor-grammar-fix
doc/install: update index.rst

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
2023-03-08 13:31:04 +10:00
Venky Shankar
9623dd2539 Merge PR #46357 into main
* refs/pull/46357/head:
	PendingReleaseNotes: add reference to the new mdsmap max_xattr_size field
	qa/tasks/mgr/dashboard/test_health: Add 'max_xattr_size' to the mdsmap schema
	mds: prevent exceeding xattrs limits on initial set
	mds: prevent clients from exceeding the xattrs key/value limits
	mon: add new configuration to limit filesystem xattrs size

Reviewed-by: Xiubo Li <xiubli@redhat.com>
Reviewed-by: Venky Shankar <vshankar@redhat.com>
2023-03-08 08:12:02 +05:30
Venky Shankar
c3ffbd5ecd Merge PR #49766 into main
* refs/pull/49766/head:
	mds: record and dump last tid for trimming completed requests (or flushes)

Reviewed-by: Rishabh Dave <ridave@redhat.com>
Reviewed-by: Kotresh Hiremath Ravishankar <khiremat@redhat.com>
Reviewed-by: Milind Changire <mchangir@redhat.com>
Reviewed-by: Dhairya Parmar <dparmar@redhat.com>
2023-03-08 08:09:36 +05:30
Venky Shankar
d78b2305c3 Merge PR #49844 into main
* refs/pull/49844/head:
	qa: enforce string client-id check for testing oldest client tid warning
	qa: optionally check pattern string match in `health detail`

Reviewed-by: Rishabh Dave <ridave@redhat.com>
Reviewed-by: Kotresh Hiremath Ravishankar <khiremat@redhat.com>
2023-03-08 08:08:17 +05:30
Venky Shankar
8f55abb330 Merge PR #49912 into main
* refs/pull/49912/head:
	qa: avoid explicit set to client mountpoint as "/"

Reviewed-by: Xiubo Li <xiubli@redhat.com>
Reviewed-by: Rishabh Dave <ridave@redhat.com>
2023-03-08 08:07:04 +05:30
Rongqi Sun
b3125f4449 test_cephadm: fix conf format
Signed-off-by: Rongqi Sun <sunrongqi@huawei.com>
2023-03-08 10:16:55 +08:00
Zac Dover
82d748ff59 doc/install: update index.rst
Update index.rst by making minor grammar improvements. This file was
long overdue for a backport to Reef, Quincy, and Pacific, so this commit
was a good way to pass a human eyeball over the text before making those
backports.

Signed-off-by: Zac Dover <zac.dover@proton.me>
2023-03-08 11:52:12 +10:00
Casey Bodley
e9720ddbe9 rgw/admin: 'bucket stats' displays non-empty mtime
Fixes: https://tracker.ceph.com/issues/58932

Signed-off-by: Casey Bodley <cbodley@redhat.com>
2023-03-07 18:14:00 -05:00
Casey Bodley
20ab42758c rgw/sysobj: read() can query size/mtime
rgw_get_system_obj() stopped calling stat() before read() in
90aec61298, which left the optional
'real_time *pmtime' argument uninitialized

when requested, read() will add a stat op to initialize size/mtime

Signed-off-by: Casey Bodley <cbodley@redhat.com>
2023-03-07 18:14:00 -05:00
Adam King
8cb83413a6
Merge pull request #50146 from adk3798/container-selinux-conflict
qa/distros: pass --allowerasing --nobest when installing container-tools

Reviewed-by: John Mulligan <jmulligan@redhat.com>
Reviewed-by: Redouane Kachach <rkachach@redhat.com>
2023-03-07 17:06:46 -05:00
Adam King
fc85d0f262
Merge pull request #50133 from adk3798/fqdn-stray
mgr/cephadm: be aware of host's shortname and FQDN

Reviewed-by: John Mulligan <jmulligan@redhat.com>
Reviewed-by: Redouane Kachach <rkachach@redhat.com>
2023-03-07 17:05:22 -05:00
Adam King
59965ec6b2
Merge pull request #50244 from phlogistonjohn/jjm-cephadm-test-call
cephadm: test call function & fix timeout argument

Reviewed-by: Adam King <adking@redhat.com>
2023-03-07 16:34:20 -05:00
Adam King
b450283a01
Merge pull request #50101 from adk3798/all-devices-raw
mgr/orchestrator: allow deploying raw mode OSDs with --all-available-devices

Reviewed-by: Guillaume Abrioux <gabrioux@ibm.com>
2023-03-07 14:50:29 -05:00
Casey Bodley
b7ee62d781
Merge pull request #45469 from zaitcev/srbac-bz1901857-3
RGW: Add a reader feature

Reviewed-by: Casey Bodley <cbodley@redhat.com>
2023-03-07 14:38:12 -05:00
John Mulligan
05304675f7 cephadm/tests: enable timeout test cases for call function
Signed-off-by: John Mulligan <jmulligan@redhat.com>
2023-03-07 13:56:42 -05:00
John Mulligan
cefe44cde8 cephadm: fix timeout argument to call function
The timeout argument to call function, for executing sub-processes, did
not function - this patch makes timeout work as (probably) intended.
Use the `process.communicate()` method rather than `tee` functions to
handle IO collection. Since no logging is done until after the exit code
is known the tee calls are not necessary. Add calls to kill the child
process when the time out occurs. This helps prevent event loop "leaks"
that generate python warnings.

Signed-off-by: John Mulligan <jmulligan@redhat.com>
2023-03-07 13:56:42 -05:00
Adam King
e597e8785b
Merge pull request #50270 from mnasiadka/cephadm_ulimit_docker
cephadm: set --ulimit nofiles with Docker

Reviewed-by: Adam King <adking@redhat.com>
2023-03-07 13:56:02 -05:00
Adam King
6443cf15d5 mgr/cephadm: be aware of host's shortname and FQDN
The idea is to gether the shortname and FQDN as part
of gather-facts, and then if we ever try to check if a certain
host is in our internal inventory by hostname, we can check
these other known names. This should avoid issues where
we think a hostname specified by FQDN is not in our
inventory because we know the host by the shortname
or vice versa.

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

Signed-off-by: Adam King <adking@redhat.com>
2023-03-07 13:50:47 -05:00
Casey Bodley
83bac8d01e cmake/boost: WITH_ASAN adds BOOST_USE_ASAN to context
also adds 'context-impl=ucontext' to the b2 command line and the
BOOST_USE_UCONTEXT define as described in:
https://www.boost.org/doc/libs/1_79_0/libs/context/doc/html/context/stack/sanitizers.html

Signed-off-by: Casey Bodley <cbodley@redhat.com>
2023-03-07 13:33:57 -05:00
Casey Bodley
8c099a5340 asan: add qa/lsan.supp for leak sanitizer suppressions
any executable that links against the heap_profiler leaks 8 bytes for a
singleton:
> -----------------------------------------------------
> Suppressions used:
> count      bytes template
> 1          8 ^MallocExtension::Register
> -----------------------------------------------------

the python script bin/ceph has a lot of leaks:
> -----------------------------------------------------
> Suppressions used:
>   count      bytes template
>     205     294983 ^PyObject_Malloc
>      10       6216 ^_PyObject_Realloc
>       1        568 ^PyMem_Calloc
>       2          2 ^PyMem_Malloc
>      63      58935 ^PyUnicode_New
>      10       7294 ^_PyBytes_FromSize
>     166     149904 ^_PyObject_GC_Alloc
>       5       3360 ^_PyObject_GC_Resize
> -----------------------------------------------------

Signed-off-by: Casey Bodley <cbodley@redhat.com>
2023-03-07 13:33:57 -05:00
Casey Bodley
3753469d1b cmake: update FindSanitizers for new libasan versions
Signed-off-by: Casey Bodley <cbodley@redhat.com>
2023-03-07 13:33:57 -05:00
Adam King
39d3ce94a0
Merge pull request #50100 from adk3798/set-unmanaged
mgr/cephadm: add commands to set services to managed/unmanaged

Reviewed-by: Anthony D'Atri <anthonyeleven@users.noreply.github.com>
Reviewed-by: John Mulligan <jmulligan@redhat.com>
Reviewed-by: Redouane Kachach <rkachach@redhat.com>
2023-03-07 13:25:17 -05:00
Adam King
a42a673056
Merge pull request #50167 from trociny/wip-58792
mgr/cephadm: don't add mgr into iscsi trusted_ip_list if it's already there

Reviewed-by: Adam King <adking@redhat.com>
2023-03-07 13:22:53 -05:00
Adam King
251c3d44c0
Merge pull request #50275 from rkachach/fix_issue_58856
mgr/rgw: adding mgr rgw module to ceph image

Reviewed-by: Adam King <adking@redhat.com>
Reviewed-by: Ernesto Puerta <epuertat@redhat.com>
2023-03-07 13:21:15 -05:00
Yuval Lifshitz
4aa930e9f9
Merge pull request #48911 from yuvalif/test-kafka-sasl-scram
rgw: test kafka sasl scram

reviewed-by: anthonyeleven, cbodley
2023-03-07 20:20:28 +02:00
Adam King
3019cf00f4
Merge pull request #50311 from trociny/wip-58051-1
mgr/cephadm: try to avoid pull when getting container image info

Reviewed-by: Adam King <adking@redhat.com>
2023-03-07 13:18:41 -05:00
Yuval Lifshitz
430edd3fc6
Merge pull request #50403 from yuvalif/wip-yuval-dont-start-notif-thread
rgw: don't start the bucket notification thread in radosgw-admin

reviewewd-by: cbodley
2023-03-07 19:04:19 +02:00
Matan
5841654405
Merge pull request #49116 from Matan-B/wip-matanb-c-balanced-reads
crimson/osd: Support balance reads

Reviewed-by: Samuel Just <sjust@redhat.com>
2023-03-07 18:58:58 +02:00
Laura Flores
f96dd0dda6
Merge pull request #50410 from ljflores/wip-tracker-58925
qa: suppress memory leak in rocksdb
2023-03-07 09:56:38 -06:00
Matan
274adef5ee
Merge pull request #49779 from Matan-B/wip-matanb-c-lru-comments
common/intrusive_lru: Improve readability

Reviewed-by: Samuel Just <sjust@redhat.com>
Reviewed-by: chunmei-liu <chunmei.liu@intel.com>
Reviewed-by: Radosław Zarzyński <rzarzyns@redhat.com>
2023-03-07 17:10:03 +02:00
Yuval Lifshitz
a94f5b36b1 rgw: unify data member initialization in RGWRados
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#c48-prefer-in-class-initializers-to-member-initializers-in-constructors-for-constant-initializers

Signed-off-by: Yuval Lifshitz <ylifshit@redhat.com>
2023-03-07 16:40:28 +02:00
Yuval Lifshitz
ff1f68e4a4 rgw: don't start the bucket notification thread in radosgw-admin
Signed-off-by: Yuval Lifshitz <ylifshit@redhat.com>
2023-03-07 14:18:17 +02:00
Rongqi Sun
0d1596e927 cephadm: eliminate duplication of sections
Signed-off-by: Rongqi Sun <sunrongqi@huawei.com>
2023-03-07 17:12:31 +08:00
Matan Breizman
b1632dbf17 crimson/osd/object_context_loader: Hide private methods
Signed-off-by: Matan Breizman <mbreizma@redhat.com>
2023-03-07 09:04:07 +00:00
Matan Breizman
3ea1d8e09d crimson/osd: Introduce with_head_and_clone_obc()
In continuation to 7ca2690be956a36f61c7729946b94ccd970dd9c7:
Now that the head ref is no longer a member of obc, we need a new
substitute way to get the head when needed.

When loading a clone object, the head object is loaded
first (See with_clone_obc). Therefore we can make use of this design
to move the loaded head forward to the relevant func (See with_head_and_clone_obc).
Usually, we wouldn't need to make use of both the head and the clone obc in the
same function. However, SnapTrimObjSubEvent::remove_or_update is an abnormal usage.

Note: We want to avoid holding any unneeded references to obcs
to allow the obc_registery to evict no longer valid obc.
Therefore, with_obc() which references only a single obc is the
preferred entry point for loading obcs.

Signed-off-by: Matan Breizman <mbreizma@redhat.com>
2023-03-07 09:04:07 +00:00
Matan Breizman
4bf99c69f2 crimson/osd/object_context_loader: with_clone_obc to move head
Signed-off-by: Matan Breizman <mbreizma@redhat.com>
2023-03-07 09:04:07 +00:00
Matan Breizman
efcf292905 crimson/osd/object_conetxt*: Rename get_ro_ss to get_head_ss
Signed-off-by: Matan Breizman <mbreizma@redhat.com>
2023-03-07 09:04:07 +00:00