17.2.9

Signed-off-by: Ceph Release Team <ceph-maintainers@ceph.io>

os/bluestore: fix _extend_log seq advance

when extending the log, the sequence was left on a bad state because it would first create a transaction to update with the current seq number but leave the "real" transaction with the same sequence number which should be `extend_log_transaction.seq + 1`.

Signed-off-by: Pere Diaz Bou <pdiabou@redhat.com>
Fixes: https://tracker.ceph.com/issues/71199
(cherry picked from commit 0210f57a5ec8a8c0ea76a72984f39dae24986cb0)
(cherry picked from commit b76623504a544feba2adf7a196ddbd59d1b8b8f4)

os/bluestore: test _extend_log sequence advance

Signed-off-by: Pere Diaz Bou <pdiabou@redhat.com>
(cherry picked from commit e785ef23339eba41bf786a68bba845f1916feb61)
(cherry picked from commit a71c33869d174a67c6c9eedca4997442a48f3a6c)

17.2.8

Signed-off-by: Ceph Release Team <ceph-maintainers@ceph.io>

mgr/dashboard: remove cherrypy_backports.py

since its mostly used only for older cherrypy versions which we don't
support anymore in any of our recent upstream releases, we could remove
it completely

Fixes: https://tracker.ceph.com/issues/68802
Signed-off-by: Nizamudeen A <nia@redhat.com>
(cherry picked from commit 7ae56f4f561999898a7411840f94e742981ca6bc)

 Conflicts:
src/pybind/mgr/prometheus/module.py
 -  resolve conflicting import by just removing the version import

(cherry picked from commit 8713c8c2266c5075d6a0eb8da6e77152cda44c11)

mgr/dashboard: update translations for quincy

Signed-off-by: Nizamudeen A <nia@redhat.com>
(cherry picked from commit 1ba918183898d869c33feb9f626469954909f996)

qa/fsx: use a specified sha1 to build the xfstest-dev

This sha1 is the latest master head and works well for our tests.

Fixes: https://tracker.ceph.com/issues/64572
Signed-off-by: Xiubo Li <xiubli@redhat.com>
(cherry picked from commit 740025da227e75575fa7732141c60c6398442e43)
(cherry picked from commit 4ccbf359add7c4f07ea088dd4693480bd8c1a242)

qa/fsx: use the master branch to build the xfstest-dev

The upstream linux has removed ALLOCSP/FREESP ioctls, and also
the same with xfsprogs-devel package. So for the old xfstest-dev
the huild will fail, because it stil will depend on them.

Fixes: https://tracker.ceph.com/issues/64572
Signed-off-by: Xiubo Li <xiubli@redhat.com>
(cherry picked from commit bc215d9e21e97382434579cb89e90936c8d9c54b)
(cherry picked from commit c10ba90650800807749dfbe07bc6efc04b1c0870)

Merge pull request #60479 from smanjara/wip-quincy-pr-56874

[quincy] qa/multisite: stabilize multisite testing

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

Merge pull request #60527 from zdover23/wip-doc-2024-10-29-backport-60470-to-quincy

quincy: doc: explain the consequence of enabling mirroring through monitor co…

doc: explain the consequence of enabling mirroring through monitor commands

Signed-off-by: Jos Collin <jcollin@redhat.com>
(cherry picked from commit ed3ce72fbc0b097131945cbb39caa0831401b94a)

Merge pull request #60458 from cbodley/wip-64399-quincy

quincy: rgw/auth: ignoring signatures for HTTP OPTIONS calls

Merge pull request #60502 from zdover23/wip-doc-2024-10-26-backport-60444-to-quincy

quincy: doc/rados: standardize markup of "clean"

Merge pull request #60505 from zdover23/wip-doc-2024-10-26-backport-60434-to-quincy

quincy: doc/glossary: add "PLP" to glossary

doc/glossary: add "PLP" to glossary

Add "PLP" to the glossary.

Co-authored-by: Anthony D'Atri <anthony.datri@gmail.com>
Signed-off-by: Zac Dover <zac.dover@proton.me>
(cherry picked from commit 599bdbf0a240ee718ed6da4104141b0f23330fbe)

doc/rados: standardize markup of "clean"

Standardize the markup around the status "clean" in the documentation so
that readers don't mistakenly get the idea that inconsistent
presentation of the word "clean" implies a never-stated difference
between one instance and the other.

Signed-off-by: Zac Dover <zac.dover@proton.me>
(cherry picked from commit c8b4155d2d27228917d4be1f11514504100c55c7)

