doc/mgr: edit grammar and formatting of rgw.rst

Improve the grammar and correct the formatting of the "Upgrading root ca
certificates" procedure that was added to the documentation in https://github.com/ceph/ceph/pull/61867

Fixes: https://tracker.ceph.com/issues/70014
Signed-off-by: Zac Dover <zac.dover@proton.me>

doc/mgr: Add root CA cert instructions to rgw.rst

Add documentation for adding fs_id in root_ca_cert upgrade path

Fixes: https://tracker.ceph.com/issues/70014
Signed-off-by: Anuradha Gadge <anuradha.gadge@ibm.com>

Merge pull request #61836 from myoungwon/fix-crimson-osd-invalid-argument-print

crimson/osd: remove invalid an argument when printing log

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

Merge pull request #61469 from ronen-fr/wip-rf-assert-perf

common/perf-counters: replace 'assert' with 'ceph_assert'

Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>

Merge pull request #61606 from rhcs-dashboard/create-storage-class

mgr/dashboard: RGW - Create Storage Class

Reviewed-by: Afreen Misbah <afreen@ibm.com>

Merge pull request #61793 from MaxKellermann/rados__includes

Add missing librbd includes

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

Merge pull request #61814 from gbregman/main

mgr/cephadm/nvmeof: Add QOS timeslice field to NVMeOF configuration

Merge pull request #47189 from pdvian/wip-crush-df

crush/CrushWrapper: Merge weights if rule contains multiple take

Reviewed-by: Kamoltat Sirivadhna <ksirivad@redhat.com>
Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>

Merge pull request #61752 from afreen23/wip-nvme-ns

Add namespace counter in nvmeof namepsaces

Reviewed-by: Afreen Misbah <afreen@ibm.com>

Merge PR #60983 into main

* refs/pull/60983/head:
script: ceph-backport no longer launches firefox

Reviewed-by: Patrick Donnelly <pdonnell@ibm.com>
Reviewed-by: John Mulligan <jmulligan@redhat.com>

Merge PR #61552 into main

* refs/pull/61552/head:
script/ceph-debug-docker.sh: fix manifest unknown error

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

Merge PR #61776 into main

* refs/pull/61776/head:
ceph-backport.sh: add squid to known milestones

Reviewed-by: Patrick Donnelly <pdonnell@ibm.com>
Reviewed-by: John Mulligan <jmulligan@redhat.com>

Merge PR #61813 into main

* refs/pull/61813/head:
common: add missing mds op names

Reviewed-by: Greg Farnum <gfarnum@redhat.com>

Merge pull request #61785 from ceph/cve-34749-fix

doc: Upgrade and unpin some python versions

mgr/dashboard: RGW - Create Storage Class

Fixes: https://tracker.ceph.com/issues/69750
Signed-off-by: Dnyaneshwari Talwekar <dtalweka@redhat.com>

mgr/cephadm/nvmeof: Add QOS timeslice field to NVMeOF configuration
Fixes https://tracker.ceph.com/issues/69952

Signed-off-by: Gil Bregman <gbregman@il.ibm.com>

Merge pull request #61702 from MaxKellermann/bluestore_includes

os/bluestore: include cleanup

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

Merge pull request #61808 from ronen-fr/wip-rf-store-ids

osd/scrub: cleanup scrub-store IDs generation

Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>

Merge pull request #61588 from rhcs-dashboard/smb-share-form-create

mgr/dashboard: SMB Create Share

Reviewed-by: Afreen Misbah <afreen@ibm.com>

mgr/dashboard: Add namespace counter in nvmeof namepsaces

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

Signed-off-by: Afreen Misbah <afreen@ibm.com>

mgr/dashboard: fix image size in nvmeof namespace create/update api

- Different name is used in POST and PATCH for `rbd_image_size`
- Using same name in both requests
- fixing typing issues in frontend

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

Signed-off-by: Afreen Misbah <afreen@ibm.com>

Merge pull request #61779 from yuvalif/wip-yuval-logging-fields-doc

doc/rgw/logging: update field description

Reviewed-By: Anthony D'Atri <anthony.datri@gmail.com>, Ali Masarwe <ali.masarwa@ibm.com>

librbd: add missing includes

Signed-off-by: Max Kellermann <max.kellermann@ionos.com>

rbd_replay: add missing include

Signed-off-by: Max Kellermann <max.kellermann@ionos.com>

SimpleRADOSStriper: add missing include

Signed-off-by: Max Kellermann <max.kellermann@ionos.com>

librados: add missing includes

Signed-off-by: Max Kellermann <max.kellermann@ionos.com>

Merge pull request #61597 from Hezko/nvmf-update-route

mgr/dashboard: namespace update route robustness

Merge pull request #61835 from nbalacha/wip-nbalacha-69978

