Merge pull request #62452 from tchaikov/librbd-std-variant

librbd: migrate from boost::variant to std::variant

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

Merge pull request #62484 from idryomov/wip-70640

mgr/rbd_support: always parse interval and start_time in Schedules::remove()

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

Merge pull request #62512 from anthonyeleven/iptableswarn

doc/rados/configuration: Add iptables change admonition to network-config.rst

doc/rados/configuration: Add iptables change admonition to network-config.rst

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

Merge pull request #62475 from tchaikov/literal-operator

common: remove spaces before literal operator

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

Merge pull request #60277 from rzarzynski/wip-tools-rados-omap-listbench

tools/rados: generalize the OMAP benchmarking to cover listing as well

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

Merge pull request #60159 from rzarzynski/wip-denc-ctcheck-struct_v

dencoding: check struct_v against DECODE_START(v, ...) at compile-time

Reviewed-by: Ilya Dryomov <idryomov@gmail.com>
Reviewed-by: Greg Farnum <gfarnum@redhat.com>
Reviewed-by: Jesse Williamson <jwilliamson@suse.de>
Reviewed-by: Adam Kupczyk <akupczyk@redhat.com>
Reviewed-by: Nitzan Mordechai <nmordech@redhat.com>

Merge pull request #62459 from zdover23/wip-doc-2025-03-24-mon_warn_pg_not_deep_scrubbed-guidance

src/common: add guidance for deep-scrubbing ratio warning

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

Merge PR #62465 into main

* refs/pull/62465/head:
qa: add test for `mgr status` command
mgr: add status command

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

Merge pull request #62215 from rkachach/fix_mgmt_gw_ha

mgr/cephadm: refactor oauth2-proxy certs and cookie-secret handling

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

Merge pull request #62477 from rhcs-dashboard/upgrade-grafana-10.4.16

mgr/cephadm: upgrade grafana to 10.4.16

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

Merge pull request #62229 from phlogistonjohn/jjm-teuth-ctdb-sebool

qa/cephadm/smb: set virt_sandbox_use_netlink selinux bool on ctdb tests

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

Merge pull request #61824 from ShwetaBhosale1/fix_issue_69942_prometheus_not_adding_host_url_in_mgr_when_first_enabled

mgr/cephadm: Deploying prometheus service for the first time, does not update PROMETHEUS_API_HOST url under mgr module

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

Merge PR #62303 into main

* refs/pull/62303/head:
test/libcephfs: add missing cleanup

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

Merge PR #62406 into main

* refs/pull/62406/head:
client: process path without inode anchor
test/client: add ll_walk test with cwd and root path

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

Merge pull request #62423 from afreen23/wip-ns-kb

mgr/dashboard: KiB not supported size for namespace creation

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

Merge pull request #61727 from Kushal-deb/fix_issue_2330954-RGW_is_not_adding_a_SAN

cephadm: Ensure wildcard SAN is included in RGW self-signed certs

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

Merge pull request #61209 from Kushal-deb/fix_issue-2253832-osd_service_issue

cephadm: link new OSDs to existing managed services

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

Merge pull request #62199 from ShwetaBhosale1/fix_issue_70373_enable_logrotate_timer

cephadm: Enable logroate.timer service in bootstrap process

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

Merge pull request #62485 from zdover23/wip-doc-2025-03-25-cephadm-certmgr

doc/cephadm: format "Configuration" in cephadm.rst

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

qa: add test for `mgr status` command

Fixes: https://tracker.ceph.com/issues/70571
Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>

Merge pull request #62282 from adk3798/cephadm-ganesha-fix-server-scope

mgr/cephadm: fix Server_Scope in ganesha.conf

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

Merge pull request #62329 from NitzanMordhai/wip-nitzan-suite-rados-cephadmn-still-running-ignore

suites/rados/cephadm: add still running warning to the ignore list

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

librbd: migrate from boost::variant to std::variant

migrate from boost::variant to std::variant

Complete migration started in commit 017f3339c, replacing boost::variant with
std::variant throughout the librbd codebase. This change is part of our ongoing
effort to reduce third-party dependencies by leveraging C++ standard library
alternatives where possible.

Benefits include:
- Improved code readability and maintainability
- Reduced external dependency surface
- More consistent API usage with other components

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

doc/cephadm: format "Configuration" in cephadm.rst

Improve the formatting in the section "Configuration" in
doc/cephadm/certmgr/cephadm.rst.

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

mgr/rbd_support: always parse interval and start_time in Schedules::remove()

