Merge pull request #51031 from cbodley/wip-59423

rgw/aio: fix recursion from tracing changes

Reviewed-by: Daniel Gryniewicz <dang@redhat.com>
Reviewed-by: Yuval Lifshitz <ylifshit@redhat.com>

Merge pull request #50975 from yuvalif/wip-yuval-fix-59381

rgw/lua: fix CopyFrom crash

Merge pull request #50965 from yuvalif/wip-yuval-fix-59383

rgw/kafka: failed to reconnect to broker after idle timeout

Merge pull request #50918 from rhcs-dashboard/delete-ceph-authx

mgr/dashboard: delete-ceph-authx

Reviewed-by: Nizamudeen A <nia@redhat.com>
Reviewed-by: Pere Diaz Bou <pdiazbou@redhat.com>

Merge PR #50089 into main

* refs/pull/50089/head:
doc: add a note for minimum compatible python version and supported distros
tools/cephfs/top/CMakeList.txt: check the minimum compatible python version for cephfs-top

Reviewed-by: Ken Dreyer <kdreyer@redhat.com>
Reviewed-by: Rishabh Dave <ridave@redhat.com>
Reviewed-by: Venky Shankar <vshankar@redhat.com>
Reviewed-by: Neeraj Pratap Singh <neesingh@redhat.com>
Reviewed-by: Dhairya Parmar <dparmar@redhat.com>

Merge pull request #50916 from petrutlucian94/avoid_pip

win32_deps_build: avoid pip

Reviewed-by: Ilya Dryomov <idryomov@gmail.com>

doc: add a note for minimum compatible python version and supported distros

Fixes: https://tracker.ceph.com/issues/58677
Signed-off-by: Jos Collin <jcollin@redhat.com>

tools/cephfs/top/CMakeList.txt: check the minimum compatible python version for cephfs-top

Fixes: https://tracker.ceph.com/issues/58677
Signed-off-by: Jos Collin <jcollin@redhat.com>

win32_deps_build: avoid pip

We're currently installing cython with pip when using Ubuntu
to cross compile Ceph for Windows. This can fail with recent
Python versions if attempting to use the global env:

  error: externally-managed-environment

  × This environment is externally managed
  ╰─> To install Python packages system-wide, try apt install
      python3-xyz, where xyz is the package you are trying to
      install.

Cython isn't really needed by the Windows build so we can go
ahead and drop it. We were hoping to use the Python bindings
on Windows, however Python extensions can't be cross compiled.

We're no longer using pip either, so we're dropping the dependency.
g++ was getting installed as a pip dependency, so we'll have to
include that instead. Note that g++ is used when building the boost
b2 tool.

While at it, we'll also ensure that git is installed.

Fixes: https://tracker.ceph.com/issues/59354
Signed-off-by: Lucian Petrut <lpetrut@cloudbasesolutions.com>

Merge pull request #50639 from bitfehler/bitfehler/doc-pg-num

doc: account for PG autoscaling being the default

Merge pull request #51043 from petrutlucian94/fix_jaeger_encoding

common/tracer: fix decoding when jaeger tracing is disabled

Reviewed-by: Casey Bodley <cbodley@redhat.com>
Reviewed-by: Ilya Dryomov <idryomov@gmail.com>
Reviewed-by: Yuval Lifshitz <ylifshit@redhat.com>

Merge pull request #49762 from Svelar/vstart2

vstart: fix text format

Reviewed-by: Zac Dover <zac.dover@proton.me>

Merge pull request #50820 from rhcs-dashboard/update-landing-page-e2e-tests

mgr/dashboard: update landing page e2e tests

Reviewed-by: Nizamudeen A <nia@redhat.com>

Merge pull request #51082 from rhcs-dashboard/skip-osds-conditionally-in-e2e

mgr/dashboard: skip Create OSDs step in Cluster expansion

Reviewed-by: Aashish Sharma <aasharma@redhat.com>

Merge pull request #51115 from rhcs-dashboard/remove-hyperlink-mgr

mgr/dashboard: remove unncessary hyperlink in landing page

Reviewed-by: Aashish Sharma <aasharma@redhat.com>

Merge pull request #51114 from colemitchell/wip-doc-2023-04-17-radosgw-s3select-formatting-1

doc/radosgw: format part of s3select

Reviewed-by: Zac Dover <zac.dover@proton.me>