Merge pull request #60498 from zdover23/wip-doc-2024-10-26-backport-58647-to-quincy

quincy: doc/cephfs: add doc for disabling mgr/volumes plugin

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

qa/multisite: disable two-zonegroup and three-zone configs.

Signed-off-by: Shilpa Jagannath <smanjara@redhat.com>
(cherry picked from commit fb3186266ee52b352ebdf4ce394d3144a23db8f2)
(cherry picked from commit 7f0ae8e03636cf81811b295f62fdb4a5fddafbf2)
(cherry picked from commit 824fce5521b1e3223767ae7e362aad77b1e48a3d)

doc/cephfs: add doc for disabling mgr/volumes plugin

Signed-off-by: Rishabh Dave <ridave@redhat.com>
(cherry picked from commit 373910de34fc9e1d98218aed0361cd9bf3badffa)

qa/rgw/multisite: add two-zone configuration

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit 14204442ac2eaaaa3f40a76403da49f0f4ed92c1)
(cherry picked from commit 627dd7b86d0c59768625b508b614fa35d008708b)
(cherry picked from commit baf6c678bb3648a6147f1eb5bd1442a300cd0110)

qa/rgw/multisite: double cluster nodes and osd count

> WARNING: The OSD cluster is overloaded and struggling to complete ops. You need more capacity to serve this level of demand.

give each cluster 2 nodes instead of 1, and 6 osds instead of 3

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit 9a80cf4bf473c95c8d0b18fcdce88be20f74f973)
(cherry picked from commit a77c2d42277e44633643bcc1f8969f90724c49ef)
(cherry picked from commit 89c09ea3a2e9f4d32bab2c4b0b5c0f529d08d4c6)

qa/rgw/multisite: reduce polling intervals from 20s to 5s

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit bc084c0ac64477abdbf54d96bc4981edf41e1343)
(cherry picked from commit 32fb2b923bc07b2228b8fe7d1c1f4bc2f3bf263b)
(cherry picked from commit 95116915489d60f78038445d31549804e0745e29)

rgw/multisite: reconfigure checkpoint delay to 90s

Signed-off-by: Shilpa Jagannath <smanjara@redhat.com>
(cherry picked from commit af7c4d83c7f5b61e907a90622c082a373f3995d9)
(cherry picked from commit e1acebe9e710f704ca59980b125e9a93f51dc71a)

Merge pull request #60484 from zdover23/wip-doc-2024-10-25-backport-60480-to-quincy

quincy: doc/cephfs: use 'p' flag to set layouts or quotas

doc/cephfs: use 'p' flag to set layouts or quotas

Remind users that clients must use the 'p' flag when writing file
layouts.

This PR is made in response to
https://www.reddit.com/r/ceph/comments/1gbad51/comment/ltkgd5v/

Co-authored-by: Zac Dover <zac.dover@proton.me>
Signed-off-by: TruongSinh Tran-Nguyen <i@truongsinh.pro>
(cherry picked from commit 13c7c9d99881c7b4491ba1a693283c047161ea8a)

Merge pull request #58769 from adk3798/quincy-orch-centos-9-teuth

quincy: qa/orch: drop centos 8 and rhel 8.6 for orch suite tests

Reviewed-by: Laura Flores <lflores@redhat.com>

Merge pull request #60463 from zdover23/wip-doc-2024-10-24-backport-60460-to-quincy

quincy: doc/start: add vstart install guide

doc/start: add vstart install guide

Add "vstart Cluster Installation and Configuration Procedure" to the
Beginner's Guide.

Signed-off-by: Zac Dover <zac.dover@proton.me>
(cherry picked from commit a42be085bd7b9159f4924853ac2b2d92c37884b0)

qa/upgrade: change pacific/octopus upgrade tests to use published image

Since we no longer build octopus or pacific, we can't use
the ci build for these tests any more

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

qa/distros: reinstall nvme-cli on centos 9 nodes

To work around a potential linking issue between
nvme-cli ad libnvme that prevents nvme-cli from
correctly generating a hostnqn, causing

nvme_fabrics: found same hostid edb4e426-766f-44c6-b127-da2a5b7446ef but different hostnqn hostnqn

messages in dmesg and the inability to setup nvme
loop devices

Fixes: https://tracker.ceph.com/issues/67684
Signed-off-by: Adam King <adking@redhat.com>
(cherry picked from commit 4e5f269c01dd94741fbddaa7f76ebe4ac155e650)

