]> git.apps.os.sepia.ceph.com Git - ceph.git/log
ceph.git
21 months agodoc/start: add Slack to get-involved 53971/head
Zac Dover [Wed, 11 Oct 2023 16:29:12 +0000 (02:29 +1000)]
doc/start: add Slack to get-involved

Add a link to the Ceph Slack to doc/start/get-involved.rst.

Signed-off-by: Zac Dover <zac.dover@proton.me>
21 months agoMerge pull request #53846 from cbodley/wip-62989-again
Casey Bodley [Wed, 11 Oct 2023 15:59:20 +0000 (16:59 +0100)]
Merge pull request #53846 from cbodley/wip-62989-again

rgw: fix http error checks in keystone/barbican/vault clients

Reviewed-by: Daniel Gryniewicz <dang@redhat.com>
21 months agoMerge pull request #53579 from Matan-B/wip-matanb-reset_purged_snaps_last
Radoslaw Zarzynski [Wed, 11 Oct 2023 15:01:44 +0000 (17:01 +0200)]
Merge pull request #53579 from Matan-B/wip-matanb-reset_purged_snaps_last

osd/OSD: introduce reset_purged_snaps_last

Reviewed-by: Samuel Just <sjust@redhat.com>
Reviewed-by: Ilya Dryomov <idryomov@gmail.com>
Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
21 months agoMerge pull request #53908 from zdover23/wip-doc-2023-10-10-troubleshooting-troublesho...
zdover23 [Wed, 11 Oct 2023 00:39:53 +0000 (10:39 +1000)]
Merge pull request #53908 from zdover23/wip-doc-2023-10-10-troubleshooting-troubleshooting-memory-profiling

doc/rados: edit memory-profiling.rst

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
21 months agoMerge pull request #53930 from ceph/wip-yuri-crontab-main
Yuri Weinstein [Tue, 10 Oct 2023 23:01:19 +0000 (16:01 -0700)]
Merge pull request #53930 from ceph/wip-yuri-crontab-main

qa/tests: Added upgrade/reef-x to the mix

Reviewed-by: Laura Flores <lflores@redhat.com>
21 months agoqa/tests: Added upgrade/reef-x to the mix 53930/head
Yuri Weinstein [Tue, 10 Oct 2023 19:00:12 +0000 (12:00 -0700)]
qa/tests: Added upgrade/reef-x to the mix

Signed-off-by: Yuri Weinstein <yweinste@redhat.com>
21 months agoMerge pull request #53085 from ceph/wip-yuriw-reef-x-main
Yuri Weinstein [Tue, 10 Oct 2023 18:55:24 +0000 (11:55 -0700)]
Merge pull request #53085 from ceph/wip-yuriw-reef-x-main

qa/tests: initial reef-x check in

Reviewed-by: Laura Flores <lflores@redhat.com>
21 months agoMerge pull request #53862 from adk3798/nvmeof-upgrade
Adam King [Tue, 10 Oct 2023 16:08:43 +0000 (12:08 -0400)]
Merge pull request #53862 from adk3798/nvmeof-upgrade

mgr/cephadm: fix upgrades with nvmeof

Reviewed-by: Redouane Kachach <rkachach@redhat.com>
21 months agoMerge pull request #53900 from ljflores/wip-fix-stress-split-tests
Laura Flores [Tue, 10 Oct 2023 15:52:01 +0000 (11:52 -0400)]
Merge pull request #53900 from ljflores/wip-fix-stress-split-tests

qa/suites/upgrade: fix env indentation in stress-split upgrade tests

21 months agodoc/rados: edit memory-profiling.rst 53908/head
Zac Dover [Tue, 10 Oct 2023 08:26:53 +0000 (18:26 +1000)]
doc/rados: edit memory-profiling.rst

Edit doc/rados/troubleshooting/memory-profiling.rst.

Co-authored-by: Anthony D'Atri <anthony.datri@gmail.com>
Signed-off-by: Zac Dover <zac.dover@proton.me>
21 months agoqa/tasks/ceph_manager: thrash - add reset_purged_snaps_last 53579/head
Matan Breizman [Thu, 31 Aug 2023 09:55:33 +0000 (09:55 +0000)]
qa/tasks/ceph_manager: thrash - add reset_purged_snaps_last

Signed-off-by: Matan Breizman <mbreizma@redhat.com>
21 months agoMerge pull request #50132 from aruniiird/add-rbd-mirror-mon-alerts
Juan Miguel Olmo [Tue, 10 Oct 2023 11:37:01 +0000 (13:37 +0200)]
Merge pull request #50132 from aruniiird/add-rbd-mirror-mon-alerts

ceph-mixin: Add RBD Mirror monitoring alerts

21 months agoMerge pull request #53719 from rishabh-d-dave/mon-fscmds
Rishabh Dave [Tue, 10 Oct 2023 05:56:35 +0000 (11:26 +0530)]
Merge pull request #53719 from rishabh-d-dave/mon-fscmds

 mon/FSCommands: clean up FSCommands

Reviewed-by: Leonid Usov <leonid.usov@ibm.com>
Reviewed-by: Xiubo Li <xiubli@redhat.com>
Reviewed-by: Kotresh HR <khiremat@redhat.com>
Reviewed-by: Laura Flores <lflores@redhat.com>
21 months agoMerge PR #53885 into main
Venky Shankar [Tue, 10 Oct 2023 04:05:37 +0000 (09:35 +0530)]
Merge PR #53885 into main

