cephadm: remove call to get_container from iscsi tcmu runner method

Remove the call to get_container from the iscsi class's
get_tcmu_runner_container method, replacing it with a call to the
daemon_to_container function. While they're roughly equivalent -
we can consider get_container legacy and only for use in cephadm.py
where daemon_to_container is more general.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: rework get_container in terms of daemon_to_container

With get_container basically equivalent to daemon_to_container except
one creates a ContainerDaemonForm and the other requires one, we
can rework get_container to act as a thin wrapper around
daemon_to_container.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: switch custom container class to use daemon_to_container function

Switch the custom container daemon type class to call
daemon_to_container instead of get_container in the class's container
method. This breaks a dependency loop between the class and the
get_container function hopefully enabling future clean ups and code
moves.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: switch tracing class to use daemon_to_container function

Switch the tracing daemon type class to call daemon_to_container instead of
get_container in the class's container method. This breaks a dependency loop
between the class and the get_container function hopefully enabling future
clean ups and code moves.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: switch keepalived class to use daemon_to_container function

Switch the keepalived daemon type class to call daemon_to_container instead of
get_container in the class's container method. This breaks a dependency loop
between the class and the get_container function hopefully enabling future
clean ups and code moves.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: switch haproxy class to use daemon_to_container function

Switch the haproxy daemon type class to call daemon_to_container instead of
get_container in the class's container method. This breaks a dependency loop
between the class and the get_container function hopefully enabling future
clean ups and code moves.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: switch ceph exporter class to use daemon_to_container function

Switch the ceph exporter daemon type class to call daemon_to_container
instead of get_container in the class's container method. This breaks a
dependency loop between the class and the get_container function
hopefully enabling future clean ups and code moves.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: switch nvmeof class to use daemon_to_container function

Switch the nvmeof daemon type class to call daemon_to_container instead of
get_container in the class's container method. This breaks a dependency loop
between the class and the get_container function hopefully enabling future
clean ups and code moves.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: switch iscsi class to use daemon_to_container function

Switch the iscsi daemon type class to call daemon_to_container instead of
get_container in the class's container method. This breaks a dependency loop
between the class and the get_container function hopefully enabling future
clean ups and code moves.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: switch nfs class to use daemon_to_container function

Switch the nfs daemon type class to call daemon_to_container instead of
get_container in the class's container method. This breaks a dependency loop
between the class and the get_container function hopefully enabling future
clean ups and code moves.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: switch monitoring class to use daemon_to_container function

Switch the monitoring daemon type class to call daemon_to_container instead of
get_container in the class's container method. This breaks a dependency loop
between the class and the get_container function hopefully enabling future
clean ups and code moves.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: switch snmp class to use daemon_to_container function

Switch the snmp daemon type class to call daemon_to_container instead of
get_container in the class's container method. This breaks a dependency loop
between the class and the get_container function hopefully enabling future
clean ups and code moves.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: switch ceph class to use daemon_to_container function

Switch the ceph daemon type class to call daemon_to_container instead of
get_container in the class's container method. This breaks a dependency loop
between the class and the get_container function hopefully enabling future
clean ups and code moves.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: add daemon_to_container utility function

Add the function daemon_to_container in the container_daemon_form.py
function. This function is roughly equivalent to `get_container` in
cephadm.py but entirely relies on subclasses implementing
ContainerDaemonForm methods and avoids special casing of daemon
types.

It duplicates the functionality provided by functions in cephadm.py with
regards to special podman specific behaviors. I feel that this is a
minor issue because due to recent refactoring those functions are now
the one- or two-liners that are copied here. I gated them behind
arguments `auto_podman_mounts` and `auto_podman_args` that default to
true in order to make the function more reusable - however I don't need
them that way and if it seems like extra complexity for no win, we can
remove them later.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: convert calls to get_container_{mounts,binds} to method calls

In get_container, convert calls to get_container_binds and
get_container_mounts to direct uses of the ContainerDaemonForm customize
methods, avoiding constructing said ContainerDaemonForm subclasses
redundantly.
This is all done in one pass because the _update_podman_mounts function
is called by get_container_mounts and trying to convert each type at a
time would result in extra calls to that function.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: handle the case where ctx may not have a config attribute

