mgr/volumes: reduce noisy health check logs

Previously, the manager logged connection cleanup messages at info level
even when no idle connections existed, adding unnecessary noise to logs.

This change logs cleanup actions at info level only when idle connections
are found, and moves the 'no idle connections' message to debug level.

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

Merge pull request #62887 from clwluvw/copy-obj-remote

rgw: report copy obj progress from the frontend CR

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

Merge pull request #65415 from kchheda3/wip-re-create-bucket

rgw/create-bucket: Do not re-create the bucket instance & bucket index shard object if if bucket already exist..

Merge pull request #64152 from cbodley/wip-63323

rgw: support S3 Object Ownership controls to disable object ACLs

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

Merge PR #65392 into main

* refs/pull/65392/head:

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

Merge PR #65577 into main

* refs/pull/65577/head:

Reviewed-by: Venky Shankar <vshankar@redhat.com>
Reviewed-by: Patrick Donnelly <pdonnell@ibm.com>

Merge pull request #64893 from zer0def/bug-57939

cephadm: pass wal/db slot count to ceph-volume

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

Merge pull request #63170 from adk3798/cephadm-nvmeof-stray

mgr/cephadm: don't mark nvmeof daemons without pool and group in name as stray

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

Merge pull request #65548 from EncoreTechnologies/cephadmin-vir-ip-networks-interfaces-check

mgr/cephadm: Add check for virtual_interface_networks

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

Merge pull request #65373 from phlogistonjohn/jjm-cephadm-build-ensure-byte-comp

cephadm: fail the script if byte-compile step fails

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

Merge pull request #65155 from phlogistonjohn/jjm-py-common-fmt-black

python-common: enable consistent code formatting with 'black'

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

Merge pull request #63962 from adk3798/cephadm-signal-containerized-daemon

cephadm: add command to send signals to daemons

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

cephfs-journal-tool:: Don't reset the journal trim position

If the fs had to go through journal recovery and reset,
the cephfs-journal-tool resets the journal trim position
because of which the old unused journal objects just stay
forever in the metadata pool. The patch fixes the issue.
Now, the old stale journal objects are trimmed during the
regular trimming cycle helping to recover space in the
metadata pool.

Fixes: https://tracker.ceph.com/issues/69708
Signed-off-by: Kotresh HR <khiremat@redhat.com>

qa: Validate cephfs-journal-tool reset trim

Validates that the cephfs-journal-tool reset
doesn't reset the trim position so that the
journal trim takes care of trimming the older
unused journal objects helping to recover the
space in metadata pool.

Fixes: https://tracker.ceph.com/issues/69708
Signed-off-by: Kotresh HR <khiremat@redhat.com>

Merge pull request #65510 from anthonyeleven/eugen

src/cephadm/cephadmlib/daemons: Fix typos in ceph.py and elsewhere

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

Merge pull request #65322 from ShwetaBhosale1/fix_issue_72345_set_healthwarning_for_ssh_timeout_during_refresh

mgr/cephadm: set health warning for host SSH timeout

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

Merge pull request #65291 from ShwetaBhosale1/fix_issue_72774_spec_reapply_causing_osd_to_apply_in_every_serve_loop

mgr/cephadm: After reapplying the osd spec, the OSD services are continuously applied in each serve loop iteration

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

Merge pull request #65149 from ShwetaBhosale1/fix_issue_72652_max_num_of_osds_to_upgrade_in_single_iteration

mgr/cephadm: Config parameter to set the max number of OSDs to upgrade in single iteration

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

Merge pull request #64642 from ShwetaBhosale1/fix_issue_72206_Allow_registry_credentials_to_define_multiple_container_registries

mgr/cephadm: Allow registry credentials to define multiple container registries

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

Merge pull request #65398 from smanjara/wip-fix-zg-api-name

rgw/multisite: update api_name during a zonegroup rename

Merge pull request #64018 from 9401adarsh/wip-adashoka-71677

rgw/notifications: add support for sharded persistent bucket notifications

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

Merge pull request #65366 from cbodley/wip-72847

rgw/admin: allow listing account's root users

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

Merge pull request #65528 from mkogan1/wip-d4n-odsync

rgw/d4n: change cache write ::open() flags

Merge pull request #65166 from myoungwon/wip-fix-rbm-mergeable-block

crimson/os/seastore: correct possible mergeable size for ool extents

