doc: users now need to provide scrub_mdsdir and recursive flags
for mdsdir scrub at CephFS root.

Fixes: https://tracker.ceph.com/issues/59350
Signed-off-by: Dhairya Parmar <dparmar@redhat.com>
(cherry picked from commit 93dfc111bf7e11b3122d2a2df0346b341164dd08)

qa: add recursive flag to test_flag_scrub_mdsdir

Code has been changed, in order to scrub ~mdsdir at root,
recursive flag also needs to be provided along with
scrub_mdsdir.

Fixes: https://tracker.ceph.com/issues/59350
Signed-off-by: Dhairya Parmar <dparmar@redhat.com>
(cherry picked from commit e40ca408a1343754b80ba1e88e0b669689ada164)

mds: remove code to bypass dumping empty header scrub info

Previously when ~mdsdir was scrubbed at CephFS root, it's header
was kept empty, thus it became necessary to not dump it's values
for 'scrub status'. Now since both the scrubs(~mdsdir and root)
run under the same header, this code is no more needed.

Fixes: https://tracker.ceph.com/issues/59350
Signed-off-by: Dhairya Parmar <dparmar@redhat.com>
(cherry picked from commit 548af10f1057fe3fb46e23887b012a664b45b7f8)

mds: dump_values no more needed

Previouly, two individual scrubs were initiated to scrub ~mdsdir
at root where the ~mdsdir scrub wasn't provided any tag thus, it
was necessary to not dump it's values for output of 'scrub start'.
Now since mdsdir and root scrub run under single header, there is
no need for this anymore, thus removing this redundant code.

Fixes: https://tracker.ceph.com/issues/59350
Signed-off-by: Dhairya Parmar <dparmar@redhat.com>
(cherry picked from commit 2f06feea33c88798e4e9d654bbff8b05e77f0681)

mds: enqueue ~mdsdir at the time of enqueing root

This would avoid the need to run individual scrubs for
~mdsdir and root, i.e. run both the scrubs under the
same header, this also helps to avoid edge case where
in case ~mdsdir is huge and it's taking time to scrub it,
the scrub status would report something like this until
root inodes kick in:

{
    "status": "scrub active (757 inodes in the stack)",
    "scrubs": {}
}

Fixes: https://tracker.ceph.com/issues/59350
Signed-off-by: Dhairya Parmar <dparmar@redhat.com>
(cherry picked from commit f85b22818b474911cfdc15e7c6be1d2979939888)

mds: adjust MDSRank::command_tag_path invocation of enqueue_scrub()

Signed-off-by: Dhairya Parmar <dparmar@redhat.com>
(cherry picked from commit f654de977b8dddcbd4a39ffb4a338b549ca02bb0)

doc/scrub: documented stray evaluation using recursive scrub

Also documented the use of new flag introduced in this PR

Fixes: https://tracker.ceph.com/issues/53724
Signed-off-by: Dhairya Parmar <dparmar@redhat.com>
(cherry picked from commit d22e60fa6900449e443a23087b16f0c451d79c2b)

qa: added testcases

- test_stray_evaluation_with_scrub
this assures that evaluating strays with scrub works fine and no
crash is detected.

- test_flag_scrub_mdsdir
test the new flag to scrub ~mdsdir at CephFS root

Fixes: https://tracker.ceph.com/issues/51824
Signed-off-by: Dhairya Parmar <dparmar@redhat.com>
(cherry picked from commit 632c8b04cc91097cda3b35f639e623c2b07abb68)

mds: make `scrub status` print flag `scrub_mdsdir`

and make sure it does not dispose JSON for recursive scrub
on ~mdsdir while scrubbing with flag 'scrub_mdsdir'

Signed-off-by: Dhairya Parmar <dparmar@redhat.com>
(cherry picked from commit b32c572192fea394d1f56fb0e3d0f6cf672d5ec3)

mds: add scrub_mdsdir to ScrubHeader

Signed-off-by: Dhairya Parmar <dparmar@redhat.com>
(cherry picked from commit c85347fefb1b5a965a6640da9cb48d867642a295)

mds: do not dump multiple JSON obj