This prevents a test failure in future refactoring changes.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: use CustomContainer.create classmethod like most other blocks

Use the CustomContainer.create classmethod like most other blocks of
code in the get_container function.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

Merge pull request #54486 from pdvian/wip-app-release

PendingReleaseNotes: Add note for POOL_APP_NOT_ENABLED

Reviewed-by: Neha Ojha <nojha@redhat.com>
Reviewed-by: Laura Flores <lflores@redhat.com>
Reviewed-by: Ilya Dryomov <idryomov@gmail.com>

Merge pull request #53650 from rhcs-dashboard/fix-62969-main

mgr/dashboard: Show the OSDs Out and Down panels as red whenever an OSD is in Out or Down state in Ceph Cluster grafana dashboard

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

Merge pull request #54490 from ceph/nvmeof-controllerid-spec

mgr/cephadm: Expose nvmeof gateway configuration parameters through specifications

Reviewed-by: Adam King <adking@redhat.com>
Reviewed-by: Aviv Caro <Aviv.Caro@ibm.com>

Merge pull request #54487 from phlogistonjohn/jjm-cephadm-zipimporter-wrkaround

cephadm: workaround issues running cephadm with relative path

Reviewed-by: Adam King <adking@redhat.com>
Reviewed-by: Michael Fritch <mfritch@suse.com>

Merge pull request #54403 from phlogistonjohn/jjm-cephadm-reorg-ctr-setup-methods

cephadm: reorganize the remaining container setup methods

Reviewed-by: Adam King <adking@redhat.com>

Merge pull request #54173 from phlogistonjohn/jjm-cephadm-jinja-2

cephadm: expand support for dependencies bundled into the zipapp

Reviewed-by: Adam King <adking@redhat.com>

Merge pull request #53803 from adk3798/regex-host-pattern

mgr/cephadm: support for regex based host patterns

Reviewed-by: John Mulligan <jmulligan@redhat.com>

Merge pull request #54529 from cbodley/wip-osd-scrubber-format-to

osd/scrubber: fix ambiguous call to format_to()

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

PendingReleaseNotes: Add note for POOL_APP_NOT_ENABLED

Adds release notes for the fix added in #47560

Signed-off-by: Prashant D <pdhange@redhat.com>

osd/scrubber: fix ambiguous call to format_to()

new call to format_to() is ambiguous between std and fmt namespaces:

ceph/src/osd/scrubber/pg_scrubber.cc:654:26:   required from here
ceph/src/osd/scrubber/scrub_job.h:241:21: error: call of overloaded ‘format_to(fmt::v9::basic_format_context<fmt::v9::appender, char>::iterator, const char [60], const double&, double, const double&, const double&, const double&, const double&, const bool&)’ is ambiguous
...
ceph/src/fmt/include/fmt/core.h:3233:17: note: candidate: ‘OutputIt fmt::v9::format_to(OutputIt, format_string<T ...>, T&& ...) [with OutputIt = appender; T = {const double&, double, const double&, const double&, const double&, const double&, const bool&}; typename std::enable_if<detail::is_output_iterator<OutputIt, char>::value, int>::type <anonymous> = 0; format_string<T ...> = basic_format_string<char, const double&, double, const double&, const double&, const double&, const double&, const bool&>]’
 3233 | FMT_INLINE auto format_to(OutputIt out, format_string<T...> fmt, T&&... args)
      |                 ^~~~~~~~~
...
/usr/include/c++/13/format:3824:5: note: candidate: ‘_Out std::format_to(_Out, format_string<_Args ...>, _Args&& ...) [with _Out = fmt::v9::appender; _Args = {const double&, double, const double&, const double&, const double&, const double&, const bool&}; format_string<_Args ...> = basic_format_string<char, const double&, double, const double&, const double&, const double&, const double&, const bool&>]’
 3824 |     format_to(_Out __out, format_string<_Args...> __fmt, _Args&&... __args)
      |     ^~~~~~~~~