doc/radosgw: format part of s3select

Partially format the 'Basic Workflow' section's introduction and 'Basic Functionalities' subsection in s3select. Nothing else is being fixed.

Signed-off-by: Cole Mitchell <cole.mitchell.ceph@gmail.com>

mgr/dashboard: remove unncessary hyperlink in landing page

Fixes: https://tracker.ceph.com/issues/59462
Signed-off-by: Nizamudeen A <nia@redhat.com>

Merge pull request #48869 from rhcs-dashboard/more-grafana-configs

mgr/dashboard: expose more grafana configs in service form

Reviewed-by: Aashish Sharma <aasharma@redhat.com>
Reviewed-by: Avan Thakkar <athakkar@redhat.com>
Reviewed-by: Ernesto Puerta <epuertat@redhat.com>
Reviewed-by: Pere Diaz Bou <pdiazbou@redhat.com>

mgr/dashboard: update landing page e2e tests

ignore the region that can vary on different clusters like charts

fixes: https://tracker.ceph.com/issues/59142
Signed-off-by: Pedro Gonzalez Gomez <pegonzal@redhat.com>

mgr/dashboard: skip Create OSDs step in Cluster expansion

Its to ensure OSDs are not deployed on all hosts because that would make
the host draining impossible

Fixes: https://tracker.ceph.com/issues/59457
Signed-off-by: Nizamudeen A <nia@redhat.com>

Merge PR #50941 into main

* refs/pull/50941/head:
doc/foundation: Update Foundation members for April 2023

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>

Merge pull request #51103 from colemitchell/wip-doc-2023-04-16-radosgw-s3select-formatting-1

doc/radosgw: format part of s3select

Reviewed-by: Zac Dover <zac.dover@proton.me>

doc/radosgw: format part of s3select

Format the first section of s3select. Nothing else is being fixed.

Signed-off-by: Cole Mitchell <cole.mitchell.ceph@gmail.com>

Merge pull request #51099 from zdover23/wip-doc-2023-04-16-dev-cephfs-mirroring-creating-users-command-formatting

doc/dev: format command in cephfs-mirroring

doc/dev: format command in cephfs-mirroring

Correctly format a command in doc/dev/cephfs-mirroring/#creating-users.

Reported by casanlin@init7.net at
https://pad.ceph.com/p/Report_Documentation_Bugs

Signed-off-by: Zac Dover <zac.dover@proton.me>

Merge pull request #51062 from zdover23/wip-doc-2023-04-13-glossary-hybrid-storage

doc/glossary: add "Hybrid Storage"

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>

doc/glossary: add "Hybrid Storage"

Add "Hybrid Storage" to the glossary.

Signed-off-by: Zac Dover <zac.dover@proton.me>

Merge pull request #51091 from cityofships/fix_confval

doc/mgr/prometheus: fix confval reference

doc/mgr/prometheus: fix confval reference

Signed-off-by: Piotr Parczewski <piotr@stackhpc.com>

Merge pull request #51086 from zdover23/wip-doc-2023-04-15-rados-ops-ceph-medic-removal

doc/rados/ops: remove ceph-medic from monitoring

doc/rados/ops: remove ceph-medic from monitoring

Remove mention of ceph-medic from doc/rados/operations/monitoring.rst,
because it is no longer supported.

Signed-off-by: Zac Dover <zac.dover@proton.me>

mgr/dashboard: expose more grafana configs in service form

Show the grafana_port and initial_admin_password in the form but disable
the password field in the edit option

Fixes: https://tracker.ceph.com/issues/58016
Signed-off-by: Nizamudeen A <nia@redhat.com>

Merge pull request #51074 from rhcs-dashboard/expression-changed-e2e

mgr/dashboard: fix cephadm e2e expression changed error

Reviewed-by: Pere Diaz Bou <pdiazbou@redhat.com>

Merge pull request #51055 from ceph/wip-yuriw-release-16.2.12-main

doc: 16.2.12 Release Notes

Reviewed-by: Josh Durgin <jdurgin@redhat.com>
Reviewed-by: Laura Flores <lflores@redhat.com>
Reviewed-by: Guillaume Abrioux <gabrioux@redhat.com>
Reviewed-by: Adam King adking@redhat.com
Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>

Merge PR #50909 into main

* refs/pull/50909/head:
qa/workunit: print the detail commands excuted in the scripts