As enqueue_scrub is invoked twice to perform stray evaluation
at root using new op scrub_mdsdir, tests fail as command
scrub start / scrub_mdsdir would return two JSON object which
JSONDecodeError cannot parse, this patch would make sure we do
not dump JSON for recursive scrub on ~mdsdir than happens along with
scrubbing root (only when using scrub_mdsdir op)

Signed-off-by: Dhairya Parmar <dparmar@redhat.com>
(cherry picked from commit 4a976b5b0110c787b9a8e39360fd41164355036d)

mds: evaluate strays while performing scrub on root path

Running scrub at CephFS root doesn't iterate over ~mdsdir,
this feature is needed but the cluster admin should have
the liberty to decide whether to evaluate strays at root
or not. Therefore a new option 'scrub_mdsdir' can be useful.

Fixes: https://tracker.ceph.com/issues/53724
Signed-off-by: Dhairya Parmar <dparmar@redhat.com>
(cherry picked from commit e65eb2bb7895287c068691afe7a7a1710af0d360)

mds: remove inode from scrub_stack if being purged

it an inode is being purged, scrub should not be ran on it.

Fixes: https://tracker.ceph.com/issues/51824
Signed-off-by: Dhairya Parmar <dparmar@redhat.com>
(cherry picked from commit ce12cd71ffa703c8e3f05617cd8261d4ff820e7e)

mds: do not scrub inode if it is purging

Scrub could evaluate strays but there was a race condition where
an inode pinned for scrub would land for purging which would indeed
lead to crashes.

Fixes: https://tracker.ceph.com/issues/51824
Signed-off-by: Dhairya Parmar <dparmar@redhat.com>
(cherry picked from commit a8680d37821854fb6d8685db85bdf2b658f12981)

Merge pull request #50797 from zdover23/wip-doc-2023-03-31-backport-50674-to-quincy

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

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

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

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

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

Merge pull request #50770 from zdover23/wip-doc-2023-03-30-backport-50742-to-quincy

quincy: doc/start: update "notify us" section

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

doc/start: update "notify us" section

Rewrite the "Notify Us" section in doc/start/documenting-ceph.rst so
that it is written in English sentences.

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

Merge pull request #50788 from zdover23/wip-doc-2023-03-30-backport-50764-to-quincy

quincy: doc/start: format procedure in documenting-ceph

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

doc/start: format procedure in documenting-ceph

Organize three steps in a procedure into one step with three sub-steps.

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

Merge pull request #50762 from zdover23/wip-doc-2023-03-30-backport-50744-to-quincy

quincy: doc/rados: add link to ops/health-checks.rst

doc/rados: add link to ops/health-checks.rst

Add an internal navigation link to
doc/rados/operations/health-checks.rst.

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

Merge pull request #50752 from zdover23/wip-doc-2023-03-30-backport-50750-to-quincy

quincy: doc/rados/operations: Fix typo in erasure-code.rst

doc/rados/operations: Fix typo in erasure-code.rst

Signed-off-by: Sainithin Artham <sai.artham.19cse@bmu.edu.in>
(cherry picked from commit 3269fea2a209928137e55db16328c581988c048a)

Merge pull request #50740 from zdover23/wip-doc-2023-03-29-backport-50723-to-quincy

quincy: doc/start: documenting-ceph - add squash procedure

doc/start: documenting-ceph - add squash procedure

Add a procedure to doc/start/documenting-ceph.rst that explains how to
perform an interactive rebase to squash commits.

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

Merge pull request #50708 from zdover23/wip-doc-2023-03-28-backport-49514-to-quincy

quincy: doc/glossary: link to CephX Config ref

doc/glossary: link to CephX Config ref

Link to the CephX Configuration Reference in the "CephX" entry in the
glossary.

Signed-off-by: Zac Dover <zac.dover@gmail.com>
(cherry picked from commit 36a8f27e969d7103975ebfe312a4be6ae9f281f5)

Merge pull request #50702 from zdover23/wip-doc-2023-03-28-backport-50697-to-quincy

quincy: doc/glossary: add "Scrubbing"

doc/glossary: add "Scrubbing"

Add "Scrubbing" 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 4a66819da48cee430ca74830e1a6f63685b6b240)

Merge pull request #50678 from zdover23/wip-doc-2023-03-27-backport-50675-to-quincy

quincy: doc/rados: clean up ops/bluestore-migration.rst