* refs/pull/53885/head:
Revert "mds: disable delegating inode ranges to clients"

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
21 months agoMerge pull request #53517 from cbodley/wip-qa-distros-s
Yuri Weinstein [Mon, 9 Oct 2023 21:04:54 +0000 (14:04 -0700)]
Merge pull request #53517 from cbodley/wip-qa-distros-s

qa/distros: remove centos/rhel8 and ubuntu20.04 from supported distros

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
Reviewed-by: Ilya Dryomov <idryomov@redhat.com>
Reviewed-by: Ali Maredia <amaredia@redhat.com>
21 months agoqa/suites/upgrade: fix env indentation in stress-split upgrade tests 53900/head
Laura Flores [Mon, 9 Oct 2023 20:24:14 +0000 (20:24 +0000)]
qa/suites/upgrade: fix env indentation in stress-split upgrade tests

This is an issue with the stress-split yaml files, as introduced in https://github.com/ceph/ceph/pull/51889.

The stress-split tests have an incorrectly-intented "env" section, which teuthology detects as an entry for "clients".

Fixes: https://tracker.ceph.com/issues/63158
Signed-off-by: Laura Flores <lflores@ibm.com>
21 months agoMerge pull request #53829 from ajarr/wip-63009
Ilya Dryomov [Mon, 9 Oct 2023 19:38:21 +0000 (21:38 +0200)]
Merge pull request #53829 from ajarr/wip-63009

librbd: kick ExclusiveLock state machine stalled waiting for lock from reacquire_lock()

Reviewed-by: Ilya Dryomov <idryomov@gmail.com>
21 months agoMerge pull request #53722 from rishabh-d-dave/mon-authmon
Rishabh Dave [Mon, 9 Oct 2023 17:23:42 +0000 (22:53 +0530)]
Merge pull request #53722 from rishabh-d-dave/mon-authmon

mon/AuthMonitor: clean up AuthMonitor

Reviewed-by: Leonid Usov <leonid.usov@ibm.com>
Reviewed-by: Xiubo Li <xiubli@redhat.com>
Reviewed-by: Kotresh HR <khiremat@redhat.com>
Reviewed-by: Laura Flores <lflores@redhat.com>
21 months agoMerge pull request #53721 from rishabh-d-dave/mon-mdsmon
Rishabh Dave [Mon, 9 Oct 2023 17:22:44 +0000 (22:52 +0530)]
Merge pull request #53721 from rishabh-d-dave/mon-mdsmon

mon/MDSMonitor: clean up MDSMonitor

Reviewed-by: Leonid Usov <leonid.usov@ibm.com>
Reviewed-by: Xiubo Li <xiubli@redhat.com>
Reviewed-by: Kotresh HR <khiremat@redhat.com>
Reviewed-by: Laura Flores <lflores@redhat.com>
21 months agoMerge pull request #53405 from rishabh-d-dave/ceph-auth-caps-val-caps
Rishabh Dave [Mon, 9 Oct 2023 17:19:28 +0000 (22:49 +0530)]
Merge pull request #53405 from rishabh-d-dave/ceph-auth-caps-val-caps

mon/AuthMonitor: make "ceph auth caps" print error messsages

Reviewed-by: Xiubo Li <xiubli@redhat.com>
Reviewed-by: Laura Flores <lflores@redhat.com>
21 months agoMerge PR #53099 into main
Patrick Donnelly [Mon, 9 Oct 2023 17:03:37 +0000 (13:03 -0400)]
Merge PR #53099 into main

* refs/pull/53099/head:
script: update ceph-debug-docker for centos 9.stream

Reviewed-by: Laura Flores <lflores@redhat.com>
21 months agoRevert "mds: disable delegating inode ranges to clients" 53885/head
Venky Shankar [Mon, 9 Oct 2023 05:06:49 +0000 (10:36 +0530)]
Revert "mds: disable delegating inode ranges to clients"

This isn't necessary -- the MDS handles delegating inode ranges
to clients from its preallocated inode set properly - the suspected
bug involving not persisting the sessionmap and causing asserts
during replay isn't an issue. The preallocated set is persisted
with the log event and the MDS correctly rebuild the set from
this information during replay.

Signed-off-by: Venky Shankar <vshankar@redhat.com>
21 months agoMerge PR #53206 into main
Patrick Donnelly [Mon, 9 Oct 2023 15:53:31 +0000 (11:53 -0400)]
Merge PR #53206 into main

* refs/pull/53206/head:
mds: use LogSegment dump for debugging

Reviewed-by: Leonid Usov <leonid.usov@ibm.com>
Reviewed-by: Venky Shankar <vshankar@redhat.com>
21 months agoMerge pull request #52254 from vedanshbhartia/coverity_uninit
Yuval Lifshitz [Mon, 9 Oct 2023 14:58:42 +0000 (17:58 +0300)]
Merge pull request #52254 from vedanshbhartia/coverity_uninit

rgw: Add coverity uninitialized variable and initialize RGWBucketEntryMetadataObject

reviewed-by: yuvalif

21 months agoMerge pull request #52328 from vedanshbhartia/coverity_1512267
Yuval Lifshitz [Mon, 9 Oct 2023 14:57:47 +0000 (17:57 +0300)]
Merge pull request #52328 from vedanshbhartia/coverity_1512267

rgw: fix potential null dereference in rgw_iam_policy.c: ParseState::do_string

reviewed-by: yuvalif

21 months agoMerge pull request #52472 from vedanshbhartia/coverity_1510724
Yuval Lifshitz [Mon, 9 Oct 2023 14:56:40 +0000 (17:56 +0300)]
Merge pull request #52472 from vedanshbhartia/coverity_1510724