qa/workunits/cephadm: enable nodejs:18 on centos 9 stream

Signed-off-by: Laura Flores <lflores@ibm.com>
(cherry picked from commit a602e5f0e0e48c40fd728ae382502826ca354828)

qa/cephadm: fix iscsi pids limit check for centos 9

Centos 9 uses cgroups v2 which has a slightly
different file location for the pids.max. This commit
updates the test to also check the new location
so the test can pass on centos 9

Signed-off-by: Adam King <adking@redhat.com>
(cherry picked from commit 97efa97c2c72dba32774d0f105ae90462b93ac4e)

tests: make sure nvmetcli and nvme-cli are up to date

Otherwise jobs end up with the following failure:

```
2024-06-25T14:22:18.659 INFO:teuthology.orchestra.run.smithi098.stderr:Failed to write to /dev/nvme-fabrics: Invalid argument
```

Also, the output of nvme list has changed so we have to update
qa/tasks/nvme_loop.py accordingly.

Fixes: https://tracker.ceph.com/issues/66707
Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
(cherry picked from commit d707c41eacb513bdcb9fec17c823bfbfdc182700)

Conflicts:
qa/distros/container-hosts/centos_9.stream.yaml

Conflicts:
qa/distros/container-hosts/centos_9.stream.yaml

qa/suites/upgrade/quincy-p2p: update supported-all-distro to centos 9

Signed-off-by: Laura Flores <lflores@ibm.com>

qa/cephadm: drop centos 8 and rhel 8.6 from test distros

Since we don't build either of these any more on
quincy

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

qa/distros: add supported-container-hosts for cephadm and upgrade suites

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit 849a58b0873154a6713b4c8d838634f13b0b806d)

Conflicts:
qa/suites/upgrade/telemetry-upgrade/quincy-x/0-random-distro$

rgw/auth: ignoring signatures for HTTP OPTIONS calls

Before [1] we always sent all HTTP OPTIONS requests to
the S3AnonymousEngine and ignored any provided AWSv4
credentials sent in the request.

That PR changed so that if we got credentials in the
request we instead sent it through the authentication
code in order to solve HTTP OPTIONS requests on tenanted
users to start working (because we need to resolve the
tenant, also called bucket tenant in the code, and we can't
only rely on the bucket name since it will not be found).

We solved this by modifying the canonical HTTP method used
when calculating the AWSv4 signature by instead using the
access-control-request-method header which worked good.

This change did not take into account that when you generated
a presigned URL for a put_object request you can also pass in
extra parameters like a canned ACL [2] to the Params variable
in for example boto3's generated_presigned_url().

Doing that will cause the client to add the x-amz-acl header
to x-amz-signedheaders and also use that in their signature
calculation.

When doing a HTTP OPTIONS calls for CORS on that presigned URL
the browser will never send a x-amz-acl header with the correct
data since that is something that the actual PUT request should
include later, so that HTTP OPTIONS call should pass even though
the signature can never be calculated correctly server-side like
verified against AWS S3 in tracker [3].

This patch as a result skips the signature calculation when doing
EC2 auth using the LocalEngine but we still need to pass the request
there in order to lookup the user to support buckets in a tenant.

For the Keystone EC2 auth we're pretty out of luck in the sense that
Keystone's API itself requires us to send the AWSv4 signature in the
request with the access_key in order to obtain a token, and we cannot
leave the signature out, we also cannot spoof the signature from
rgw -> keystone since we don't have access to the secret_key if it's
not in our cache.

For that approach we simply pass on to get_access_token() that if it's
an HTTP OPTIONS and we find the access_key in the cache we pull that
and ignore verifying signature and pass it on for validation. This means
that the cache must be warm if using Keystone auth and adding extra
params to a presigned URL.

This partly makes some of the commits in [1] redundant for EC2
LocalEngine auth but we still need it for tenanted bucket support.

[1] https://github.com/ceph/ceph/pull/52673
[2] https://docs.aws.amazon.com/AmazonS3/latest/userguide/acl-overview.html#canned-acl
[3] https://tracker.ceph.com/issues/64308

Fixes: https://tracker.ceph.com/issues/64308
Signed-off-by: Tobias Urdin <tobias.urdin@binero.se>
(cherry picked from commit fe15b52edb5d228d2ed56679c62cf48493ae2d54)

Conflicts: no optional_yield argument in quincy
src/rgw/rgw_auth_keystone.cc
src/rgw/rgw_auth_keystone.h