doc/rados: clean up ops/bluestore-migration.rst

Clean up internal links, fix the numbering of a procedure, and implement
Anthony D'Atri's suggestions in
https://github.com/ceph/ceph/pull/50487 and
https://github.com/ceph/ceph/pull/50488.

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

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

Merge pull request #50672 from zdover23/wip-doc-2023-03-26-backport-50654-to-quincy

quincy: doc/glossary: add "User"

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

doc/glossary: add "User"

Add "User" to glossary.rst.

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

Merge pull request #50662 from zdover23/wip-doc-2023-03-25-backport-50660-to-quincy

quincy: doc/operations: fix prompt in bluestore-migration

doc/operations: fix prompt in bluestore-migration

Fix a single prompt in bluestore-migration.rst.

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

Merge pull request #50625 from cbodley/wip-58276

quincy: rgw: rgw_parse_url_bucket() rejects empty bucket names after 'tenant:'

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

Merge pull request #50575 from rzarzynski/wip-59101-quincy

quincy: msg/async: don't abort when public addrs mismatch bind addrs

Merge pull request #50641 from zdover23/wip-doc-2023-03-23-backport-50634-to-quincy

quincy: doc/rados: edit user-management.rst (1 of x)

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

doc/rados: edit user-management.rst (1 of x)

Edit doc/rados/operations/user-management.rst. (1 of x)

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

Merge pull request #50608 from rhcs-dashboard/wip-58743-quincy

quincy: mgr/dashboard: Landing page v3

Reviewed-by: Pegonzal <NOT@FOUND>
Reviewed-by: bryanmontalvan <NOT@FOUND>

rgw: rgw_parse_url_bucket() rejects empty bucket names after 'tenant:'

fail with ERR_INVALID_BUCKET_NAME if rgw_parse_url_bucket() finds a
'tenant:' but empty bucket name

Fixes: https://tracker.ceph.com/issues/58111
Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit 8f8bb1d2b743a2681d40cd8f25dedd0f221b49c7)

Conflicts:
src/rgw/rgw_bucket.cc: renamed to driver/rados/rgw_bucket

msg/async: don't abort when public addrs mismatch bind addrs