rbd-mirror: fix possible recursive lock of ImageReplayer::m_lock

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

crimson/osd: remove invalid argurment when printing log

Signed-off-by: Myoungwon Oh <myoungwon.oh@samsung.com>

osd/scrub: clean-up scrub-store IDs generation

The Scrub Store IDs (used to identify OMAP entries used for storing
object-specific error data) are generated by attaching some prefix to
the text representation of virtual (invented) hobjects.

This commit cleans up the code by moving the name generation
process (creating temporary un-needed objects, then
extracting the to_str() representation of their identity)
into separate functions.

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

mgr/dashboard: ns update route robustness

improve namespace update endpoint

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

Signed-off-by: Tomer Haskalovitch <il033030@Tomers-MBP.lan>

rbd-mirror: fix possible recursive lock of ImageReplayer::m_lock

If periodic status update (LambdaContext which is queued from
handle_update_mirror_image_replay_status()) races with shutdown and
ends up being the last in-flight operation that shutdown was pending
on, we attempt to recursively acquire m_lock in shut_down() because
m_in_flight_op_tracker.finish_op() is called with m_lock (and also
m_threads->timer_lock) held.  These locks are needed only for the call
to schedule_update_mirror_image_replay_status() and should be unlocked
immediately.

Fixes: https://tracker.ceph.com/issues/69978
Co-authored-by: Ilya Dryomov <idryomov@gmail.com>
Signed-off-by: N Balachandran <nithya.balachandran@ibm.com>

Merge pull request #61827 from tchaikov/python3.12-on-ubuntu-noble

do_cmake: use Python 3.12 on ubuntu >= 24

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

Merge pull request #61794 from MaxKellermann/cls__includes

cls: add missing includes

Reviewed-by: Adam Emerson <aemerson@redhat.com>
Reviewed-by: Ilya Dryomov <idryomov@redhat.com>
Reviewed-by: Venky Shankar <vshankar@redhat.com>

Merge pull request #61608 from MaxKellermann/mon_includes

mon: include cleanup

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

Merge pull request #61618 from ronen-fr/wip-rf-perf-sections

common/perf_counters: use the RAII helpers when dumping

Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>

Merge pull request #61678 from phlogistonjohn/jjm-bwc-fixes-1

build-with-container fixes exec bit, dnf cache dir option

Reviewed-by: Adam King <adking@redhat.com>
Reviewed-by: Redouane Kachach <rkachach@ibm.com>

cls/numops: add missing include

Signed-off-by: Max Kellermann <max.kellermann@ionos.com>

cls/rbd: add missing includes

Signed-off-by: Max Kellermann <max.kellermann@ionos.com>

cls/cephfs: add missing forward declaration

Signed-off-by: Max Kellermann <max.kellermann@ionos.com>

cls/2pc_queue: add missing include

Signed-off-by: Max Kellermann <max.kellermann@ionos.com>

journal: add missing includes

Signed-off-by: Max Kellermann <max.kellermann@ionos.com>

mgr/dashboard: SMB Create Share

Fixes: https://tracker.ceph.com/issues/69733
Signed-off-by: Dnyaneshwari Talwekar <dtalweka@redhat.com>

do_cmake: use Python 3.12 on ubuntu >= 24

the "official" Python shipped along with Ubuntu 24.04 (Noble Numbat) is
Python 3.12. And some of our building have been upgraded to Ubuntu
24.04. But we are still using Python 3.10 on Ubuntu >= 22, this breaks
the build. And CMake fails like:

```
CMake Error at /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
  Could NOT find Python3 (missing: Python3_EXECUTABLE Python3_INCLUDE_DIRS
  Python3_LIBRARIES Interpreter Development Development.Module
  Development.Embed) (Required is exact version "3.10")

      Reason given by package:
          Interpreter: Wrong version for the interpreter "/bin/python3"

Call Stack (most recent call first):
  /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:600 (_FPHSA_FAILURE_MESSAGE)
  cmake/modules/FindPython/Support.cmake:3863 (find_package_handle_standard_args)
  cmake/modules/FindPython3.cmake:545 (include)
  CMakeLists.txt:597 (find_package)
```

This build failure should also happen on developers who build Ceph on
Ubuntu >= 24.

In this change, we use Python 3.12 on Ubuntu >= 24

Signed-off-by: Kefu Chai <tchaikov@gmail.com>

Merge pull request #61768 from idryomov/wip-69911

librbd: fix mirror image status summary in a namespace

Reviewed-by: N Balachandran <nibalach@redhat.com>

Merge pull request #61698 from afreen23/wip-fix-nsapi

mgr/dashboard: (refactor)fix image size in nvmeof namespace create/update api

Reviewed-by: Afreen Misbah <afreen@ibm.com>

Merge pull request #61734 from rhcs-dashboard/delete-storage-class

mgr/dashboard: RGW - Delete storage class