rgw: Remove unnecessary null check from valid_s3_bucket_name

reviwed-by: soumyakoduri , yuvalif

21 months agoMerge pull request #52734 from vedanshbhartia/coverity_ostream
Yuval Lifshitz [Mon, 9 Oct 2023 14:54:34 +0000 (17:54 +0300)]
Merge pull request #52734 from vedanshbhartia/coverity_ostream

rgw: Restore ostream format state after changing it

reviwed-by: yuvalif

21 months agoMerge pull request #52326 from yuvalif/wip-yuval-lua-reload
Yuval Lifshitz [Mon, 9 Oct 2023 14:53:15 +0000 (17:53 +0300)]
Merge pull request #52326 from yuvalif/wip-yuval-lua-reload

rgw/lua: support reloading lua packages on all RGWs

reviwed-by: dang, cbodle, anthonyeleven

21 months agoMerge pull request #53890 from zdover23/wip-doc-2023-10-09-troubleshooting-troublesho...
zdover23 [Mon, 9 Oct 2023 13:53:05 +0000 (23:53 +1000)]
Merge pull request #53890 from zdover23/wip-doc-2023-10-09-troubleshooting-troubleshooting-mon-4-of-x

doc/rados: edit troubleshooting-mon.rst (4 of x)

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
21 months agoMerge pull request #53892 from rishabh-d-dave/fix-test_cephfs.py
Rishabh Dave [Mon, 9 Oct 2023 13:47:07 +0000 (19:17 +0530)]
Merge pull request #53892 from rishabh-d-dave/fix-test_cephfs.py

src/test/pybind: don't use decorator "with_setup"

Reviewed-by: Venky Shankar <vshankar@redhat.com>
21 months agodoc/rados: edit troubleshooting-mon.rst (4 of x) 53890/head
Zac Dover [Mon, 9 Oct 2023 08:55:07 +0000 (18:55 +1000)]
doc/rados: edit troubleshooting-mon.rst (4 of x)

Edit doc/rados/troubleshooting/troubleshooting-mon.rst.

Follows https://github.com/ceph/ceph/pull/53875

Co-authored-by: Anthony D'Atri <anthony.datri@gmail.com>
Signed-off-by: Zac Dover <zac.dover@proton.me>
21 months agoMerge pull request #53884 from leonid-s-usov/rgw-posix-test
Daniel Gryniewicz [Mon, 9 Oct 2023 12:57:13 +0000 (08:57 -0400)]
Merge pull request #53884 from leonid-s-usov/rgw-posix-test

test/rgw: don't compile POSIX test unless enabled

Reviewed-by: Daniel Gryniewicz <dang@redhat.com>
21 months agoMerge PR #53873 into main
Venky Shankar [Mon, 9 Oct 2023 12:19:30 +0000 (17:49 +0530)]
Merge PR #53873 into main

* refs/pull/53873/head:
qa: typo fix when checking for perf counter - s/md_thresh_evicted/mdthresh_evicted
qa: lower mds_session_metadata_threshold for tests

Reviewed-by: Milind Changire <mchangir@redhat.com>
Reviewed-by: Rishabh Dave <ridave@redhat.com>
Reviewed-by: Leonid Usov <leonid.usov@ibm.com>
21 months agosrc/test/pybind: don't use decorator "with_setup" 53892/head
Rishabh Dave [Mon, 9 Oct 2023 11:28:16 +0000 (16:58 +0530)]
src/test/pybind: don't use decorator "with_setup"

Signed-off-by: Rishabh Dave <ridave@redhat.com>
21 months agoqa: typo fix when checking for perf counter - s/md_thresh_evicted/mdthresh_evicted 53873/head
Venky Shankar [Mon, 9 Oct 2023 04:29:45 +0000 (09:59 +0530)]
qa: typo fix when checking for perf counter - s/md_thresh_evicted/mdthresh_evicted

Signed-off-by: Venky Shankar <vshankar@redhat.com>
21 months agoqa: lower mds_session_metadata_threshold for tests
Venky Shankar [Sat, 7 Oct 2023 15:12:29 +0000 (20:42 +0530)]
qa: lower mds_session_metadata_threshold for tests

... and increase the number of files that are created so as to
hit the threshold with a high probability.

Fixes: http://tracker.ceph.com/issues/62873
Signed-off-by: Venky Shankar <vshankar@redhat.com>
21 months agoMerge pull request #53817 from cloudbehl/active-alert-filter
Nizamudeen A [Mon, 9 Oct 2023 05:11:21 +0000 (10:41 +0530)]
Merge pull request #53817 from cloudbehl/active-alert-filter

mgr/dashboard: Filter active alerts

Reviewed-by: Nizamudeen A <nia@redhat.com>
21 months agotest/rgw: don't compile POSIX test unless enabled 53884/head
Leonid Usov [Sun, 8 Oct 2023 14:14:23 +0000 (17:14 +0300)]
test/rgw: don't compile POSIX test unless enabled

Signed-off-by: Leonid Usov <leonid.usov@ibm.com>
21 months agolibrbd/ManagedLock: kickstart ExclusiveLock state machine 53829/head
Ramana Raja [Mon, 2 Oct 2023 16:39:26 +0000 (12:39 -0400)]
librbd/ManagedLock: kickstart ExclusiveLock state machine

... that is stalled waiting for lock. Do this when trying to reacquire
lock in the ImageWatcher's rewatch mechanism. This would enable the
ExclusiveLock state machine to propagate the blocklist error to the
caller trying to perform an image operation requiring an exclusive
lock.