gcc (GCC) 13.2.1 20231011 (Red Hat 13.2.1-4)

Signed-off-by: Casey Bodley <cbodley@redhat.com>

Merge pull request #54171 from smanjara/wip-fix-archive-version-status

rgw/multisite: set versioning status in archive zone upon bucket creation

Reviewed-by: Casey Bodley <cbodley@redhat.com>

Merge pull request #54514 from guits/fix-cv-raw-list

ceph-volume: fix a regression in `raw list`

cephadm: workaround issues running cephadm with relative path

Implement a workaround for the jinja2 package loader not correctly
finding a template inside the cephadmlib package when run as
a zipapp. See docstring in the shim class for more details.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

ceph-volume: fix a regression in `raw list`

'ceph-volume raw list' is broken for a specific use case (rook).

rook copies devices from /dev/ to /mnt for specific/internal needs.

when ceph-volume raw list is passed a device from /mnt then
ceph-volume ignores it and return an empty dict.
That prevent rook from creating OSDs properly.

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

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

Merge pull request #54363 from ronen-fr/wip-rf-force-scrub

 osd/scrub: modify "scrub" & "deep_scrub" ASOK commands

Reviewed-by: Nitzan Mordechai <nmordech@redhat.com>
Reviewed-by: Samuel Just <sjust@redhat.com>
Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>

Merge pull request #54489 from Suyashd999/main

qa/suites/rbd: Cleanup of MIRROR_IMAGE_MODE

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

Merge PR #54471 into main

* refs/pull/54471/head:
qa: use correct imports to resolve fuse_mount and kernel_mount

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

tests/standalone: fix scrub-related tests following command changes

Using
ceph tell $pgid [deep]-scrub
to initiate an 'operator initiated' scrub, and
ceph tell $pgid schedule[-deep]-scrub
for causing a 'periodic scrub' to be scheduled.

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

mgr/cephadm: Expose nvmeof gateway configuration parameters through specifications

- min_controller_id, max_controller_id: Enable the specification of minimum and maximum controller IDs utilized by the SPDK.
  Having distinct controller IDs is vital for configuring multipath setups.
- enable_spdk_discovery_controller: Manage whether the SPDK or ceph-nvmeof discovery service is employed. The default value is set to False.

Signed-off-by: Alexander Indenbaum <aindenba@redhat.com>

Merge pull request #53047 from xxhdx1985126/wip-crimson-interrupt-exception

crimson/common/interruptible_future: deal with exceptions thrown from seastar::future::get() and seastar::thread::yield()

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

Merge pull request #54497 from zdover23/wip-doc-2023-11-14-glossary-quorum

doc/glossary: add "Quorum" to glossary

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

doc/glossary: add "Quorum" to glossary

Add the term "Quorum" to the glossary and link to the part of
architecture.rst concerning Monitors. The sticky header at the top of
the docs.ceph.com website gets in the way of the location linked to in
this commit, but fatigue and disgust prevent me from spending time today
trial-and-erroring my way through the hostile and ill-documented
wilderness of scroll-margin so that the link goes where it should.

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

Merge PR #54259 into main

* refs/pull/54259/head:
mds,client: update the oldest_client_tid via the renew caps
mds: add trim_completed_request_list() helper

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

cephadm: eliminate the _update_pids_limit function

Eliminate the _update_pids_limit function by moving the logic into the
various daemon type classes `customize_container_args` functions allow
each class to decide if it needs to run with unlimited-pids or not.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: use class methods when setting up ceph based daemon containers

Now that the ceph and ceph exporter classes have all the necessary
methods, rework the get_container function to use those class methods
in a common manner.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: add container envs methods to ceph and ceph exporter classes

These, currently unused, methods will be used in a subsequent change to
move logic for setting up containers out of the get_containers function
into the daemon type classes in a common way.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: add methods to ceph daemon type class

Add unused methods to the ceph class in order to prepare for moving
the logic for setting up the various ceph deaemon types to be
handled by the class.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: add more ContainerDeamonForm methods to the monitoring class