Before the 69b47c805fdd2ecd4f58547d58c9f019fc62d447 (PR #50153)
a mismatch (in number or types of stored `entity_addr_t`) between
public addrs and bind addrs vectors was ignored and the former
was taking over anything else -- it was possible to e.g. bind to
both v1 and v2 addresses but expose v2 only. Unfortunately, that's
exactly how Rook configures ceph-mon:

```
debug 2023-03-16T21:01:48.389+0000 7f99822bf8c0  0 starting mon.a rank 0 at public addrs v2:172.30.122.144:3300/0 at bind addrs [v2:10.129.2.21:3300/0,v1:10.129.2.21:6789/0] mon_data /var/lib/ceph/mon/ceph-a fsid acc14d1b-fb2b-4f01-8b61-6e7cb26e9200
```

The consequnece is the following abort:

```
ceph version 17.2.5-1338.el9cp (5adce3015143c7c2cc135a71368be194744f5761) quincy (stable)
1: (ceph::__ceph_abort(char const*, int, char const*, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&)+0xd3) [0x7ff05392c1a6]
2: /usr/lib64/ceph/libceph-common.so.2(+0x165ac1) [0x7ff05394eac1]
3: (AsyncMessenger::bindv(entity_addrvec_t const&, std::optional<entity_addrvec_t>)+0x1fe) [0x7ff053baa0ce]
4: main()
5: /lib64/libc.so.6(+0x3feb0) [0x7ff053048eb0]
6: __libc_start_main()
7: _start()
debug *** Caught signal (Aborted) **  in thread 7ff052ab18c0 thread_name:ceph-mon 2023-03-16T09:56:35.995+0000 7ff052ab18c0 -1 /builddir/build/BUILD/ceph-17.2.5-1338-g484e8dbb/src/msg/msg_types.h: In function 'void entity_addr_t::set_port(int)' thread 7ff052ab18c0 time 2023-03-16T09:56:35.996339+0000 /builddir/build/BUILD/ceph-17.2.5-1338-g484e8dbb/src/msg/msg_types.h: 359: ceph_abort_msg("abort() called")
```

This commit brings the original logic back but in a way that
preserves the port numbers figured out by. e.g. `Processor::bind`.

Fixes: https://tracker.ceph.com/issues/59100
Signed-off-by: Radosław Zarzyński <rzarzyns@redhat.com>
(cherry picked from commit 68fbdcf9b0937e8102b86d0bf84815554b62328d)

Merge pull request #50619 from zdover23/wip-doc-2023-03-22-backport-50606-to-quincy

quincy: doc/rados: line-edit erasure-code.rst

doc/rados: line-edit erasure-code.rst

Line-edit doc/rados/operations/erasure-code.rst.

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

mgr/dashboard: add button toggle to switch between new and old landing page

Signed-off-by: Pedro Gonzalez Gomez <pegonzal@redhat.com>
(cherry picked from commit 21dc93ae11db02bc8e0a2175f21fa7dd5e423b00)

mgr/dashboard: improve the logic on the alert viewer

only fetch the alerts when you toggle the alert viewer. don't fetch it
every 5 second cause its going to cause a heavy toll on the api.

Fixes: https://tracker.ceph.com/issues/58867
Signed-off-by: Nizamudeen A <nia@redhat.com>
(cherry picked from commit 6cb0679aaf28b5f128ca3cae5f962cac50d084af)
(cherry picked from commit 32a7d2c8ba2aff76f130a3a1c4fb13e810a53e12)

mgr/dashboard: fix prometheus api error on landing page v3

When no prometheus is configured in the cluster, it gives out error
while polling to the prometheus endpoint. so a proper check needs to be
added there.

Also some improvements to the exisiting behaviour

Fixes: https://tracker.ceph.com/issues/58867
Signed-off-by: Nizamudeen A <nia@redhat.com>
(cherry picked from commit 2d80e57dd4a60cbf52074d982ba3ebd4471a14ca)
(cherry picked from commit 0c6ab8be77890ab73cd86970c20269f8444e6acd)

mgr/dashboard: dashboard-v3: cluster-utilization card

mgr/dashboard: some improvements

mgr/dashboard: updated promqls and improved tooltips

mgr/dashboard: some fixes and improvements

mgr/dashboard: added square box plugin and improved/fixed code

mgr/dashboard: added maxValue option for graphs and added tooltips to text

mgr/dashboard: removed unnecessary variables

mgr/dashboard: fix openapi

tracker: https://tracker.ceph.com/issues/57863
Signed-off-by: Pedro Gonzalez Gomez <pegonzal@redhat.com>
(cherry picked from commit 7f7996fa55199ce0062612e89fc441467337e8c9)

mgr/dashboard: dashboard-v3: inventory card

This commit starts the inventory-card which is one of the cards which
will be located in the landing-page revamp

tracker: https://tracker.ceph.com/issues/58065
Signed-off-by: bryanmontalvan <bmontalv@redhat.com>
Signed-off-by: Nizamudeen A <nia@redhat.com>
Signed-off-by: Pedro Gonzalez Gomez <pegonzal@redhat.com>
(cherry picked from commit c63523119dee6cc82ffae5162f2270ac7d06d02f)

mgr/dashboard: dashboard-v3: status card

This commit is the bare-bones work of the status card. The only logic
written in this commit is the Cluster health status icon.

tracker: https://tracker.ceph.com/issues/58728
Signed-off-by: bryanmontalvan <bmontalv@redhat.com>
mgr/dashboard: introduce active alerts to status cards

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

xmgr/dashboard: dashboard-v3: capacity card

tracker: https://tracker.ceph.com/issues/57862
Signed-off-by: Pedro Gonzalez Gomez <pegonzal@redhat.com>
mgr/dashboard: fix openapi tox error

Signed-off-by: Pedro Gonzalez Gomez <pegonzal@redhat.com>
mgr/dashboard: capacity-card set interval and display bytes used

Signed-off-by: Pedro Gonzalez Gomez <pegonzal@redhat.com>
mgr/dashboard: Landing Page v3

Signed-off-by: Pedro Gonzalez Gomez <pegonzal@redhat.com>
(cherry picked from commit 95e20d27b3d0e861ff572481e9426f09e386c6d8)

mgr/dashboard: dashboard-v3: details card

mgr/dashboard: added interface, changed html table to list, using mgrModuleService now

tracker: https://tracker.ceph.com/issues/58731
Signed-off-by: Pedro Gonzalez Gomez <pegonzal@redhat.com>
(cherry picked from commit cc93d5308002c21efae155cf97b6bddd73f9db17)

mgr/dashboard: dashboard-v3: landing-page revamp

This commit add the following
- A new dashboard component, which will exist in parallel with in the
  current landing-page
- Created a route for this dashboard `/dashboard_3`
- Created a bare-bones bootstrap grid with mock-up card components

Signed-off-by: bryanmontalvan <bmontalv@redhat.com>
Signed-off-by: Pedro Gonzalez Gomez <pegonzal@redhat.com>
mgr/dashboard: changes to first layout

CHANGES:

 - Renamed dashboardcomponents

 - Removed unnecesary styling

 - Added unit tests

Signed-off-by: Pedro Gonzalez Gomez <pegonzal@redhat.com>
Moved router.url logic inside html template

This commit removes the `this.router.url` logic which was located in the
`workbench-layout.component.ts` file and moved it into the HTML template
section.

Signed-off-by: bryanmontalvan <bmontalv@redhat.com>
mgr/dashboard: syntax changes from bootstrap 4 to 5

Signed-off-by: Pedro Gonzalez Gomez <pegonzal@redhat.com>
mgr/dashboard: small fixes and improvements over all cards and layout

- all cards placed evenly with the same height
- increased font size on details card and adjusted margin
- changed capacity card legend to: "Used"
- adjusted cluster utilization card margins and increased graphs height
- improved status card toggle
- changed orchestror to orchestrator
- switched IPS/OPS graph colors

Signed-off-by: Pedro Gonzalez Gomez <pegonzal@redhat.com>
mgr/dashboard: added recovery throghput graph, improve promqls, fix inventory's card lines

Signed-off-by: Pedro Gonzalez Gomez <pegonzal@redhat.com>
(cherry picked from commit 6fbc9ae01a892f5ce34e7bbf409361fe45641f47)

Merge pull request #50590 from zdover23/wip-doc-2023-03-20-backport-50488-to-quincy

quincy: doc/rados: edit operations/bs-migration (2 of x)

Merge pull request #50587 from zdover23/wip-doc-2023-03-20-backport-50487-to-quincy

quincy: doc/rados: edit operations/bs-migration (1 of x)

doc/rados: edit operations/bs-migration (2 of x)

Disambiguate and improve the English language in
doc/rados/operations/bluestore-migration.rst up to but not including the
section called "Whole Host Replacement".

Co-authored-by: Cole Mitchell <cole.mitchell.ceph@gmail.com>
Signed-off-by: Zac Dover <zac.dover@proton.me>
(cherry picked from commit ca803a24c64059023733e21d755edb9c6c973ecf)

doc/rados: edit operations/bs-migration (1 of x)

Disambiguate and improve the English language in
doc/rados/operations/bluestore-migration.rst up to but not including the
section called "Whole Host Replacement".

Co-authored-by: Cole Mitchell <cole.mitchell.ceph@gmail.com>
Signed-off-by: Zac Dover <zac.dover@proton.me>
(cherry picked from commit 96166b463b1f8800e59f80faebfba1208248725e)

Merge pull request #50582 from zdover23/wip-doc-2023-03-19-backport-50579-to-quincy

quincy: doc/rados: line edit mon-lookup-dns top matter

doc/rados: line edit mon-lookup-dns top matter

Improve the syntax of the top matter of
doc/rados/operations/mon-lookup-dns.rst.

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

Merge pull request #50505 from ifed01/wip-ifed-report-min-alloc-size-qui

quincy: os/bluestore: report min_alloc_size through "ceph osd metadata"

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

Merge pull request #50319 from ifed01/wip-ifed-log-before-assert-in-avl-qui

quincy: os/bluestore: log before assert in AvlAllocator

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

Merge pull request #50566 from zdover23/wip-doc-2023-03-17-backport-50563-to-quincy

quincy: doc/rados: line-edit upmap.rst

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

doc/rados: line-edit upmap.rst

Edit all of doc/rados/upmap.rst.

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

Merge pull request #50485 from cbodley/wip-58955

quincy: rgw/admin: 'bucket stats' displays non-empty time

Reviewed-by: Shilpa Jagannath <smanjara@redhat.com>

Merge pull request #50526 from epuertat/wip-59051-quincy

quincy: mgr/dashboard: force TLS 1.3

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

Merge pull request #50474 from rzarzynski/wip-aclamk-harmonize-bluefs-log-quincy

quincy: os/bluestore: BlueFS: harmonize log read and writes modes

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

mgr/dashboard: force TLS 1.3

Fixes: https://tracker.ceph.com/issues/58942
Signed-off-by: Ernesto Puerta <epuertat@redhat.com>
(cherry picked from commit cd8946631b222dfff1727083d75dfe2dd825d1a4)

os/bluestore: report min_alloc_size through "ceph osd metadata"

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

Merge pull request #50444 from adk3798/wip-58709-quincy

quincy: Add per OSD crush_device_class definition
Reviewed-by: Redouane Kachach <rkachach@redhat.com>

Merge pull request #50448 from adk3798/wip-58883-quincy

quincy: cephadm: set pids-limit unlimited for all ceph daemons

Reviewed-by: Redouane Kachach <rkachach@redhat.com>

Merge pull request #49930 from adk3798/wip-58198-quincy

quincy: mgr/prometheus: fix module crash when trying to collect OSDs metrics

Reviewed-by: Redouane Kachach <rkachach@redhat.com>

Merge pull request #50170 from trociny/wip-58774-quincy

quincy: mgr/cephadm: try to avoid pull when getting container image info

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

Merge pull request #50370 from guits/wip-58910-quincy

quincy: drive_group: fix limit filter in drive_selection.selector

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

Merge pull request #50445 from adk3798/wip-58761-quincy

quincy: cephadm: using short hostname to create the initial mon and mgr

Reviewed-by: Redouane Kachach <rkachach@redhat.com>

Merge pull request #50451 from rhcs-dashboard/wip-58720-quincy

quincy: mgr/dashboard: move service_instances logic to backend

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

Merge pull request #50353 from ceph/wip-yuri-quincy-reef-quincy

qa/tests: added quincy client upgrade => reef

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

rgw/admin: 'bucket stats' displays non-empty mtime

Fixes: https://tracker.ceph.com/issues/58932
Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit e9720ddbe9ae033e14c2f3def51a06a4e204d4c8)