Commit 1b62447071a9 ("mgr/rbd_support: fix schedule remove") addressed
the issue that it was concerned with in a rather suboptimal way: instead
of moving the parsing of interval and start_time upfront to be able to
bail early, it wrapped from_string() constructors with try/finally and
left the conditional behavior in place.

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

Merge pull request #62411 from rhcs-dashboard/notification-icon-fix

mgr/dashboard: fix notification icon size

Merge pull request #62410 from rhcs-dashboard/custom-pool-dashboard

mgr/dashboard: use existing pools for cephfs vol creation

Merge pull request #62027 from rhcs-dashboard/admin-ops-api-for-rgw-accounts

mgr/dashboard: Use admin ops API for rgw accounts

Merge pull request #62415 from rhcs-dashboard/tenanted-bucket-lifecycle-fix

mgr/dashboard: tenanted bucket lifecycle management fix

Merge pull request #62425 from cyx1231st/wip-seastore-cleanups

crimson/os/seastore: misc cleanups

Reviewed-by: Xuehan Xu <xuxuehan@qianxin.com>
Reviewed-by: Matan Breizman <mbreizma@redhat.com>

mgr/cephadm: upgrade grafana to 10.4.16

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

common: remove spaces before literal operator

clang 20 complains when compiling a user-defined literal operator
with spaces before the underscore. so let's drop the space to be
compliant to
[CWG2521](https://cplusplus.github.io/CWG/issues/2521.html).

```
/home/kefu/dev/ceph/src/common/options.h:415:41: warning: identifier '_min' preceded by whitespace in a literal operator declaration is deprecated [-Wdeprecated-literal-operator]
  415 | constexpr unsigned long long operator"" _min (unsigned long long min) {
      |                              ~~~~~~~~~~~^~~~
      |                              operator""_min
```

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

mgr/dashboard: KiB not supported size for namespace creation

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

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

Merge pull request #62464 from idryomov/wip-70632

librbd: respect rbd_default_snapshot_quiesce_mode in group_snap_create()

Reviewed-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>

Merge pull request #62422 from ljflores/wip-pg-upmap-prim-release-note

PendingReleaseNotes; doc/rados/operations: document "rm-pg-upmap-primary-{all}" commands

Merge pull request #62269 from nbalacha/wip-nbalacha-rbd-mirrorpool-errmsg

rbd: print the return codes on error for mirror pool commands

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

mgr: add status command

The Rook operator relies on a `status` (asok) command to the mgr to verify it
is operational. However, this command was actually served by the CephFS
`Client` library that the ceph-mgr daemon statically linked in and used for
sending MDS commands. That statically linked client was removed via 048fc68c
(case insensitive directory tree feature).

So, add a legitimate ceph-mgr status command which simply outputs an empty
dictionary and returns success (0). TBD on adding useful information.

Fixes: 048fc68c517f50b9978457f478ca4638f01caa09
Fixes: https://tracker.ceph.com/issues/70571
Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>

Merge pull request #54957 from jzhu116-bloomberg/wip-63799

rgw: revert PR #41897 to allow multiple delete markers to be created

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

librbd: don't use public API flags in Group::snap_create()

Use internal flags obtained from snap_create_flags_api_to_internal()
instead -- internal_flags variable was assigned but not used.

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

librbd: respect rbd_default_snapshot_quiesce_mode in group_snap_create()

Make group_snap_create() behave the same as snap_create() and
mirror_image_create_snapshot(): APIs that don't take RBD_SNAP_CREATE_
flags explicitly should respect rbd_default_snapshot_quiesce_mode
option.

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

Merge pull request #60684 from salieri11/wip-igolikov-asok-56442

mds: add MDS asok command for dumping stray directories

Merge PR #62439 into main

* refs/pull/62439/head:
.github/workflows,script: add test for missing .qa links
qa: add missing .qa links

Reviewed-by: Ernesto Puerta <epuertat@redhat.com>

src/common: add guidance for deep-scrubbing ratio warning

Add an explanation of how to set the value of
"mon_warn_pg_not_deep_scrubbed_ratio" to the confval definition of that
variable. Although this variable contains the string "mon", it is set on
the Manager. I have added a note to direct users to set this value on
the Manager.

This issue was pointed out by Petr Tlapa on Slack in late March of 2025.

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

mgr/dashboard: tenanted bucket lifecycle management fix

Fixes: https://tracker.ceph.com/issues/70588
Signed-off-by: Naman Munet <naman.munet@ibm.com>
Before: As the tenant was not passed in lifecycle calls made through bucket details > tiering tab, & also account_id was not resolved to the user of the bucket, hence not able to manage the Tiering configuration lifecycle changes
After: Account_id is resolved to user_id and tenant is additionally passed to the lifecycle API so as to handle the tenanted bucket scenario

Merge pull request #62430 from ankitk18/fix/alignment-action-btn

mgr/dashboard:fixed alignment of create action button

rbd: print the return codes on error

The calls in the mirror pool cli do not always print the error code.
This is now done in order to make debugging easier.

Fixes: https://tracker.ceph.com/issues/70444
Signed-off-by: N Balachandran <nithya.balachandran@ibm.com>

Merge pull request #62366 from anthonyeleven/improve-radosgw-layout

doc/radosgw: Improve layout.rst

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

Merge pull request #62442 from anthonyeleven/improve-cephadm-svc-osd

doc/cephadm/services: Add mention of --zap for OSD removal

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

doc/cephadm/services: Add mention of --zap for OSD removal

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

Merge pull request #62413 from ceph/pin-actions

workflows: Pin specific SHAs

PendingReleaseNotes; doc/rados/operations: document "rm-pg-upmap-primary-{all}" commands

Added a release note for `ceph osd rm-pg-upmap-primary` and documentation to
complement #62190.

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

.github/workflows,script: add test for missing .qa links

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

mgr/dashboard:fixed alignment of create action button

Fixes:https://tracker.ceph.com/issues/70433
Signed-off-by: Ankit Kumar <51ankitkp@gmail.com>

qa: add missing .qa links

    find qa/suites/ -type d -execdir ln -sfT ../.qa/ {}/.qa \;

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

Merge pull request #62370 from laimis9133/main

doc/rados/troubleshooting/log-and-debug: add separate section for Monitors

mgr/dashboard: e2e for cephfs creation with existing pools

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

mgr/dashboard: use existing pools for cephfs vol creation

We can use the newly introduced data and metadata params to create a vol
with those pools.

UI is being intelligent by filtering out the used pools and only uses
the pools that are labeled by cephfs and also not in use. To figure out
a pool is in use or not, we are fetching the pool stats and checking its
used_bytes.

Note: Using ec pools for data pool layout is something discouraged
according to offical doc: https://docs.ceph.com/en/latest/cephfs/createfs/#creating-a-file-system
We can force it but for now I have disabled it entirely in the dashboard
unless people say its okay to do it.

One more extra thing I am doing here is to add a note on deleting a
filesystem that the underlying pools and mds daemons will be removed.

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

Merge pull request #62101 from rhcs-dashboard/nfs-fix

mgr/dashboard: NFS - Fix Edit NFS export request

Merge pull request #60631 from kamoltat/wip-ksirivad-fix-3az-unset

[3AZ] src/mon : Allow user to specify crush_rule, size and min_size when unsetting a stretch pool

Merge pull request #62100 from rhcs-dashboard/bucket-lifecycle-fixes

mgr/dashboard: bucket lifecycle fixes after using xmltodict package

Reviewed-by: Afreen Misbah <afreen@ibm.com>
Reviewed-by: Aashish Sharma <aasharma@redhat.com>

mgr/dashboard: Use admin ops API for rgw accounts

Fixes: https://tracker.ceph.com/issues/69487
Signed-off-by: Naman Munet <naman.munet@ibm.com>

mgr/dashboard: NFS - Fix Edit NFS export request

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

crimson/os/seastore/transaction: cleanup add_absent/present_to_retired_set

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

Merge pull request #61646 from aclamk/wip-aclamk-teuto-better-bluestore-testing

qa/rados: Augmented bluestore testing

Merge pull request #61745 from soumyakoduri/wip-skoduri-glacier-cli

rgw/cloud-restore [PART1] : Add Restore support from Glacier/Tape cloud endpoints

Merge pull request #62426 from anthonyeleven/fixup-db_slots

doc/cephadm/services: Correct indentation in osd.rst

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

doc/cephadm/services: Correct indentation in osd.rst

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

crimson/os/seastore/cache: make sure on_cache is called after extent initialization

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

crimson/os/seastore: misc renames

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

crimson/os/seastore: duplicate_for_write() to move set_laddr() into Cache

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

crimson/os/seastore: improve is_logical() checks

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

crimson/os/seastore/lba_manager: drop unused get_mappings()

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

crimson/os/seastore: drop unused inc/decref_extent and dec_ref with tests

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

Merge pull request #61732 from rishabh-d-dave/mgr-vol-pools

mgr/vol: allow passing pool names to "fs volume create" cmd

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

Merge pull request #61084 from adamemerson/wip-boost-1.87

Boost 1.87

Reviewed-by: Matan Breizman <Matan.Brz@gmail.com>

Merge pull request #62285 from cbodley/wip-rgw-iam-arnlike

rgw/iam: add policy evaluation for Arn-based Conditions

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

Merge pull request #62368 from ivancich/wip-radoslist-improvements

rgw: radoslist improvements primarily to better support gap list tool

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

Merge pull request #62166 from rkachach/fix_issue_70352

mgr/cephadm: use ip_hash to enforce the same prometheues upstream

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

mgr/cephadm: set the default loglevel for mgmt-gateway to warn

Signed-off-by: Redouane Kachach <rkachach@ibm.com>

workflows: Pin specific SHAs

Fixes https://www.wiz.io/blog/new-github-action-supply-chain-attack-reviewdog-action-setup

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

Merge pull request #61873 from rhcs-dashboard/nfs-create-cluster

mgr/dashboard: NFS Cluster and Export Listing

Merge pull request #62387 from rhcs-dashboard/user-form-display-name-fix

mgr/dashboard: Not able to edit user display/full name in user form

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

mgr/dashboard: fix notification icon size

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

Merge pull request #62397 from aainscow/interval_set_qf

test/common: skip google tests which create core dumps in test_interval_set

Reviewed-by: Bill Scales <bill_scales@uk.ibm.com>
Reviewed-by: Adam Kupczyk <akupczyk@ibm.com>

Merge pull request #61937 from vshankar/wip-file-block-diff

mds/client: file blockdiff support

Merge pull request #62244 from ronen-fr/wip-rf-cast-conf

common, osd: add cmd_getval_cast_or()

Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
Reviewed-by: Alex Ainscow <aainscow@uk.ibm.com>

cephadm: Enable logroate.timer service in bootstrap process

Fixes: https://tracker.ceph.com/issues/70373
Signed-off-by: Shweta Bhosale <Shweta.Bhosale1@ibm.com>

PendingReleaseNote: add note that "volume create" accepts pool names...

in args.

Add a release note that "ceph fs volume create" command allows users to
pass pool names to "ceph fs volume create" command.

Signed-off-by: Rishabh Dave <ridave@redhat.com>

doc/cephfs: mention new options for "fs volume create" cmd

Command "ceph fs volume create" accepts 2 new options to allow users to
pass data and metadata pool name. Update docs to include mention of both
the options.

Signed-off-by: Rishabh Dave <ridave@redhat.com>

mgr/nfs: NFS Cluster and Export Listing

Fixes: https://tracker.ceph.com/issues/70022
Signed-off-by: Achint Kaur <ackaur@redhat.com>

Merge pull request #61075 from rhcs-dashboard/topicmanagment

mgr/dashboard: Add RGW topics endpoint creation for create ,delete and list in dashboard

Merge pull request #62323 from clwluvw/bucket-replication-mismatch

rgw: reject PutBucketReplication policies with mismatched statuses, skip sync on update

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

Merge pull request #62149 from rhcs-dashboard/fix-access-perm-ui

mgr/dashboard: fix access control permissions for roles

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

rgw: radolist clean up, calculate obj info slightly earlier

With recent changes, it's better to determine bucket and object names
earlier.

Signed-off-by: J. Eric Ivancich <ivancich@redhat.com>

rgw: modify radoslist to better support the rgw-gap-list tool

When the `radosgw-admin bucket radoslist ...` sub-command was
introduced, it was written specifically for finding orphans. It has
since been updated to work for finding gaps, that is indexed RGW
objects that are missing one or more supporting rados objects.

When a head object was not found, it was ignored. Now it does produce
output with the oid and related information for the missing head
object.

Signed-off-by: J. Eric Ivancich <ivancich@redhat.com>

rgw: fix regression in radoslist with SLO manifests

A regression was inadvertently introduced in commit
bcd7883d7212c96ebfb89c938c79fc7efbb80d2f that then prevented
`radosgw-admin bucket radoslist ...` from working properly with
buckets using SLO manifests. This corrects that regression.

Signed-off-by: J. Eric Ivancich <ivancich@redhat.com>

client: process path without inode anchor

ll_walk was wrongly coded to make the input file path relative to ino=0. That inode
does not exist and isn't considered the root inode.

Fixes: https://tracker.ceph.com/issues/70573
Fixes: e08210dda01f4e154561fcf008084e6d640e4683
Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>

test/client: add ll_walk test with cwd and root path

Fixes: https://tracker.ceph.com/issues/70573
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>

Merge pull request #62394 from laimis9133/laims9133-notiffix

doc/radosgw/notifications: fix topic details

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