Previous attempt, e66db763, to fix the hang due to exclusive lock
acquisiton (stuck waiting for lock) racing with client blocklisting
did not always work. e66db763 kickstarted the ExclusiveLock state
machine when the ImageWatcher tried to schedule a exclusive lock
request and the blocklisting was detected. However, there is a short
window between a watch getting deregistered and client blocklisting
getting detected as part of rewatching. If hit when trying to schedule
a lock request, the ExclusiveLock state machine wasn't kickstarted,
blocklist error wasn't propagated, and the hang resurfaced.

A more robust approach is taken to resume the ExclusiveLock state
machine stuck waiting for lock during client blocklisting. Whenever
a client's ImageWatcher loses connection to the cluster, as it happens
during blocklising, the ImageWatcher initiates a mechanism to rewatch
the image and tries to reacquire the lock. Piggyback on this rewatch
mechanism that gets triggered during client blocklisting. And when
trying to reacquire the lock, kickstart the ExclusiveLock state
machine stalled waiting for lock (STATE_WAITING_FOR_LOCK).

Fixes: https://tracker.ceph.com/issues/63009
Signed-off-by: Ramana Raja <rraja@redhat.com>
21 months agoMerge pull request #53874 from zdover23/wip-doc-2023-10-07-rados-troubleshooting...
zdover23 [Sun, 8 Oct 2023 05:10:20 +0000 (16:10 +1100)]
Merge pull request #53874 from zdover23/wip-doc-2023-10-07-rados-troubleshooting-community

doc/rados: edit troubleshooting/community.rst

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
21 months agoMerge pull request #53875 from zdover23/wip-doc-2023-10-07-troubleshooting-troublesho...
zdover23 [Sun, 8 Oct 2023 04:50:45 +0000 (15:50 +1100)]
Merge pull request #53875 from zdover23/wip-doc-2023-10-07-troubleshooting-troubleshooting-mon-3-of-x

doc/rados: edit troubleshooting-mon.rst (3 of x)

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
21 months agodoc/rados: edit troubleshooting/community.rst 53874/head
Zac Dover [Sat, 7 Oct 2023 20:39:23 +0000 (06:39 +1000)]
doc/rados: edit troubleshooting/community.rst

Edit doc/rados/troubleshooting/community.rst.

Co-authored-by: Anthony D'Atri <anthony.datri@gmail.com>
Signed-off-by: Zac Dover <zac.dover@proton.me>
21 months agodoc/rados: edit troubleshooting-mon.rst (3 of x) 53875/head
Zac Dover [Sat, 7 Oct 2023 21:33:17 +0000 (07:33 +1000)]
doc/rados: edit troubleshooting-mon.rst (3 of x)

Edit doc/rados/troubleshooting/troubleshooting-mon.rst.

Follows https://github.com/ceph/ceph/pull/52827

Co-authored-by: Anthony D'Atri <anthony.datri@gmail.com>
Signed-off-by: Zac Dover <zac.dover@proton.me>
21 months agoMerge pull request #53876 from zdover23/wip-doc-2023-10-08-architecture-rbd-sentence...
Anthony D'Atri [Sun, 8 Oct 2023 00:45:38 +0000 (20:45 -0400)]
Merge pull request #53876 from zdover23/wip-doc-2023-10-08-architecture-rbd-sentence-repair

doc/architecture: repair RBD sentence

21 months agodoc/architecture: repair RBD sentence 53876/head
Zac Dover [Sat, 7 Oct 2023 21:43:43 +0000 (07:43 +1000)]
doc/architecture: repair RBD sentence

Improve an ambiguous sentence in doc/architecture.rst.

The problem presented by the original sentence is that the phrasal verb
"to provide with" is implicated in one of its possible readings.
Interpreted in that way, the sentence seems to express the incorrect
idea that RBD furnishes block devices with snapshotting and cloning, as
though snapshotting and cloning are being delivered to the block
devices. In fact, snapshotting and cloning are just features of RBD, and
are features that are described on this page:
https://docs.ceph.com/en/quincy/rbd/rbd-snapshot/.

Signed-off-by: Zac Dover <zac.dover@proton.me>
21 months agoMerge pull request #53790 from zdover23/wip-doc-2023-10-03-architecture-17-of-x
zdover23 [Sat, 7 Oct 2023 04:51:27 +0000 (15:51 +1100)]
Merge pull request #53790 from zdover23/wip-doc-2023-10-03-architecture-17-of-x

doc/architecture: edit "Peering and Sets"

21 months agodoc/architecture: edit "Peering and Sets" 53790/head
Zac Dover [Tue, 3 Oct 2023 12:11:50 +0000 (22:11 +1000)]
doc/architecture: edit "Peering and Sets"

Edit the English in the section "Peering and Sets" in the file
doc/architecture.rst.

Co-authored-by: Anthony D'Atri <anthony.datri@gmail.com>
Signed-off-by: Zac Dover <zac.dover@proton.me>
21 months agoMerge PR #53855 into main
Patrick Donnelly [Sat, 7 Oct 2023 00:15:23 +0000 (20:15 -0400)]
Merge PR #53855 into main

* refs/pull/53855/head:
script: add option for debug build

Reviewed-by: Ilya Dryomov <idryomov@redhat.com>
21 months agoscript: add option for debug build 53855/head
Patrick Donnelly [Fri, 6 Oct 2023 00:11:29 +0000 (20:11 -0400)]
script: add option for debug build