Reviewed-by: Venky Shankar <vshankar@redhat.com>
Reviewed-by: Dhairya Parmar <dparmar@redhat.com>

mgr/dashboard: delete-ceph-authx

Fixes: https://tracker.ceph.com/issues/59365
Signed-off-by: Pedro Gonzalez Gomez <pegonzal@redhat.com>

Merge pull request #49531 from rhcs-dashboard/fix-rbd-snapshot-creation

mgr/dashboard: Fix rbd snapshot creation

Reviewed-by: VasishtaShastry <NOT@FOUND>
Reviewed-by: Nizamudeen A <nia@redhat.com>
Reviewed-by: Pere Diaz Bou <pdiazbou@redhat.com>
Reviewed-by: sunilangadi2 <NOT@FOUND>

mgr/dashboard: fix cephadm e2e expression changed error

tried to fix this issue from the daemon component sometime ago several
times but it didn't work. So force ignoring the error

Fixes: https://tracker.ceph.com/issues/59444
Signed-off-by: Nizamudeen A <nia@redhat.com>

Merge pull request #50939 from pkalever/logging_improve

librbd: on notify_quiesce() show attempts in a better format

Reviewed-by: Mykola Golub <mgolub@suse.com>
Reviewed-by: N Balachandran <nibalach@redhat.com>
Reviewed-by: Ilya Dryomov <idryomov@gmail.com>

librbd: on notify_quiesce() show attempts in a better format

notify_quiesce() currently shows number of attempts in descending order,
this might be bit confusing to read.

Example: on the very first attempt,
2023-04-04T19:45:56.096+0530 7ff8ba7fc640 10 librbd::ImageWatcher:
  0x7ff898008b30 notify_quiesce: async_request_id=[4151,140705343226832,23] attempts=10

I initially misread the above means 10 attempts where done.

This commit tries to pick the format that is used by
ImageWatcher<I>::handle_payload() and ImageWatcher<I>::notify_async_progress()

Fixes: https://tracker.ceph.com/issues/59379
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>

common/tracer: fix decoding when jaeger tracing is disabled