Add methods customize_{container_args,process_args} & default_entrypoint
to the monitoring daemon type class. Use those methods in the
get_container function.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: add more ContainerDeamonForm methods to the tracing class

Add methods customize_{container_envs,process_args} & default_entrypoint
to the tracing daemon type class. Use those methods in the get_container
function.
This change tries to adapt to, but not fix, the rather strange
set_configuration static method that side-effects the class's(!)
property.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: add more ContainerDeamonForm methods to the nfs class

Add methods customize_{container_envs,process_args} & default_entrypoint
to the nfs daemon type class. Use those methods in the get_container
function.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: add more ContainerDeamonForm methods to the ceph exporter class

Add methods customize_process_args & default_entrypoint to the ceph
exporter daemon type class. Use those methods in the get_container
function.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: add more ContainerDeamonForm methods to the haproxy class

Add methods customize_{container_args,process_args} to the haproxy
daemon type class. Use those methods in the get_container function.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: add more ContainerDeamonForm methods to the keepalived class

Add methods customize_{container_args,container_envs} to the keepalived
daemon type class. Use those methods in the get_container function.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: add more ContainerDeamonForm methods to the nvmeof class

Add method customize_container_args to the nvmeof daemon type class. Use
that method in the get_container function.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: add more ContainerDeamonForm methods to the iscsi class

Add method default_entrypoint to the iscsi daemon type class. Use that
method in the get_container function.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: add more ContainerDeamonForm methods to the custom container class

Add methods customize_{process_args,container_envs,container_args} and
default_entrypoint to the custom container daemon type class. Use those
methods in the get_container function.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: add more ContainerDeamonForm methods to the snmp class

Add methods customize_{container,process}_args to the snmp gateway
daemon type class. Use those methods in the get_container function.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: update additional container daemon form base class methods

Fix the function signature for the previously added stub
customize_container_args.
Add methods for customize_process_args and customize_container_envs.
The "process args" are args passed to the daemon process / entrypoint.
Container args are passed to the container engine.
Add a default_entrypoint method to return the default entrypoint for
a particular daemon-in-container.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

doc/cephadm: document regex based host pattern

Signed-off-by: Adam King <adking@redhat.com>

mgr/cephadm: support for regex based host patterns

For example, with hosts vm-00, vm-01, and vm-02
I was able to provide the placement

service_type: node-exporter
service_name: node-exporter
placement:
  host_pattern:
    pattern: vm-00|vm-02
    pattern_type: regex

and it placed the node-exporter daemons on vm-00
and vm-02 but not vm-01. Obviously there are more
advanced scenarios that justify this than listing
two hosts, but using "|" as an OR like that is an
example of something you can't do with the fnmatch
version of the host pattern

Signed-off-by: Adam King <adking@redhat.com>

Merge pull request #54401 from adk3798/osd-rm-stop-reweight

mgr/cephadm: fix reweighting of OSD when OSD removal is stopped

Reviewed-by: Michael Fritch <mfritch@suse.com>

Merge pull request #54399 from phlogistonjohn/jjm-cephadm-reorg-ctr-mounts

cephadm: reorganize container mounts functions

Reviewed-by: Adam King <adking@redhat.com>

Merge pull request #54496 from zdover23/wip-doc-2023-11-14-rados-troubleshooting-mon-initial-rewrite

doc/rados: edit "troubleshooting-mon"

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

Merge pull request #54499 from yuvalif/wip-yuval-63530

doc/rgw/notifications: specify which event types are enabled by default

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

doc/rados: edit "troubleshooting-mon"

Edit the text in the "Initial Troubleshooting" section of
doc/rados/troubleshooting/troubleshooting-mon.rst.

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

doc/rgw/notifications: specify which event types are enabled by default

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

Merge pull request #54431 from Matan-B/wip-matanb-crimson-extend-snaptrimsubevent-lifetime

crimson/osd/osd_operations/snaptrim_event: fix lifetime management

Reviewed-by: Samuel Just <sjust@redhat.com>
Reviewed-by: Yingxin Cheng <yingxin.cheng@intel.com>
Reviewed-by: Radosław Zarzyński <rzarzyns@redhat.com>