See: https://github.com/ceph/ceph-build/pull/2167

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
21 months agomgr/cephadm: fix upgrades with nvmeof 53862/head
Adam King [Fri, 6 Oct 2023 15:20:57 +0000 (11:20 -0400)]
mgr/cephadm: fix upgrades with nvmeof

Currently, nvmeof was being treated as if it used
a ceph image during upgrades. This would cause logging
of messages like (I've removed the nvmeof daemon id)

log [WRN] :     Upgrade daemon: nvmeof.<id>: Cannot redeploy
nvmeof.<id> with a new image: Supported types are: mgr, mon,
crash, osd, mds, rgw, rbd-mirror, cephfs-mirror, ceph-exporter,
iscsi, nfs

and if you had set a custom image for the
mgr/cephadm/container_image_nvmeof setting, this would
be undone as part of the upgrade process.

Fixes: https://tracker.ceph.com/issues/63127
Signed-off-by: Adam King <adking@redhat.com>
21 months agoMerge pull request #53417 from jrchyang/fix_mclock_scheduling_slow_main
Yuri Weinstein [Fri, 6 Oct 2023 13:58:11 +0000 (06:58 -0700)]
Merge pull request #53417 from jrchyang/fix_mclock_scheduling_slow_main

osd: fix: slow scheduling when item_cost is large

Reviewed-by: Sridhar Seshasayee <sseshasa@redhat.com>
21 months agoMerge pull request #52430 from vedanshbhartia/coverity_datarace
Yuval Lifshitz [Fri, 6 Oct 2023 10:07:31 +0000 (13:07 +0300)]
Merge pull request #52430 from vedanshbhartia/coverity_datarace

rgw: Add coverity annotations for missing mutex locks

reviewed-by: yuvalif, mkogan1

21 months agorgw/lua/doc: support reloading lua packages on all RGWs 52326/head
Yuval Lifshitz [Wed, 5 Jul 2023 18:33:55 +0000 (18:33 +0000)]
rgw/lua/doc: support reloading lua packages on all RGWs

without requiring a restart of the RGWs
test instructions:
https://gist.github.com/yuvalif/95b8ed9ea73ab4591c59644a050e01e2
also use capitalized "Lua" in logs/doc

Signed-off-by: Yuval Lifshitz <ylifshit@ibm.com>
21 months agomon/AuthMonitor: check if entity is absent before creating it 53405/head
Rishabh Dave [Thu, 28 Sep 2023 17:34:51 +0000 (23:04 +0530)]
mon/AuthMonitor: check if entity is absent before creating it

Although this code path is not used for creating entities yet, it is
better to fix the bug sooner than later. Method
AuthMonitor::_update_or_create_entity() must exit (with appropriate
error code) when entity to be created on the Ceph cluster is already
present.

Signed-off-by: Rishabh Dave <ridave@redhat.com>
21 months agomds/MDSAuthCaps: re-word an error message for better clarity
Rishabh Dave [Thu, 28 Sep 2023 14:29:24 +0000 (19:59 +0530)]
mds/MDSAuthCaps: re-word an error message for better clarity

Signed-off-by: Rishabh Dave <ridave@redhat.com>
21 months agoMerge pull request #53834 from dparmar18/remove-egg-fragment-from-doc
zdover23 [Thu, 5 Oct 2023 23:21:14 +0000 (10:21 +1100)]
Merge pull request #53834 from dparmar18/remove-egg-fragment-from-doc

doc: remove egg fragment from dev/developer_guide/running-tests-locally

Reviewed-by: John Mulligan <jmulligan@redhat.com>
Reviewed-by: Rishabh Dave <ridave@redhat.com>
Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
Reviewed-by: Zac Dover <zac.dover@proton.me>
21 months agorgw: fix http error checks in keystone/barbican/vault clients 53846/head
Casey Bodley [Thu, 5 Oct 2023 15:59:52 +0000 (11:59 -0400)]
rgw: fix http error checks in keystone/barbican/vault clients

when RGWHTTPManager encounters an http error, it uses
rgw_http_error_to_errno() to map that to a negative posix error code.
RGWHTTPClient::process() returns that mapped error code, and exposes the
original http error via get_http_status()

the http client code for keystone, barbican, and vault were returning
early on the errors from process(), so weren't getting to the http error
checks

these clients now check for specific http errors before testing the
result of process()

Fixes: https://tracker.ceph.com/issues/62989
Signed-off-by: Casey Bodley <cbodley@redhat.com>
21 months agoMerge pull request #52296 from asm0deuz/bz2153448_ipv6
Adam King [Thu, 5 Oct 2023 15:43:53 +0000 (11:43 -0400)]
Merge pull request #52296 from asm0deuz/bz2153448_ipv6

mgr/cephadm: ceph orch add fails when ipv6 address is surrounded by square brackets.

Reviewed-by: Adam King <adking@redhat.com>
21 months agoMerge pull request #53621 from phlogistonjohn/jjm-cephadm-dtypes-common
Adam King [Thu, 5 Oct 2023 15:41:47 +0000 (11:41 -0400)]
Merge pull request #53621 from phlogistonjohn/jjm-cephadm-dtypes-common

cephadm: introduce Daemon Forms

Reviewed-by: Adam King <adking@redhat.com>
21 months agoMerge PR #53836 into main
Venky Shankar [Thu, 5 Oct 2023 15:28:33 +0000 (20:58 +0530)]
Merge PR #53836 into main

* refs/pull/53836/head:
PendingReleaseNotes: add a note about disallowing delegating inodes
qa: start testing mds_client_delegate_inos_pct config
mds: disable delegating inode ranges to clients

Reviewed-by: Milind Changire <mchangir@redhat.com>
Reviewed-by: Dhairya Parmar <dparmar@redhat.com>
21 months agodoc: remove egg fragment from dev/developer_guide/running-tests-locally 53834/head
Dhairya Parmar [Thu, 5 Oct 2023 08:12:31 +0000 (13:42 +0530)]
doc: remove egg fragment from dev/developer_guide/running-tests-locally

DEPRECATION: git+https://github.com/ceph/teuthology#egg=teuthology
[test] contains an egg fragment with a non-PEP 508 name pip 25.0 will enforce
this behaviour change. A possible replacement is to use the req @ url syntax,
and remove the egg fragment. Discussion can be found at
https://github.com/pypa/pip/issues/11617

Signed-off-by: Dhairya Parmar <dparmar@redhat.com>
21 months agosrc/MDSMonitor: make use of imported namespace symbols 53721/head
Rishabh Dave [Wed, 4 Oct 2023 18:52:51 +0000 (00:22 +0530)]
src/MDSMonitor: make use of imported namespace symbols

Symbols imported into current namespace should be used directly; there
is no need to mention their parent namespace while using them. IOW, to use
"std::string" after it has been imported, just write "string" instead of
"std::string".

Signed-off-by: Rishabh Dave <ridave@redhat.com>
21 months agomon/MDSMonitor: remove unused namespace imports
Rishabh Dave [Thu, 28 Sep 2023 18:05:22 +0000 (23:35 +0530)]
mon/MDSMonitor: remove unused namespace imports

Remove symbols imported into the current namespace since they are not being
used.

Signed-off-by: Rishabh Dave <ridave@redhat.com>
21 months agomon/FSCommands: import std::ostream into current namespace 53719/head
Rishabh Dave [Wed, 4 Oct 2023 18:08:47 +0000 (23:38 +0530)]
mon/FSCommands: import std::ostream into current namespace

Signed-off-by: Rishabh Dave <ridave@redhat.com>
21 months agorgw/lua: install lua packages in temp directory
Yuval Lifshitz [Tue, 4 Jul 2023 16:27:38 +0000 (16:27 +0000)]
rgw/lua: install lua packages in temp directory

and switch to it only once installation is done.
this is needed for cases where installation can happen
while RGW is running

Signed-off-by: Yuval Lifshitz <ylifshit@ibm.com>
21 months agorgw/lua/doc: fix luajson example
Yuval Lifshitz [Tue, 4 Jul 2023 15:38:04 +0000 (15:38 +0000)]
rgw/lua/doc: fix luajson example

Signed-off-by: Yuval Lifshitz <ylifshit@ibm.com>
21 months agomon/FSCommands: make use of imported namespace symbols
Rishabh Dave [Wed, 4 Oct 2023 18:16:32 +0000 (23:46 +0530)]
mon/FSCommands: make use of imported namespace symbols

There's no need to mention the "home" namespace of a symbol while using
it after it has been imported into the current namespace. IOW, no need to
write, for example, "std::string" after it has been imported from its
namespace; instead simply writing "string" will suffice.

Signed-off-by: Rishabh Dave <ridave@redhat.com>
21 months agomon/FSCommands: remove unused namespace imports
Rishabh Dave [Thu, 28 Sep 2023 17:10:19 +0000 (22:40 +0530)]
mon/FSCommands: remove unused namespace imports

When symbols imported into the current namespace won't be used, don't
import them.

Signed-off-by: Rishabh Dave <ridave@redhat.com>
21 months agomon/AuthMonitor: make use of imported namespace symbols 53722/head
Rishabh Dave [Wed, 4 Oct 2023 18:22:27 +0000 (23:52 +0530)]
mon/AuthMonitor: make use of imported namespace symbols

Once a symbol has been imported into the current namespace, no need to
mention the original namespace while using it. IOW, no need to write
"std::string" after it has been imported from the namespace "std" into the
current namespace.

Signed-off-by: Rishabh Dave <ridave@redhat.com>
21 months agomon/AuthMonitor: remove unused namespace imports
Rishabh Dave [Thu, 28 Sep 2023 18:14:09 +0000 (23:44 +0530)]
mon/AuthMonitor: remove unused namespace imports

Don't import items into current namespace when they'll be never used.

Signed-off-by: Rishabh Dave <ridave@redhat.com>
21 months agoMerge pull request #53700 from cloudbehl/cephfs-snapshot
Nizamudeen A [Thu, 5 Oct 2023 14:10:32 +0000 (19:40 +0530)]
Merge pull request #53700 from cloudbehl/cephfs-snapshot

mgr/dashboard: Fixed cephfs snapshot listing in directories

Reviewed-by: Avan Thakkar <athakkar@redhat.com>
Reviewed-by: Nizamudeen A <nia@redhat.com>
21 months agoMerge pull request #53782 from sachinpunadikar/vstart-fix
Adam King [Thu, 5 Oct 2023 12:37:56 +0000 (08:37 -0400)]
Merge pull request #53782 from sachinpunadikar/vstart-fix

vstart: exclude default route during cluster setup

Reviewed-by: Adam King <adking@redhat.com>
21 months agomgr/dashboard: Filter active alerts 53817/head
cloudbehl [Wed, 4 Oct 2023 11:05:26 +0000 (16:35 +0530)]
mgr/dashboard: Filter active alerts

- List alerts which are not suppressed

fixes: https://tracker.ceph.com/issues/62990

Signed-off-by: cloudbehl <cloudbehl@gmail.com>
21 months agoMerge pull request #53819 from zdover23/wip-doc-2023-10-04-rados-troubleshooting...
zdover23 [Thu, 5 Oct 2023 11:43:02 +0000 (22:43 +1100)]
Merge pull request #53819 from zdover23/wip-doc-2023-10-04-rados-troubleshooting-index

doc/rados: edit troubleshooting.rst

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
21 months agoPendingReleaseNotes: add a note about disallowing delegating inodes 53836/head
Venky Shankar [Thu, 5 Oct 2023 11:24:04 +0000 (16:54 +0530)]
PendingReleaseNotes: add a note about disallowing delegating inodes

Signed-off-by: Venky Shankar <vshankar@redhat.com>
21 months agoqa: start testing mds_client_delegate_inos_pct config
Venky Shankar [Wed, 4 Oct 2023 14:20:12 +0000 (19:50 +0530)]
qa: start testing mds_client_delegate_inos_pct config

Signed-off-by: Venky Shankar <vshankar@redhat.com>
21 months agomds: disable delegating inode ranges to clients
Venky Shankar [Thu, 5 Oct 2023 09:26:32 +0000 (14:56 +0530)]
mds: disable delegating inode ranges to clients

Fixes: http://tracker.ceph.com/issues/63103
Signed-off-by: Venky Shankar <vshankar@redhat.com>
21 months agoMerge pull request #52735 from vedanshbhartia/coverity_minlong
Yuval Lifshitz [Thu, 5 Oct 2023 07:50:21 +0000 (10:50 +0300)]
Merge pull request #52735 from vedanshbhartia/coverity_minlong

rgw: Add coverity annotation for warning about tautological comparison

reviewed-by: thotz, yuvalif

21 months agoMerge pull request #53788 from AliMasarweh/wip-alimasa-topic-stats-help
Ali Masarwa [Thu, 5 Oct 2023 07:46:37 +0000 (10:46 +0300)]
Merge pull request #53788 from AliMasarweh/wip-alimasa-topic-stats-help

RGW: add the missing help print for command 'topic stats'

Reviewed-by: thotz <thottanjiffin@gmail.com>, Daniel Gryniewicz <dang@redhat.com>
21 months agoMerge pull request #53789 from rhcs-dashboard/rgw-overview-fix
Nizamudeen A [Thu, 5 Oct 2023 06:55:11 +0000 (12:25 +0530)]
Merge pull request #53789 from rhcs-dashboard/rgw-overview-fix

mgr/dashboard: fix rgw inventory card and broken shadows

Reviewed-by: Aashish Sharma <aasharma@redhat.com>
Reviewed-by: Ankush Behl <cloudbehl@gmail.com>
21 months agovstart: exclude default route during cluster setup 53782/head
Sachin Punadikar [Tue, 3 Oct 2023 07:46:31 +0000 (03:46 -0400)]
vstart: exclude default route during cluster setup

"ip route list" may list default route, and that needs to be excluded
while doing cluster setup
Typical o/p of ip route list:
$ ip route list
default via 10.8.159.254 dev eno1 proto dhcp src 10.8.152.13 metric 100
10.8.152.0/21 dev eno1 proto kernel scope link src 10.8.152.13 metric 100

Signed-off-by: Sachin Punadikar <sachin.punadikar@ibm.com>
21 months agoMerge pull request #52313 from linuxbox2/wip-fix-disable-dashboard
Nizamudeen A [Thu, 5 Oct 2023 03:42:25 +0000 (09:12 +0530)]
Merge pull request #52313 from linuxbox2/wip-fix-disable-dashboard

dashboard: regression, make install fails w/dashboard disabled

Reviewed-by: Ernesto Puerta <epuertat@redhat.com>
21 months agocephadm: convert cephadm agent to a daemon form 53621/head
John Mulligan [Wed, 4 Oct 2023 17:43:47 +0000 (13:43 -0400)]
cephadm: convert cephadm agent to a daemon form

The cephadm agent is a bit special in that it will not be converted
to a ContainerDaemonForm (it is not containerized) but we still want
to have it registered as a DeamonForm so that the deamon_type can be
passed to create and have it resolve correctly.

Signed-off-by: John Mulligan <jmulligan@redhat.com>
21 months agocephadm: convert SNMPGateway to a ContainerDaemonForm
John Mulligan [Sat, 23 Sep 2023 19:28:40 +0000 (15:28 -0400)]
cephadm: convert SNMPGateway to a ContainerDaemonForm

Signed-off-by: John Mulligan <jmulligan@redhat.com>
21 months agocephadm: convert CustomContainer to a ContainerDaemonForm
John Mulligan [Sat, 23 Sep 2023 19:28:25 +0000 (15:28 -0400)]
cephadm: convert CustomContainer to a ContainerDaemonForm

Signed-off-by: John Mulligan <jmulligan@redhat.com>
21 months agocephadm: convert NFSGanesha to a ContainerDaemonForm
John Mulligan [Sat, 23 Sep 2023 19:27:46 +0000 (15:27 -0400)]
cephadm: convert NFSGanesha to a ContainerDaemonForm

Signed-off-by: John Mulligan <jmulligan@redhat.com>
21 months agocephadm: add func to deploy any generic ContainerDaemonForm
John Mulligan [Sat, 23 Sep 2023 19:26:03 +0000 (15:26 -0400)]
cephadm: add func to deploy any generic ContainerDaemonForm

While there are no ContainerDaemonForms implemented yet, add a function
that uses the ContainerDaemonForm methods to construct a deployment
for the container based daemons.

Signed-off-by: John Mulligan <jmulligan@redhat.com>
21 months agocephadm: add ContainerDaemonForm
John Mulligan [Sat, 23 Sep 2023 19:23:42 +0000 (15:23 -0400)]
cephadm: add ContainerDaemonForm

Add a supplemental DaemonForm subclass that helps deploy container
based daemons in a standard fashion. Most of these methods are
optional and should have sensible defaults.

Signed-off-by: John Mulligan <jmulligan@redhat.com>
21 months agocephadm: move DeploymentType to deploy.py
John Mulligan [Sat, 23 Sep 2023 19:22:40 +0000 (15:22 -0400)]
cephadm: move DeploymentType to deploy.py

The DeploymentType is used by a number of other classes and functions
and has no dependencies beyond enum and is safe to move.

Signed-off-by: John Mulligan <jmulligan@redhat.com>
21 months agocephadm: move firewalld related items to firewalld.py
John Mulligan [Fri, 22 Sep 2023 17:34:58 +0000 (13:34 -0400)]
cephadm: move firewalld related items to firewalld.py

Signed-off-by: John Mulligan <jmulligan@redhat.com>
21 months agocephadm: remove direct daemon-class deps from firewall
John Mulligan [Fri, 22 Sep 2023 17:26:18 +0000 (13:26 -0400)]
cephadm: remove direct daemon-class deps from firewall

Signed-off-by: John Mulligan <jmulligan@redhat.com>
21 months agocephadm: move sysctl specific functions to sysctl.py
John Mulligan [Fri, 22 Sep 2023 15:14:34 +0000 (11:14 -0400)]
cephadm: move sysctl specific functions to sysctl.py

Signed-off-by: John Mulligan <jmulligan@redhat.com>
21 months agocephadm: remove direct daemon-type deps from sysctl
John Mulligan [Thu, 21 Sep 2023 21:57:43 +0000 (17:57 -0400)]
cephadm: remove direct daemon-type deps from sysctl

Using the appropriate daemon form we can break the direct dependency
that the sysctl setup function has on particular classes and use
a generic interface.

Signed-off-by: John Mulligan <jmulligan@redhat.com>
21 months agocephadm: add test_daemon_form.py
John Mulligan [Thu, 21 Sep 2023 20:21:25 +0000 (16:21 -0400)]
cephadm: add test_daemon_form.py

Signed-off-by: John Mulligan <jmulligan@redhat.com>
21 months agocephadm: introduce daemon forms to cephadm.py
John Mulligan [Wed, 20 Sep 2023 14:17:29 +0000 (10:17 -0400)]
cephadm: introduce daemon forms to cephadm.py

Introduce the DeamonForm base class to cephadm.py and make various
daemon-type classes into fully fleged deamon form classes.

Some classes already had a semi-standard `init` classmethod for
instantiation. In these cases the new `create` classmethod is a thin
wrapper over the existing method. In cases where the class was not
already being instantiated a minimal set of methods are added.

Signed-off-by: John Mulligan <jmulligan@redhat.com>
21 months agocephadm: add daemon_form.py: bases and funcs for daemon forms
John Mulligan [Thu, 21 Sep 2023 20:21:00 +0000 (16:21 -0400)]
cephadm: add daemon_form.py: bases and funcs for daemon forms

Create daemon_form.py containing the DaemonForm class and a few
subclasses and utility functions for working with DaemonForms.
In a future commit, DaemonForm will become the base class for
the current assortment of classes named after the daemon or
family of daemon they help manage.

A daemon form, think "form" as in "template" or "mold", assists
in setting up, creating, and managing daemons controlled with
cephadm. Because cephadm supports a variety of services the
DaemonForm is an abstract base class and the module also supports
additional ABCs that may be used by DaemonForms to implement
optional features.

The daemon forms that are expected to be used directly must be
registered using the provided decorator. This is an explicit extra
step so that common bases that inherit from DaemonForm can be
implemented. Plus explicit is better than implicit. :-)
All DeamonForm subclasses are expected to provide a small set
of standard methods so that the types can be chosen, instantiated,
and used a common manner.