Conflicts:
src/rgw/driver/rados/rgw_bucket.cc -> src/rgw/rgw_bucket.cc

Merge pull request #50453 from zdover23/wip-doc-2023-03-09-backport-50439-to-quincy

quincy: doc/rados/operations: Improve wording, capitalization, formatting

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

Merge pull request #50477 from zdover23/wip-doc-2023-03-11-backport-50470-to-quincy

quincy: doc/index: remove "uniquely" from landing page

doc/index: remove "uniquely" from landing page

Change "Ceph uniquely delivers[...]" to "Ceph delivers[...]"

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

Merge pull request #50446 from adk3798/wip-58776-quincy

quincy: mgr/cephadm: fix backends service in haproxy config with multiple nfs of same rank

Reviewed-by: Redouane Kachach <rkachach@redhat.com>

Merge pull request #50447 from adk3798/wip-58778-quincy

quincy: cephadm: Adding poststop actions and setting TimeoutStartSec to 200s

Reviewed-by: Redouane Kachach <rkachach@redhat.com>

Merge pull request #50327 from Matan-B/wip-matanb-quincy-check-pg-num

quincy: mon/OSDMointor: Simplify check_pg_num()

os/bluestore: BlueFS: harmonize log read and writes modes

BlueFS log has always been written in non-buffered mode.
Reading of it depends on bluefs_buffered_io option.
It is strongly suspected that this causes some wierd problems.