Reviewed-by: Xuehan Xu <xuxuehan@qianxin.com>

Merge pull request #59608 from VallariAg/wip-nvmeof-teuthology-v7-upgrade

qa/suites/nvmeof: add upgrade sub-suite

Merge pull request #65565 from salieri11/igolikov-bug-73053

mds: remove unused enum values for subvolume metrics

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

Merge pull request #65394 from athanatos/sjust/wip-seastore-cleanups-2025-08

seastore: misc cleanups, coroutine conversions

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

Merge pull request #65399 from anthonyeleven/improve-global-yml

src/common/options: Improve global.yaml.in

Merge pull request #65372 from adamemerson/wip-70916-bis

test/neorados: Catch timeouts in Watch/Poll test

Merge pull request #65532 from phlogistonjohn/jjm-jenk-hint

github: add a note reminding to use one command per comment

Merge pull request #65553 from ceph/pr-checklist-fix

.github/workflows/pr-checklist.yml: Update

test/.../test_cbjournal: destroy CircularBoundedJournal instance prior to reactor shutdown

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

crimson/.../circular_bounded_journal: inline do_submit_record into submit_record

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

crimson/.../circular_bounded_journal: add journal submission metrics

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

crimson/.../circular_bounded_journal: convert do_submit_record to coroutine

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

crimson/.../circular_bounded_journal: convert submit_record to coroutine

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

crimson/.../transaction_manager: convert do_submit_transaction to coroutine

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

crimson/.../extent_placement_manager: convert BackgroundProcess::run() to coroutine

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

Merge pull request #64978 from athanatos/sjust/wip-mclock-cleanup

mclock: avoid using config keys for profiles, remove MonClient from MclockConfig

Reviewed-by: Matan Breizman <mbreizma@redhat.com>
Reviewed-by: Mohit Agrawal <moagrawa@redhat.com>
Reviewed-by: Sridhar Seshasayee <sseshasa@redhat.com>

cephadm: add command to send signals to daemons

This command figures out the container id for the
daemon based on the given name and then uses
a docker/podman to send that signal to the container

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

qa: Add nvmeof:upgrade suite

Signed-off-by: Vallari Agrawal <vallari.agrawal@ibm.com>

Merge PR #65516 into main

* refs/pull/65516/head:
common: remove unused attribute

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

Merge pull request #65559 from nbalacha/wip-nbalacha-cleanup-1

rgw/bucket-logging: minor code cleanup

mds: remove unused enum values for subvolume metrics

Signed-off-by: Igor Golikov <igolikov@redhat.com>
Fixes: https://tracker.ceph.com/issues/73053

mgr/cephadm: set a healthwarning for host SSH timeout

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

Merge pull request #63089 from smanjara/wip-fix-object-lock-rule

rgw/multisite: sync put-object-lock-configuration

Merge pull request #64428 from sungjoon-koh/add-wildcard-support-cond-read

rgw: add wildcard "*" support for conditional read

rgw/bucket-logging: minor code cleanup

Fixes a typo in a variable name and the placement
of a comment.

Signed-off-by: N Balachandran <nithya.balachandran@ibm.com>

Merge pull request #65481 from rhcs-dashboard/snapshot-schdule-interval

mgr/dashboard: fix missing schedule interval in rbd API

.github/workflows/pr-checklist.yml: Update

This newer version of the action does not fail mysteriously if a contributer
removes the example Jenkins commands from the end of the PR description.

Signed-off-by: Zack Cerza <zack@cerza.org>

Merge pull request #62469 from igomon-bloomberg/wip-timestamp-ver-epoch

rgw: implements timestamp-based epochs for versioned objects

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

Merge pull request #64917 from aainscow/ec_fixpack3_pr

EC fixpack 3

Merge pull request #65370 from kchheda3/wip-fixed-acl-account-migration

rgw/account: bucket acls are not completely migrated once the user is migrated to an account

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

mgr/cephadm: Change spec to ingress_spec

Changed from using spec to ingress_spec to be consistent with the function and fix the spec errors.

Fixes: https://tracker.ceph.com/issues/66874
Signed-off-by: Bradley Bishop <bishopbm1@gmail.com>

rgw: implement timestamp-based epochs.

Signed-off-by: Igor Gomon <igomon@bloomberg.net>

mgr/cephadm: Add check for virtual_interface_networks