Reviewed-by: Afreen Misbah <afreen@ibm.com>
Reviewed-by: Ankush Behl <cloudbehl@gmail.com>
Reviewed-by: Nizamudeen A <nia@redhat.com>

common/perf_counters: use the RAII helpers when dumping

Using Formatter::ObjectSection & Formatter::ArraySection
to guarantee consistent dump sections opening & closing.

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

Merge PR #61763 into main

* refs/pull/61763/head:
pybind/mgr/mgr_module: turn off all automatic transactions

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

Merge pull request #61688 from dmick/wip-ceph-release-docs

doc/dev/release-process.rst: release builds cannot build containers

Merge PR #61680 into main

* refs/pull/61680/head:
mds: do not schedule quiesce for non-head imported inodes

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

Merge PR #61675 into main

* refs/pull/61675/head:
qa: do read checks with non-empty file

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

common: add missing mds op names

Fixes: 8970fef4ecc4dc28011bb57c5150fbdc691279d0
Fixes: 7f1d066756383adae37d61cc2b2f5d99bc167f65
Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>

Merge pull request #61700 from MaxKellermann/rgw_includes

rgw: add missing includes

Reviewed-by: Adam Emerson <aemerson@redhat.com>

Merge pull request #60791 from Jayaprakash-ibm/jaya-branch

blk/kernel : Make bdev stop immediately

Reviewed-by : Adam Kupczyk <akupczyk@ibm.com>
Reviewed-by : Igor Fedotov <igor.fedotov@croit.io>

Merge pull request #56925 from ifed01/wip-ifed-dencoder-crc32

tools/ceph-dencoder: introduce add_crc32c command to append crc32 for…

Reviewed-by: Md Mahamudur Rahaman Sajib <mahamudur.sajib@croit.io>

Merge pull request #61428 from rhcs-dashboard/fix-69575-main

mgr/dashboard: fix service/daemon events not showing up in service details

Reviewed-by: Afreen Misbah <afreen@ibm.com>

Merge pull request #61769 from Matan-B/wip-matanb-clang-14-only

script/lib-build: Use clang 14

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

Merge pull request #61089 from pecastro/mgr_proto_build_generation

pybind/mgr: Protobuffer file generation for grpc deps.

Reviewed-by: baum <NOT@FOUND>
Reviewed-by: ceph-jenkins <NOT@FOUND>
Reviewed-by: Ernesto Puerta <epuertat@redhat.com>
Reviewed-by: Nizamudeen A <nia@redhat.com>

mon/ConfigMonitor: include cleanup

Signed-off-by: Max Kellermann <max.kellermann@ionos.com>

mon/MonMap: include cleanup

Signed-off-by: Max Kellermann <max.kellermann@ionos.com>

mon/mon_types: include cleanup

Signed-off-by: Max Kellermann <max.kellermann@ionos.com>

mon/MDSMonitor: include cleanup

Signed-off-by: Max Kellermann <max.kellermann@ionos.com>

mon/PGMap: include cleanup

Signed-off-by: Max Kellermann <max.kellermann@ionos.com>

mon/FSCommands: include cleanup

Signed-off-by: Max Kellermann <max.kellermann@ionos.com>

mon/Monitor: add missing includes

Signed-off-by: Max Kellermann <max.kellermann@ionos.com>

mon/Paxos: include cleanup

Signed-off-by: Max Kellermann <max.kellermann@ionos.com>

mon/ConfigMap: add missing includes

Signed-off-by: Max Kellermann <max.kellermann@ionos.com>

mon/MonClient: add missing include

Signed-off-by: Max Kellermann <max.kellermann@ionos.com>

mon/MgrMap: add missing includes

Signed-off-by: Max Kellermann <max.kellermann@ionos.com>

mon/Session: add missing includes

Signed-off-by: Max Kellermann <max.kellermann@ionos.com>

mon/ElectionLogic: add missing includes

Signed-off-by: Max Kellermann <max.kellermann@ionos.com>

krbd: include cleanup

Signed-off-by: Max Kellermann <max.kellermann@ionos.com>

Merge pull request #60994 from MaxKellermann/formatter

Formatter code cleanup

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

Merge pull request #61759 from yuvalif/wip-yuval-boto-example-bucket-param

rgw/notification/logging: add bucket as context param

Merge pull request #60305 from MaxKellermann/blk_interface

blk/CMakeLists.txt: generate interface library if `libblk` is disabled

Reviewed-by: Kefu Chai <tchaikov@gmail.com>

Merge pull request #61699 from MaxKellermann/crimson_includes

crimson/backref_entry: add missing includes

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

Merge pull request #61705 from MaxKellermann/neorados_includes

neorados: include cleanup

Reviewed-by: Adam Emerson <aemerson@redhat.com>

doc/rgw/logging: update field description