We aren't currently using jaeger tracing on Windows. The issue is
that Windows hosts (or any other host that doesn't use jaeger)
are experiencing message decoding failures after a recent change [1].

This change updates the tracer encoding so that messages from
non-jaeger hosts may be decoded by services that use jaeger.

[1] https://github.com/ceph/ceph/pull/47457

Signed-off-by: Lucian Petrut <lpetrut@cloudbasesolutions.com>

Merge pull request #51051 from idryomov/wip-59431

qa/suites/rbd: install qemu-utils in addition to qemu-block-extra on Ubuntu

Reviewed-by: Ramana Raja <rraja@redhat.com>

Merge pull request #43546 from ifed01/wip-ifed-snapdiff3-good

cephfs: implement snapdiff

Reviewed-by: Venky Shankar <vshankar@redhat.com>

doc: 16.2.12 Release Notes

Signed-off-by: Yuri Weinstein <yweinste@redhat.com>

Merge pull request #50877 from rhcs-dashboard/add-rbd-mirror-force-promote

mgr/dashboard: rbd-mirror force promotion

Reviewed-by: Nizamudeen A <nia@redhat.com>

Merge pull request #50666 from athanatos/sjust/wip-scrub-interval-pr-49959-2

osd/scrubber: simplify scrub resource handling and interval changes

Merge pull request #51008 from guits/cv-tests-fix-rpm-issue-main

ceph-volume/tests: fix an issue with rpm

qa/suites/rbd: install qemu-utils in addition to qemu-block-extra on Ubuntu

qemu-utils is usually pre-installed but, due to what appears to be
a Ubuntu packaging bug, it's not upgraded when qemu-block-extra is
installed:

  The following NEW packages will be installed:
    qemu-block-extra
  The following packages will be upgraded:
    qemu-system-common qemu-system-data qemu-system-gui qemu-system-x86

However, the version of the block driver must match exactly the version
of the qemu-img tool, so the above leads to:

  $ qemu-img convert -f qcow2 -O raw /home/ubuntu/cephtest/qemu/base.client.0.0.qcow2 rbd:rbd/client.0.0
  Failed to initialize module: /usr/lib/x86_64-linux-gnu/qemu/block-rbd.so
  Note: only modules from the same build can be loaded.
  qemu: module block-block-rbd not found, do you want to install qemu-block-extra package?
  qemu-img: Unknown protocol 'rbd'

Fixes: https://tracker.ceph.com/issues/59431
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>

Merge pull request #50840 from zhscn/segment-nonce-collision

crimson/os/seatore: avoid segment nonce collision

Reviewed-by: Yingxin Cheng <yingxin.cheng@intel.com>

mgr/dashboard: fix rbd mirror snapshot creation

There are two types of snapshots that can be created on a snapshot based mirroring image - Normal Snapshot(same as journal based snapshot) and Nirror Image Snapshot. Till now Dashboard allowed only Mirror image snapshot, this PR intends to enable both the types

Signed-off-by: Aashish Sharma <aasharma@redhat.com>

qa/standalone/scrub/osd-scrub-dump.sh: drop unnecessary primary lookup

1e44d86b2 swapped this to a pg tell command which doesn't actually
need the primary specified.  Drop the now unnecessary lookup.

Signed-off-by: Samuel Just <sjust@redhat.com>

osd/: remove PL::reschedule_scrub, notify scrubber on config/pool change directly

As with on_info_history_change(), we don't need to deal with scrub
scheduling during peering. Once we've gone active, the scrubber itself
would be the origin of any stat changes that could affect scrub
scheduling.  The other possible change vectors would be OSD config
changes or pool config changes.

PG::reschedule_scrub becomes PG::on_scrub_schedule_input_change.  Should
be called in all cases where an input to scrub scheduling changes.

OSD::resched_all_scrubs() calls PG::on_scrub_schedule_input_change
unconditionally
now to deal with changes to osd_scrub_(min|max)_interval.

PG::set_last_[deep_]scrub_stamp now invoke
PG::on_scrub_schedule_input_change directly.

PG::handle_activate_map() now calls PG::on_scrub_schedule_input_change
directly to deal with changes to scrub related pool options.

Signed-off-by: Samuel Just <sjust@redhat.com>

osd/: remove PeeringListener::on_info_history_change()

The only usage of this method was to notify scrub that the pg history
has been updated during split or peering.  That shouldn't be necessary.
Scrub does not schedule itself prior to activation, and we necessarily
must have an authoritative history by that point.

Signed-off-by: Samuel Just <sjust@redhat.com>

osd/scrubber: associate replica state with state machine states

Moves responsibility for owning and resetting replica state to state
machine events.

Signed-off-by: Samuel Just <sjust@redhat.com>

osd/scrub: split on_pg_activate from on_new_interval

Separate and clarify handling of interval termination, pg activation,
and configuration change.

A primary PG now registers with its OSD for scrubbing only on
activation: on_pg_activate() called from PG::on_activate().

When the interval ends, the scrubber is notified via on_interval_change,
which is responsible for cleaning up any active or replica state
associated with scrub.

Configuration changes are still handled by update_scrub_job().

Signed-off-by: Ronen Friedman <rfriedma@redhat.com>
Signed-off-by: Samuel Just <sjust@redhat.com>

osd/scrubber/scrub_machine*: s/ActiveReplica/ReplicaBuildingMap/g

Signed-off-by: Samuel Just <sjust@redhat.com>

scrubber/pg_scrubber: regularize message interval checks

OpRequestRef::sent_epoch can be used to uniformly check incoming
messages against the current interval.  Every caller should do it first
thing before any other processing.

Signed-off-by: Samuel Just <sjust@redhat.com>

osd/scrubber: make PgScrubber::check_interval const

Signed-off-by: Samuel Just <sjust@redhat.com>

osd/scrubber: rename advance_token to reset_replica_state

Also removes misleading comment -- because primaries may
freely reset scrub, there may well be valid scrub state.

Signed-off-by: Samuel Just <sjust@redhat.com>

osd/scrubber: move m_remote_osd_resource.reset() into advance_token()

Signed-off-by: Samuel Just <sjust@redhat.com>

osd/scrubber: simplify existing reservation case in handle_scrub_reserve_request

First, the is_stale() case should be impossible.  This patch leaves an
error message rather than an assert.

Second, this patch removes the special handling for receiving a reservation
when m_remote_osd_resource is already populated.  The next patch will exploit
the fact that all calls to m_remote_osd_resource.reset() and advance_token()
occur together.

Signed-off-by: Samuel Just <sjust@redhat.com>

osd/scrubber: remove unused PgScrubber::send_full_reset

Signed-off-by: Samuel Just <sjust@redhat.com>

osd/scrub: remove PgScrubber::unregister_from_osd()

and inline its functionality into its sole caller
(PgScrubber::rm_from_osd_scrubbing())

Signed-off-by: Ronen Friedman <rfriedma@redhat.com>

osd/: misc scrub related formatting fixes -- no functional changes

Signed-off-by: Ronen Friedman <rfriedma@redhat.com>

Merge pull request #50993 from zdover23/wip-doc-2023-04-11-rados-operations-monitoring-3-of-3

doc/rados/operations: edit monitoring.rst

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>

Merge pull request #50245 from baergj/bluefs-perf-stats-write-count

os/bluestore: Add bluefs write op count metrics.

Reviewed-by: Igor Fedotov <ifedotov@suse.com>

doc/rados/operations: edit monitoring.rst

Line-edit the final third of doc/rados/operations/monitoring.rst.

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

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

Co-authored-by: Anthony D'Atri <anthony.datri@gmail.com>
Signed-off-by: Zac Dover <zac.dover@proton.me>

rgw/aio: fix recursion from tracing changes

Fixes: https://tracker.ceph.com/issues/59423
Signed-off-by: Casey Bodley <cbodley@redhat.com>

mgr/dashboard: rbd-mirror force promotion

resolves: https://tracker.ceph.com/issues/59327
Signed-off-by: Pedro Gonzalez Gomez <pegonzal@redhat.com>

ceph-volume/tests: fix an issue with rpm

Typical error seen in the CI:

```
error: /var/cache/dnf/baseos-00fe51d07def85f0/packages/kernel-core-4.18.0-483.el8.x86_64.rpm: signature hdr data: BAD, no. of bytes(459772) out of range
```

Upgrading `rpm` fixes this issue.

Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>

Merge pull request #50885 from rhcs-dashboard/evict-fs

mgr/dashboard: fix eviction of all FS clients

Reviewed-by: Aashish Sharma <aasharma@redhat.com>
Reviewed-by: Nizamudeen A <nia@redhat.com>

Merge pull request #50646 from rishabh-d-dave/doc-fs-subcmds

doc/cephfs: add missing "ceph fs" commands to man page

Reviewed-by: Venky Shankar <vshankar@redhat.com>

Merge pull request #50628 from jianwei1216/support_bluestore_db_wal_physical_disk

vstart: support bluestore db/wal physical disk

crimson/os/seastore: check the type of segment head and segment tail during mount

Signed-off-by: Zhang Song <zhangsong325@gmail.com>

crimson/os/seastore: avoid segment nonce collision

Signed-off-by: Zhang Song <zhangsong325@gmail.com>

Merge pull request #50867 from rhcs-dashboard/skip-osds-expand-cluster

mgr/dashboard: fix create osd default selected as recommended not working

Reviewed-by: Aashish Sharma <aasharma@redhat.com>
Reviewed-by: Pere Diaz Bou <pdiazbou@redhat.com>

Merge PR #50716 into main

* refs/pull/50716/head:
doc: updating the doc with chamges related to delay option in cephfs-top
cephfs-top: fixing the -d [--delay] option in cephfs-top

Reviewed-by: Venky Shankar <vshankar@redhat.com>
Reviewed-by: Jos Collin <jcollin@redhat.com>

Merge pull request #50839 from zhscn/io-order

crimson/os/seastore: ensure transaction commit order

Reviewed-by: Yingxin Cheng <yingxin.cheng@intel.com>

test/libcephfs: add cases for last dir testing.

Signed-off-by: Igor Fedotov <igor.fedotov@croit.io>

Merge pull request #50185 from ethanwu-syno/bluefs_tracker_56210

os/bluestore/bluefs: fix dir_link might add link that already exists in compact log

Reviewed-by: Igor Fedotov <ifedotov@suse.com>
Reviewed-by: Adam Kupczyk <akupczyk@redhat.com>

Merge pull request #50699 from yaarith/telemetry-leaderboard-doc

mgr/telemetry: add leaderboard description and documentation

Reviewed-by: Zac Dover <zac.dover@proton.me>
Reviewed-by: Laura Flores <lflores@redhat.com>
Reviewed-by: Nizamudeen A <nia@redhat.com>

Merge pull request #50721 from ceph/wip-yuriw-release-17.2.6-main

doc: 17.2.6 Release Notes

doc: 17.2.6 Release Notes

Signed-off-by: Yuri Weinstein <yweinste@redhat.com>
Signed-off-by: Laura Flores <lflores@redhat.com>

rgw/lua: fix CopyFrom crash

Fixes: https://tracker.ceph.com/issues/59381
Signed-off-by: Yuval Lifshitz <ylifshit@redhat.com>

include,win: move uid_t/gid_t declarations to win32/fs_compat.h

Signed-off-by: Igor Fedotov <igor.fedotov@croit.io>
Signed-off-by: Lucian Petrut <petrutlucian94@gmail.com>

pybind/cephfs: add snapdiff api binding

Signed-off-by: Igor Fedotov <ifedotov@croit.io>

test/libcephfs: add readdir_snapdiff

Signed-off-by: Denis Barahtanov <denis.barahtanov@croit.io>
Signed-off-by: Igor Fedotov <igor.fedotov@croit.io>

cephfs: imlplement readdir_snapdiff API

Signed-off-by: Denis Barahtanov <denis.barahtanov@croit.io>
Signed-off-by: Igor Fedotov <igor.fedotov@croit.io>

Merge pull request #50829 from zdover23/wip-doc-2023-04-02-rados-operations-health-checks-6-of-x

doc/rados/ops: edit health-checks.rst (6 of x)

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>

doc/rados/ops: edit health-checks.rst (6 of x)

Edit docs/rados/operations/health-checks.rst (6 of x).

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

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

Co-authored-by: Anthony D'Atri <anthony.datri@gmail.com>
Signed-off-by: Zac Dover <zac.dover@proton.me>

Merge pull request #50828 from zdover23/wip-doc-2023-04-02-rados-operations-health-checks-5-of-x

doc/rados/ops: edit health checks.rst (5 of x)

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>

doc/rados/ops: edit health checks.rst (5 of x)

Edit docs/rados/operations/health-checks.rst (5 of x).

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

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

Co-authored-by: Anthony D'Atri <anthony.datri@gmail.com>
Signed-off-by: Zac Dover <zac.dover@proton.me>

Merge PR #47399 into main

* refs/pull/47399/head:
mds: wait the linkmerge/migrate to finish after unlink
mds: remove false is_auth() check for remote_dn

Reviewed-by: Ramana Raja <rraja@redhat.com>
Reviewed-by: Venky Shankar <vshankar@redhat.com>
Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
Reviewed-by: Rishabh Dave <ridave@redhat.com>

rgw/kafka: failed to reconnect to broker after idle timeout

This is a regression due to: 8713c3169c0f9df1d2fc23ff2b82ede1e25be282

To reproduce, define a topic and a notification pointing to a kafka broker.
Stop the broker and upload objects to the cluster, wait for 30 seconds, and then start the broker.

Fixes: https://tracker.ceph.com/issues/59383?next_issue_id=59381
Signed-off-by: Yuval Lifshitz <ylifshit@redhat.com>

Merge pull request #50958 from zdover23/wip-doc-2023-04-09-rados-ops-monitoring-osd-pg-hyphen

doc/rados/ops: add hyphen to mon-osd-pg.rst

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>

doc/rados/ops: add hyphen to mon-osd-pg.rst

Remove confusing parentheses and add a clearer (as compared to the
parentheses) hyphen (actually an em-dash, or at least it is intended
to be an em-dash) to doc/rados/operations/monitoring-osd-pg.rst

Co-authored-by: Anthony D'Atri <anthony.datri@gmail.com>
Signed-off-by: Zac Dover <zac.dover@proton.me>

Merge pull request #50827 from zdover23/wip-doc-2023-04-02-rados-operations-health-checks-4-of-x

doc/rados/ops: edit health-checks.rst (4 of x)

Reveiwed-by: Anthony D'Atri <anthony.datri@gmail.com>

Merge pull request #50826 from zdover23/wip-doc-2023-04-02-rados-operations-health-checks-3-of-x

doc/rados/ops: edit health-checks.rst (3 of x)

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>

Merge pull request #50907 from zdover23/wip-doc-2023-04-06-rados-configuration-auth-config-ref

doc/rados/config: edit auth-config-ref

Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>