Added an additional look up for if a match with the ceph networks is not found it will check the virtual_interface_networks list from the config that will enable matching to those networks, enabling dummy IP usage as described in the docs: https://docs.ceph.com/en/reef/cephadm/services/rgw/#selecting-network-interfaces-for-the-virtual-ip.

Fixes: https://tracker.ceph.com/issues/66874
Signed-off-by: Bradley Bishop <bishopbm1@gmail.com>

Merge pull request #65536 from bluikko/doc-services-mon-improvements-cephadm

doc/cephadm: Fix errors and improvements in services/monitoring.rst

Merge pull request #65498 from adamemerson/wip-rgw-weekly-baseline-prio-subset

qa: Run fewer RGW jobs at higher priority every week

Reviewed-by: Casey Bodley <cbodley@redhat.com>
Reviewed-by: J. Eric Ivancich <ivancich@redhat.com>

Merge pull request #65530 from rkachach/fix_issue_73025

doc: fixing starting release of mgmt-gateway/oauth2-proxy services

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

Merge pull request #64771 from Matan-B/wip-matanb-crimson-seastore-coro

crimson/os/seastore/segment_manager: Check segments prior to mkfs

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

Merge pull request #64970 from edwinzrodriguez/ceph-wip-72479

crush/mapper: Mark pushed variable as maybe unused

Merge pull request #65227 from tchaikov/wip-generate-brace-init

src: replace push_back(T{}) with emplace_back()

Reviewed-by: Ronen Friedman <rfriedma@redhat.com>
Reviewed-by: Adam C. Emerson <aemerson@redhat.com>
Reviewed-by: Ilya Dryomov <idryomov@gmail.com>

doc/cephadm: Fix errors and small improvements in services/monitoring.rst

Add that certmgr is new in version Tentacle.

Fix broken links with formatting rendered as text.

Capitalize consistently in text HTTP, YAML, URL, Alertmanager, Grafana,
centralized logging; Prometheus or use inline preformat with lower case
when referring to the module name.

Use inline preformat consistently for CLI commands within text
paragraphs.

Use title case in section titles.

Trim section title underline syntax.

Use consistently pre-formatted blocks bash prompt, YAML.

Start all block formatting paragraphs at the left margin instead of
indenting.

Signed-off-by: Ville Ojamo <14869000+bluikko@users.noreply.github.com>

doc: github: add a note reminding to use one command per comment

When I started using ceph I also thought it would be natural to issue >1
command per comment, but it doesn't work. I saw someone else doing it
recently and thought it would be good to have this reminder in the PR
template.

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

src/common/options: Improve global.yaml.in

Improve and correct option descriptions. Filestore options are marked deprecated to stress that Filestore should no longer be used.

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

Merge pull request #65514 from phlogistonjohn/jjm-bwc-arg-grouping

build-with-container: add argument groups to organize options

build-with-container: add argument groups to organize options

Use the argparse add_argument_group feature to organize the mass of
arguments into more sensible categories. Hopefully, someone reading
over the `--help` output can now more easily see options that
are useful rather than being overwhelmed by a wall of text.

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

doc: fixing starting release of mgmt-gateway/oauth2-proxy services

Fixes: https://tracker.ceph.com/issues/73025
Signed-off-by: Redouane Kachach <rkachach@ibm.com>

Merge pull request #65456 from yuvalif/wip-yuval-72940

rgw/logging: rollover objects when conf changes

src/cephadm/cephadmlib/daemons: Fix typos in ceph.py and Dashboard messages

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

python-common/tox.ini: enable black formatting

Initially this is only for a few smaller packages/files that I don't
need to wrestle with to make it work with black.

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

python-common/fs: reformat earmarking.py with black

Use the black formatter (with a single manual tweak) to reformat
the earmarking.py file. This formatting will be enforced by a future
change.

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

python-common: reformat utils.py with black

Use the black formatter (with a single manual tweak) to reformat
the utils.py file. This formatting will be enforced by a future
change.

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

python-common/cephadm: reformat images.py with black

Use the black formatter (with a single manual tweak) to reformat
the images.py file. This formatting will be enforced by a future
change.

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

Merge pull request #65521 from shraddhaag/wip-shraddhaag-availability-release-notes

doc: update data availability release notes

Merge pull request #64728 from rhcs-dashboard/wip-aashish-log-promtail-main