this is updating documentation based on the change in
this PR: https://github.com/ceph/ceph/pull/61388

Signed-off-by: Yuval Lifshitz <ylifshit@ibm.com>

Merge pull request #61775 from yuvalif/wip-yuval-fix-loggin-obj-name

rgw/logging: flush command should print flushed log object name

mgr/dashboard: RGW - Delete Storage Class

Fixes: https://tracker.ceph.com/issues/69880
Signed-off-by: Dnyaneshwari Talwekar <dtalweka@redhat.com>

doc/dev/release-process.rst: release builds cannot build containers

Document that container images are not built by the ceph-build job,
but must be done manually after the package signing and upload
to download.ceph.com.

Signed-off-by: Dan Mick <dan.mick@redhat.com>
Signed-off-by: Zac Dover <zac.dover@proton.me>

doc: Upgrade and unpin some python versions

https://access.redhat.com/security/cve/CVE-2022-34749

https://bugzilla.redhat.com/show_bug.cgi?id=2255448

Fixes https://github.com/ceph/ceph/pull/44222

Signed-off-by: David Galloway <david.galloway@ibm.com>

Merge pull request #61746 from adamemerson/wip-boost.1.87-fixes

Boost 1.87 Build Fixes

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

Merge pull request #60995 from MaxKellermann/messages__includes

messages/*: add missing includes

Merge pull request #61418 from cbodley/wip-rgw-multisite-sync-delta-counters

rgw: add labeled counters for sync deltas of shards

Reviewed-by: Adam Emerson <aemerson@redhat.com>

Merge pull request #61290 from BBoozmen/wip-oozmen-67846

RGW/S3: truncate S3 LastModified response (timestamp) to exact seconds

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

Merge pull request #61009 from cbodley/wip-69169

rgw/rados: resolve CopyObject crash from remote zonegroup

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

Merge pull request #61090 from soumyakoduri/wip-skoduri-cloud-restore

rgw/cloud-restore: Do not send internal headers to cloud-endpoint

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

Merge pull request #61358 from richardpoole/main

rgw: allow management of `accounts` user caps

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

Merge pull request #61573 from clwluvw/log-new-req-id

rgw: log request_id on start new request

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

mgr/dashboard: fix image size in nvmeof namespace create/update api

- Different name is used in POST and PATCH for `rbd_image_size`
- Using same name in both requests
- fixing typing issues in frontend

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

Signed-off-by: Afreen Misbah <afreen@ibm.com>

Merge pull request #60175 from MaxKellermann/misc_optimizations

Misc optimizations

ceph-backport.sh: add squid to known milestones

Avoids: WARNING: Milestone ->squid<- is unknown to the script: falling back to GitHub API

Signed-off-by: Jos Collin <jcollin@redhat.com>

rgw/logging: flush command should print flushed log object name

and not the next pending log object name

Signed-off-by: Yuval Lifshitz <ylifshit@ibm.com>

Merge pull request #61621 from joscollin/wip-B69669-fix-exception-smaller-pad-width

cephfs-top: exception when terminal size greater than PAD_WIDTH

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

script/lib-build: Use clang 14

In attempt to update to clang16 some of the code is not yet adapted
due to deprecated functions.
Let's disable clang 16 usage until https://github.com/ceph/ceph/pull/61740
is merged.

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

qa/workunits/rbd: use create_image_and_enable_mirror() in bootstrap tests

The reason create_image() + enable_mirror() happens to work for
PARENT_POOL is that PARENT_POOL is enabled for mirroring in image mode
unconditionally, unlike POOL, POOL/NS1 or PARENT_POOL/NS1 for which
MIRROR_POOL_MODE setting is respected.  This isn't immediately obvious
because it's done in setup_pools() in rbd_mirror_helpers.sh.

Switch to create_image_and_enable_mirror() for clarity.

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

rbd: don't clone ioctx before calling get_mirror_peer_sites()

get_mirror_peer_sites() transitions to the default namespace
internally, doing it in advance at the call sites is redundant.

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

librbd: fix mirror image status summary in a namespace

For the purposes of the summary with image counts, "rbd mirror pool
status" command is supposed to count each image only once.  To this
end, for unidirectional mirroring the status of the receiving site
should be taken while for bidirectional mirroring the statuses should
be combined/reduced.  For example, if mirroring is enabled on a single
image and everything is in order, the summary is expected to be

  image health: OK
  images: 1 total
      1 replaying

on both clusters even though on the primary the local status is
MIRROR_IMAGE_STATUS_STATE_STOPPED and only on the secondary it's
MIRROR_IMAGE_STATUS_STATE_REPLAYING.

Currently this isn't the case for custom namespaces.  In the same
scenario the primary ends up reporting

  image health: OK
  images: 1 total
      1 stopped

based solely on the local status in a namespace.

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