Merge pull request #60437 from zdover23/wip-doc-2024-10-23-backport-60433-to-quincy

quincy: doc/cephfs: rearrange subvolume group information

doc/cephfs: rearrange subvolume group information

Rearranging the documentation to align it with CephFS encapsulation design.

Signed-off-by: Indira Sawant <indira.sawant@ibm.com>
(cherry picked from commit f6cf7ae5281f5740ac78bfa5e2a22913353b8167)

Merge pull request #60314 from vshankar/wip-client-secfix

quincy: client: disallow unprivileged users to escalate root privileges

Reviewed-by: Venky Shankar vshankar@redhat.com

Merge pull request #58843 from lxbsz/wip-67163

quincy: mds: try to choose a new batch head in request_clientup()

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

Merge pull request #58296 from batrick/wip-66624-quincy

quincy: mds: do remove the cap when seqs equal or larger than last issue

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

Merge pull request #54549 from k0ste/wip-62817-quincy

quincy: osd: ensure async recovery does not drop a pg below min_size

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

Merge pull request #60124 from adamemerson/wip-40692-quincy

quincy: global: Call getnam_r with a 64KiB buffer on the heap

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

Merge pull request #59498 from pponnuvel/wip-67606-quincy

quincy: os/bluestore: allow use BtreeAllocator

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

Merge pull request #57406 from k0ste/wip-58815-quincy

quincy: rados: Set snappy as default value in ms_osd_compression_algorithm

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

Merge pull request #57363 from batrick/wip-65871-quincy

quincy: common/StackStringStream: update pointer to newly allocated memory in overflow()

Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
Reviewed-by: Rongqi Sun <sunrongqi@huawei.com>

Merge pull request #54557 from k0ste/wip-57795-quincy

quincy: common:  intrusive_lru destructor add

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

Merge pull request #60337 from dmick/wip-quincy-container-ceph-node-proxy

quincy: ceph-node-proxy not present, not part of container

Merge pull request #60334 from cbodley/wip-doc-rgw-multisite-secondary-commit-quincy

quincy: doc/radosgw/multisite: fix Configuring Secondary Zones -> Updating the Period

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

Merge pull request #56702 from joscollin/wip-65320-quincy

quincy: cephfs-mirror: various fixes

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

Merge pull request #58333 from mchangir/wip-66671-quincy

quincy: qa: cleanup snapshots before subvolume delete

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

Merge pull request #58724 from neesingh-rh/wip-67076-quincy

quincy: mds: fix session/client evict command.

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

Merge pull request #58773 from lxbsz/wip-66668

quincy: mds: set the correct WRLOCK flag always in wrlock_force()

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

Merge pull request #58799 from neesingh-rh/wip-66870-quincy

quincy: mds: prevent scrubbing for standby-replay MDS

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

Merge pull request #58806 from lxbsz/wip-67150

quincy: cephfs: Fixed a bug in the readdir_cache_cb function that may have us…

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

Merge pull request #58807 from lxbsz/wip-67119

quincy: qa: ignore container checkpoint/restore related selinux denials for c…

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

Merge pull request #59037 from vshankar/wip-quincy-rm-centos8

quincy: qa/cephfs: no reliance on centos

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

Merge pull request #59067 from joscollin/wip-67381-quincy

quincy: qa/suites: drop --show-reachable=yes from fs:valgrind tests

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

quincy: ceph-node-proxy not present, not part of container

Signed-off-by: Dan Mick <dmick@redhat.com>

doc/radosgw/multisite: fix Configuring Secondary Zones -> Updating the Period

this was copy/pasted from Configuring a Master Zone -> Update the Period
but still referred to the 'master zone'

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit cfe254758b1ca9647c4dcfb13b6a3310558b88d2)

Merge pull request #59403 from Matan-B/wip-67665-quincy

quincy: mon/OSDMonitor: Add force-remove-snap mon command

Merge pull request #59308 from batrick/wip-67476-quincy

quincy: mon/MonClient: handle ms_handle_fast_authentication return

Merge pull request #59278 from k0ste/wip-67240-quincy

quincy: mon: fix inconsistencies in class param

Merge pull request #60201 from zdover23/wip-doc-2024-10-09-backport-60163-to-quincy

quincy: doc/man: supplant "wsync" with "nowsync" as the default

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

Merge pull request #55216 from ifed01/wip-ifed-refactor-spillover-tests-qui

quincy: test/store_test: refactor spillover tests

Reviewed-by: Pere Diaz Bou <pere-altea@hotmail.com>