Possibly fixes: https://tracker.ceph.com/issues/54019

Signed-off-by: Adam Kupczyk <akupczyk@ibm.com>
(cherry picked from commit 80e9c5d1c328448f0ef2a836c8a312296e6f49d7)

mgr/cephadm: fix haproxy nfs backend server ip gathering

Fixes: https://tracker.ceph.com/issues/58465
Previously, if there were 2 nfs daemons of the same
rank, we could not check the rank generation, which
is intended to mark which one is the "real" on of that
rank in cases where we cannot remove the other one due
to its host being offline. The nfs of a given rank with
the highest rank_generation is the one we want haproxy
to use for its backend IP. Since we didn't actually
check this, it was random, depending on what order we
happened to iterate over the nfs daemons of the same
rank, which IP we actually got. If the nfs with the
lower rank_generation on an offline host happened
to come later in the iterations, we'd use that one
for the IP, which is incorrect.

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

Conflicts:
src/pybind/mgr/cephadm/tests/test_services.py

mgr/cephadm: don't attempt daemon actions for daemons on offline hosts

They'll just fail anyway, and it will waste time waiting
for the connection to timeout. We have other places in
the serve loop that will check if the host is back
online.

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

doc/rados/operations: Improve wording, capitalization, formatting

Signed-off-by: Anthony D'Atri <anthonyeleven@users.noreply.github.com>
(cherry picked from commit 5f2060c0829992b31a0d17e23d7b1d2fb2280ac0)