Merge pull request #54416 from Matan-B/wip-matanb-crimson-extend-pg-lifetime

crimson/osd/pg: extend pg lifetime on snap_trimq iteration

Reviewed-by: Samuel Just <sjust@redhat.com>
Reviewed-by: Yingxin Cheng <yingxin.cheng@intel.com>
Reviewed-by: Radosław Zarzyński <rzarzyns@redhat.com>

qa/suites/rbd: Cleanup of MIRROR_IMAGE_MODE

Fixes: https://tracker.ceph.com/issues/63431
Signed-off-by: Suyash Dongre <suyashd999@gmail.com>

Merge pull request #54375 from ajarr/wip-63421

pybind/mgr: remove __del__() of mgr modules

Reviewed-by: Wido den Hollander <wido@42on.com>
Reviewed-by: Venky Shankar <vshankar@redhat.com>
Reviewed-by: Ilya Dryomov <idryomov@gmail.com>

Merge pull request #54479 from anthonyeleven/anthonyeleven/ceph

doc/start: fix typo in hardware-recommendations.rst

pybind/mgr: remove __del__() of mgr modules

It's strongly recommended for objects that have references to
external resources (e.g., files) to explicitly release them.
Python doesn't guarantee garbage collection of objects and hence
doesn't guarantee freeing of external resources that occur on
garbage collection.

The __del__() methods in the python mgr modules may not even be
called since garbage collection of objects is not guaranteed in python.
And some of the __del__() methods try to cleanup that seem redundant.

- In volumes/module.py, vc.shutdown() is called in Module.shutdown().
  No need to call it again in Module.__del__()

- In telegraf/basesocket.py, BaseSocker.close() is called in
  BaseSocket.__exit__(). No need to call it again in
  BaseSocket.__del__().

- In mgr_module.py, MgrModuleLoggingMixin._unconfigure_logging() is
  called in MgrModule.__init__() and MgrStandbyModule.__init__(). No
  need to call it in MgrModule.__del__() and
  MgrStandbyModule.__del__().|

- In dashboard/services/cephfs.py, the libcephfs mount is not
  shutdown explicitly by the mgr module. However, the cython libcephfs
  bindings has a LibCephFS.__dealloc__() finalizer method that calls
  LibCephFS.shutdown(). This should unmount and cleanup the ceph mount
  handle.

Remove the __del__() of the python mgr modules.

Fixes: https://tracker.ceph.com/issues/63421
Signed-off-by: Ramana Raja <rraja@redhat.com>

crimson/osd/pg: PG::on_active_actmap() indention

Signed-off-by: Matan Breizman <mbreizma@redhat.com>

crimson/osd/pg: extend pg lifetime on snap_trimq iteration

Beacuse the loop's returned future is ignored,
we should cover the scenario where the pg is removed and the
snap_trimq iteration didn't complete yet.

Fixes: https://tracker.ceph.com/issues/61653
Signed-off-by: Matan Breizman <mbreizma@redhat.com>

Merge pull request #53655 from ifed01/wip-ifed-peering-cleanup

osd: getting rid off redundant std::map/set::count() calls.

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

Merge pull request #53597 from ifed01/wip-ifed-bluefs-perf-counters

os/bluestore: add more bluefs perf counters

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

Merge pull request #53530 from ifed01/wip-ifed-fast-shutdown-assert

osd: do not assert on fast shutdown timeout

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

Merge pull request #52855 from Matan-B/wip-matanb-monstore-osd_snap

tools/ceph_monstore_tool: additional commands and doc

Reviewed-by: Nitzan Mordechai <nmordech@redhat.com>

Merge pull request #52406 from Matan-B/wip-matanb-mon-44069-commits

mon/Config*: Cleanups and ConfigMap unit test

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

doc/start: fix typo in hardware-recommendations.rst

Signed-off-by: Anthony D'Atri <anthonyeleven@users.noreply.github.com>

Merge pull request #54473 from zdover23/wip-doc-2023-11-13-rados-troubleshooting-mon-initial