client: disallow unprivileged users to escalate root privileges

An unprivileged user can `chmod 777` a directory owned by root
and gain access. Fix this bug and also add a test case for the
same.

Signed-off-by: Xiubo Li <xiubli@redhat.com>
Signed-off-by: Venky Shankar <vshankar@redhat.com>

test/store_test: refactor spillover tests

Fixes: https://tracker.ceph.com/issues/62401
Signed-off-by: Igor Fedotov <igor.fedotov@croit.io>
(cherry picked from commit 5019495659359bbc19c8ab3988e3d9e12c5f58c1)

 Conflicts:
src/test/objectstore/store_test.cc
 * trivial

os/bluestore: provide some reference for BlueFSVolumeSelector's interface.

Signed-off-by: Igor Fedotov <igor.fedotov@croit.io>
(cherry picked from commit 243d21ef1c18d561b35acf309e8e1c13b0781658)

os/bluestore: rework vselector calls

We can provide fnode delta to vseector now. Which is a bit more
effective.

Signed-off-by: Igor Fedotov <igor.fedotov@croit.io>
(cherry picked from commit 3e04ac48779f62394ebd9587298823bf6b20a570)

 Conflicts:
src/os/bluestore/BlueFS.cc
 * trival

Merge pull request #60230 from dmick/wip-68470-quincy

quincy: Add Containerfile and build.sh to build it.

Merge pull request #59442 from amathuria/wip-65946-quincy

quincy: msg/async: Encode message once features are set

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

Merge pull request #54356 from leonid-s-usov/bp/mds-raise/quincy

quincy: common/admin_socket: add a command to raise a signal

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

Merge pull request #48310 from k0ste/wip-56637-quincy

quincy: log: Make log_max_recent have an effect again.

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

container/build.sh: fix arm architecture tagging

The wrong string was used for comparison, and for tagging,
so the arm64 branch and sha1 images overwrote and destroyed
the amd64 images.

Signed-off-by: Dan Mick <dmick@redhat.com>
(cherry picked from commit 7e03ee798f4ed3aa4a0bb1a9e9d62df52e54406d)

Merge pull request #55775 from ifed01/wip-ifed-no-death-tests-qui

quincy: test/store_test: get rid off assert_death.

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

Merge pull request #60299 from zdover23/wip-doc-2024-10-14-backport-60242-to-quincy

quincy: doc: SubmittingPatches-backports - remove backports team

Merge pull request #60274 from zdover23/wip-doc-2024-10-12-backport-60243-to-quincy

quincy: doc/dev: remove "Stable Releases and Backports"

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

doc: SubmittingPatches-backports - remove backports team

Remove all references to the "Stable Releases and Backports Team", which
as of October 2024 does not exist.

Fixes: https://tracker.ceph.com/issues/68471
Co-authored-by: Laura Flores <lflores@ibm.com>
Signed-off-by: Zac Dover <zac.dover@proton.me>
(cherry picked from commit 0317b5f87ac22399f6242d72f0bb9924794687de)

doc/dev: remove "Stable Releases and Backports"

Remove mention of "Stable Releases and Backports" from
doc/dev/developer_guide/essentials.rst.

Co-authored-by: Laura Flores <lflores@ibm.com>
Signed-off-by: Zac Dover <zac.dover@proton.me>
(cherry picked from commit 95916c91906604c516e78b550010e9de511fc1e9)

Merge pull request #60271 from zdover23/wip-doc-2024-10-12-backport-59507-to-quincy

quincy: doc/rbd: add namespace information for mirror commands

doc/rbd: add namespace information for mirror commands

Updates the rbd man page to clarify  that some mirror commands
can apply to pools and namespaces.

Signed-off-by: N Balachandran <nibalach@redhat.com>
(cherry picked from commit 5e64748927d029ceffc27edbc6cbf7c774a7b291)

Merge pull request #60235 from zdover23/wip-doc-2024-10-10-backport-60204-to-quincy

quincy: doc: fix email

doc: fix email

Signed-off-by: Ernesto Puerta <37327689+epuertat@users.noreply.github.com>
(cherry picked from commit 512eea1af52126e9b082744ee7b870c12c23d55d)

Add Containerfile and build.sh to build it.

The intent is to replace ceph-container.git, at first for ci containers
only, and eventually production containers as well.

There is code present for production containers, including
a separate "make-manifest-list.py" to scan for and glue the two
arch-specific containers into a 'manifest-list' 'fat' container,
but that code is not yet fully tested.