mgr/dashboard: Migrate from promtail to grafana alloy

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

rgw/d4n: change cache write ::open() flags

changing the default flags to O_DSYNC
a small performance improvement is observable

Signed-off-by: Mark Kogan <mkogan@ibm.com>

Merge pull request #65170 from rhcs-dashboard/tiering-tracker-72569

[mgr/dashboard]: Mismatch in the object size observed in Advance section in Tiering.

Reviewed-by: Naman Munet <nmunet@redhat.com>
Reviewed-by: Pedro Gonzalez Gomez <pegonzal@redhat.com>

crimson/os/seastore/segment_manager: Check segments prior to mkfs

Verify Segment count for prior to mkfs
* ExtentPlacementManager::init - dynamic_max_rewrite_generation segments
* SegmentedJournal segment (1)

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

crimson/os/seastore: BlockSegmentManager::primary_mkfs coroutines

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

common/options/crimson.yaml.in: add hot_tier_generations minimum

See ExtentPlacementManager::init

dynamic_max_rewrite_generation = hot_tier_generations - 1;
ceph_assert(dynamic_max_rewrite_generation > MIN_REWRITE_GENERATION);

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

crimson/os/seastore/epm: Add logs and comments

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

crimson/tools/store_bench: update example, switch to --debug

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

crimson/os/seastore: Remove error on expected logic

In read_meta we should not use ERROR when the key is not found
as this is expected.
Same in _mkfs, if mkfs_done exists we shouldn't use ERROR.
This can be confusing when debugging is disabled and few ERROR log
lines are printed.

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

crimson/os/seastore: add logs around mkfs

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

crimson/os/seastore: SeaStore::mkfs() into coroutines

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

crimson/os/seastore: prepare_meta() into coroutines

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

crimson/os/seastore: test_mkfs() into coroutines

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

Merge pull request #65494 from yuvalif/wip-yuval-72543

rgw/logging: add error message when log_record fails

doc: update data availability release notes

Signed-off-by: Shraddha Agrawal <shraddhaag@ibm.com>

Merge pull request #65388 from jamiepryde/main-choose-pdw-over-rmw

osd: Choose PDW over traditional RMW if number of reads required is equal

Merge pull request #65181 from qiuxinyidian/lttng

rados/lttng: add lttng open *tp.so error message

Merge pull request #65172 from shraddhaag/wip-shraddhaag-availability-frequency-config

mon: add config option to change availability score update interval

qa/rgw: add test for cross-zonegroup copy obj

Signed-off-by: Seena Fallah <seenafallah@gmail.com>

rgw: report copy obj progress from the frontend CR

To resolve issues with reporting progress during object copy
operations, we now report progress from the frontend CR instead
of from the curl thread. Reporting progress directly from the
curl thread was causing issues where the final send_response()
call in RGWCopyObj() would not resuming the yield after writing
to client.

By running the fetch_remote_obj() in a separate coroutine and
reporting the progress back through the frontend coroutine, we
avoid this blockage.

Fixes: https://tracker.ceph.com/issues/70840
Signed-off-by: Seena Fallah <seenafallah@gmail.com>

Merge PR #65499 into main

* refs/pull/65499/head:
script/redmine-upkeep: reset merge state after PR change

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

common: remove unused attribute

This resolves an obnoxious compiler warning:

    [79/768] Building CXX object src/mgr/CMakeFiles/ceph-mgr.dir/__/ceph_mgr.cc.o
    In file included from /run/host/home/sepian/ceph/src/global/global_init.h:23,
                     from /run/host/home/sepian/ceph/src/ceph_mgr.cc:27:
    /run/host/home/sepian/ceph/src/common/ceph_context.h:153:42: warning: ‘maybe_unused’ attribute ignored [-Wattributes]
      153 |   [[maybe_unused]] std::unique_ptr<char> _ex_handler;
          |                                          ^~~~~~~~~~~

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

Merge pull request #65350 from adamemerson/wip-perfcounters-unique-string

common: Allow PerfCounters to return a provided service ID

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

Merge pull request #65507 from MaxKellermann/log__includes

log: add missing includes

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

rgw/create_bucket: for swift:create_bucket(), should stop creating new bucket_instance_id object and bucket_index shard objects in rados and just update the bucket_instance_info

Signed-off-by: kchheda3 <kchheda3@bloomberg.net>