Merge pull request #50407 from rzarzynski/wip-57696-quincy

quincy: mon/LogMonitor: Fix log last

Reviewed-by: Neha Ojha <nojha@redhat.com>

Merge pull request #50406 from rzarzynski/wip-58169-quincy

quincy: src/crush: extra logging to debug CPU burn in test_with_fork()

Reviewed-by: Deepika Upadhyay <dupadhya@redhat.com>

Merge pull request #50405 from rzarzynski/wip-58334-quincy

quincy: osd: mds: suggest clock skew when failing to obtain rotating service keys

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

Merge pull request #49989 from pdvian/wip-58639-quincy

quincy: mon: Fix condition to check for ceph version mismatch

Reviewed-by: Neha Ojha <nojha@redhat.com>

Merge pull request #47927 from rzarzynski/wip-55543-quincy

quincy: cmake: re-enable TCMalloc and allocator related cleanups

Reviewed-by: Kefu Chai <kchai@redhat.com>

Merge pull request #50388 from zdover23/wip-doc-2023-03-06-backport-50386-to-quincy

quincy: doc/rgw - fix grammar in table in s3.rst

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

Merge pull request #50125 from rhcs-dashboard/wip-58729-quincy

quincy: mgr/dashboard: fix rbd mirroring daemon health status

Reviewed-by: Pegonzal <NOT@FOUND>
Reviewed-by: Pere Diaz Bou <pdiazbou@redhat.com>
Reviewed-by: siddharthchanana9 <NOT@FOUND>

Merge pull request #50450 from rhcs-dashboard/wip-57483-quincy

quincy: mgr/dashboard: cephadm dashboard e2e fixes

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

Merge pull request #50459 from rhcs-dashboard/custom-image-kcli-bootstrap-quincy

quincy: mgr/dashboard: custom image for kcli bootstrap script

Reviewed-by: Pegonzal <NOT@FOUND>

mgr/dashboard: custom image for kcli bootstrap script

the stable branches like quincy pulls from the quay.io/ceph/ceph:v17 to
bootstrap the ceph cluster in test environments. This will cause issues
because the branches are changing constantly but the image is not. So
using the quay.ceph.io repo to bring the cluster in test environment.

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

 Conflicts:
src/pybind/mgr/dashboard/ci/cephadm/bootstrap-cluster.sh
  - Reject the incoming changes and change the image tag to quincy

osd/OSDMapping: Remove get_num_acting_pgs

Introduced here: bd9813f5e1a3addca1a57360d58b50b120e0e5f3
As check_pg_num() is changed, this method is no longer needed.

Signed-off-by: Matan Breizman <mbreizma@redhat.com>
(cherry picked from commit 0816eb98dba42af57681d361f5244594ef2f53dc)

mon/OSDMonitor: Skip check_pg_num on pool size decrease

When changing the pool size we use check_pg_num to not exceed
`mon_max_pg_per_osd` value. This check should only be applied
when increasing the size to avoid underflows.
(Same already applied when changing pg_num)

Signed-off-by: Matan Breizman <mbreizma@redhat.com>
(cherry picked from commit a7c09bb82cc7749b05a0f60477c9407a177086d1)