This code will not be used until a corresponding change to the
Jenkins jobs in ceph-build.git is pushed.

Note that this tooling does not authenticate to the container repo;
it is assumed that will be done elsewhere.  Authentication is
verified by pushing a minimal image to the requested repo.

Signed-off-by: Dan Mick <dmick@redhat.com>
(cherry picked from commit 5c40a5c1737cf0628c899c4e9f788017b8b4fca8)

rados: Set snappy as default value in ms_osd_compression_algorithm instead of list of algorithms.

The OSDs choose the preferred algorithm based on mutual understanding between OSDs. The proposed solution to this problem was to set "snappy" as default algorithm and provide flexibility to user to change the algorithm of their choice.

Fixes: https://tracker.ceph.com/issues/58410
Signed-off-by: shreyanshjain7174 <ssanchet@redhat.com>
(cherry picked from commit 158986a3b73a42db8e31f3366912e906c743f145)

log: Make log_max_recent have an effect again.

The log improvements in a747aeac13daf3dba43343120659e802cb569f6b
unfortunately left log_max_recent broken because m_max_recent wasn't
used anymore.

Eliminate m_max_recent and set the capacity of the m_recent ring buffer
when log_max_recent changes. In order to call set_capacity(),
ConcreteEntry needed its move constructor set noexcept.

I haven't followed the boost code all the way down but I suspect that
setting the ring buffer capacity to anything less than 1 entry will
probably cause problems, so restrict log_max_recent to >=1.

Also fix a wrong variable used for printing the max new entries during
"log dump".

Signed-off-by: Joshua Baergen <jbaergen@digitalocean.com>
(cherry picked from commit 3d59ba1671e3e76326fb706a76b8d9638d782924)

Merge pull request #55598 from NitzanMordhai/wip-63976-quincy

quincy: Tools/rados: Improve Error Messaging for Object Name Resolution

Reviewed-by: Matan Breizman <Matan.Brz@gmail.com>
Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
Reviewed-by: Laura Flores <lflores@redhat.com>

Merge pull request #55597 from NitzanMordhai/wip-63982-quincy

quincy: mon/Monitor: during shutdown don't accept new authentication and crea…

Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
Reviewed-by: Matan Breizman <Matan.Brz@gmail.com>

Merge pull request #54555 from k0ste/wip-57473-quincy

quincy: mgr: fix a race condition in DaemonServer::handle_report()

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

Merge pull request #54012 from k0ste/wip-63036-quincy

quincy: mgr: add throttle policy for DaemonServer

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

Merge pull request #50193 from trociny/wip-58806-quincy

quincy: mgr: don't dump global config holding gil

Reviewed-by: Tim Serong <tserong@suse.com>

doc/man: supplant "wsync" with "nowsync" as the default

Change the default from "wsync" to "nowsync". "nowsync" has been the
default since kernel version 5.16.

This information was reported on
https://pad.ceph.com/p/Report_Documentation_Bugs by Benjamin Mare on 07
Oct 2024.

The commit relevant to this change can be seen here, in Linus's main
branch:
https://github.com/torvalds/linux/commit/f7a67b463fb83a4b9b11ceaa8ec4950b8fb7f902

Co-authored-by: Anthony D'Atri <anthony.datri@gmail.com>
Signed-off-by: Zac Dover <zac.dover@proton.me>
(cherry picked from commit 9f0af5d75dc64ef0520fd78f29815d2a824a859b)

Merge pull request #59523 from badone/wip-67822-quincy

quincy: test/lazy-omap-stats: Convert to boost::regex

Reviewed-by: Laura Flores <lflores@redhat.com>

Merge pull request #60064 from sajibreadd/wip-68325-quincy

quincy: common/Formatter: dump inf/nan as null

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

Merge pull request #59532 from ifed01/wip-ifed-fix-spillover-qui

quincy: test/store_test: enforce sync compactions for spillover tests

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

Merge pull request #59468 from sajibreadd/wip-67039-quincy

quincy: os/bluestore: Warning added for slow operations and stalled read

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

Merge pull request #58674 from ifed01/wip-ifed-many-many-extents-read-qui

quincy: blk/aio: fix long batch (64+K entries) submission

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

Merge pull request #56138 from ifed01/wip-ifed-fix-global-repair-stats-qui

quincy: os/store_test: Retune tests to current code

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

Merge pull request #55133 from ifed01/wip-ifed-refresh-perf-counters-qui

quincy: os: introduce ObjectStore::refresh_perf_counters() method

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