Signed-off-by: John Mulligan <jmulligan@redhat.com>
21 months agocephadm: add some unit test coverage for deploying nfs, snmp
John Mulligan [Mon, 25 Sep 2023 17:58:35 +0000 (13:58 -0400)]
cephadm: add some unit test coverage for deploying nfs, snmp

Signed-off-by: John Mulligan <jmulligan@redhat.com>
21 months agodoc/rados: edit troubleshooting.rst 53819/head
Zac Dover [Wed, 4 Oct 2023 11:14:13 +0000 (21:14 +1000)]
doc/rados: edit troubleshooting.rst

Edit doc/rads/troubleshooting.rst to remove some language that sounds
quite close to marketing language.

Signed-off-by: Zac Dover <zac.dover@proton.me>
21 months agodashboard: regression, make install fails w/dashboard disabled 52313/head
Matt Benjamin [Tue, 4 Jul 2023 20:33:25 +0000 (16:33 -0400)]
dashboard: regression, make install fails w/dashboard disabled

https://tracker.ceph.com/issues/63100

Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
21 months agoMerge pull request #53436 from rhcs-dashboard/fix-bootstrap-script
Nizamudeen A [Wed, 4 Oct 2023 13:13:09 +0000 (18:43 +0530)]
Merge pull request #53436 from rhcs-dashboard/fix-bootstrap-script

mgr/dashboard: fix bootstrap script for cephadm installation

Reviewed-by: Adam King <adking@redhat.com>
Reviewed-by: Nizamudeen A <nia@redhat.com>
Reviewed-by: John Mulligan <jmulligan@redhat.com>