doc/rados: format "initial troubleshooting"

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

doc/rados: format "initial troubleshooting"

Format the steps in the "Initial Troubleshooting" section of
doc/rados/troubleshooting/troubleshooting-mon.rst. A near-future PR (not
this one) will add context to this section and explain that the steps
described here are the first steps that you should undertake when you
determine that you have an unresponsive or down Monitor. This PR is
merely for formatting.

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

qa: use correct imports to resolve fuse_mount and kernel_mount

Fixes: https://tracker.ceph.com/issues/62706
Signed-off-by: Milind Changire <mchangir@redhat.com>

Merge PR #54338 into main

* refs/pull/54338/head:
script: enable checkout of specific sha1 as base

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

crimson/osd/osd_operations/snaptrim_event: fix lifetime management

The operation's id and future returned when starting SnapTrimObjSubEvent
is emplaced into subop_blocker.
Later on, we await the completion of all the started operations futures.
Before this patch, we only stored the op id in the subop_blocker vector
which allowed `op` to go out of scope and lose all its references
(and get deleted) before exiting.

Storing the operation as a reference instead of the id
will maintain the SnapTrimObjSubEvent operation lifetime.

Fixes: https://tracker.ceph.com/issues/63299
Signed-off-by: Matan Breizman <mbreizma@redhat.com>

Merge pull request #54457 from zdover23/wip-doc-2023-11-12-rados-config-ceph-conf-config-ceph

doc/config: edit "ceph-conf.rst"

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

Merge pull request #54458 from zdover23/wip-doc-2023-11-12-rados-troubleshooting-mon-parallel-structure

doc/rados: parallelize t-mon headings

doc/rados: parallelize t-mon headings

Give parallel structure to the questions in the Q&A section of the "The
Cluster Has Quorum But At Least One Monitor Is Down" subsection of the
"Most Common Monitor Issues" section of
doc/rados/troubleshooting/troubleshooting-mon.rst.

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

doc/config: edit "ceph-conf.rst"

Edit the first section of doc/rados/configuration/ceph-conf.rst.
Initially I just wanted to change "series" to "set", but once I got my
hands dirty I ended up simplifying some sentences.

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

cephadm: use customize_container_mounts in get_container_mounts

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: change ceph & exporter to customize_container_mounts method

Unlike the other types Ceph and CephExporter share the underlying
method. There was no other use of get_container_mounts on the class
so it could be converted to be customize_container_mounts.
Because there's an extra arg that passes from get_container_mounts
top-level function to Ceph.get_ceph_mounts, that function was not
changed.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: add customize_container_mounts method to monitoring class

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: add customize_container_mounts method to nfs class

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: add customize_container_mounts method to haproxy class

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: add customize_container_mounts method to keepalived class

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: add customize_container_mounts method to nvmeof class

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: add customize_container_mounts method to iscsi class

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: add customize_container_mounts method to custom container

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: only call get_container_mounts for types that have mounts

This helps clarify what types will eventually have to be converted
away from calling get_container_mounts directly and to using common
methods.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

cephadm: move volume mounts assignment to a variable

Move the call to get_container_mounts out of the function call block.
This will aid with the next refactoring steps, so that the uses
of get_container_mounts can be brought into the get_container call
directly.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

Merge pull request #54452 from zdover23/wip-doc-2023-11-11-rados-troubleshooting-mon-common-issues-5-of-x

doc/rados: edit t-mon "common issues" (5 of x)

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

osd/scrub: modify (deep)scrub ASOK commands

Fix all instances of 'deep_scrub' to the documented format (deep-scrub);

Make all formats of the documented command ('ceph tell $pg
[deep-]scrub', and the old 'ceph pg scrub $pgid') trigger an
operator-priority scrub;

Introduce a dev/tests-only format for the required functionality
of "trigger a scrub by modifying the timestamps".

Change some implementation details.

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

doc/rados: edit t-mon "common issues" (5 of x)

Edit the fifth part of the section "Most Common Monitor Issues" in
doc/rados/troubleshooting/troubleshooting-mon.rst.

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

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