Merge pull request #53732 from pereman2/bluefs-expand-bug

os/bluestore: fix bluefs _extend_log seq update

Merge pull request #53986 from zdover23/wip-doc-2023-10-12-rados-troubleshooting-troubleshooting-osd-2-of-x

doc/rados: Edit troubleshooting-osd (2 of x)

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

doc/rados: Edit troubleshooting-osd (2 of x)

Edit doc/rados/troubleshooting/troubleshooting.rst (2 of x).

Follows https://github.com/ceph/ceph/pull/53936.

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

Merge pull request #53234 from xxhdx1985126/wip-seastore-onode-manager-replay

crimson/os/seastore/onode_manager: populate delta recorders for each onode modification

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

Merge pull request #53914 from idryomov/wip-63149

qa/suites/rbd: drop cache tiering workload tests

Reviewed-by: Ramana Raja <rraja@redhat.com>
Reviewed-by: Mykola Golub <mgolub@suse.com>

Merge pull request #51416 from adk3798/jaeger-agent-dep

mgr/cephadm: make jaeger-collector a dep for jaeger-agent

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

Merge pull request #53980 from ceph/wip-yuri-crontab-main

qa/tests: fixed typo and added `--force-priority` in missing places

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

Merge pull request #53062 from xxhdx1985126/wip-seastore-list-objects

crimson/os/seastore: return ghobject_t::max as the end when list_objects reaches the end of the listing

Reviewed-by: Yingxin Cheng <yingxin.cheng@intel.com>
Reviewed-by: Chunmei Liu <chunmei.liu@intel.com>

Merge pull request #53936 from zdover23/wip-doc-2023-10-11-rados-troubleshooting-troubleshooting-osd-1-of-x

doc/rados: edit troubleshooting-osd (1 of x)

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

doc/rados: edit troubleshooting-osd (1 of x)

Edit doc/rados/troubleshooting/troubleshooting-osd.

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

Merge pull request #53971 from zdover23/wip-doc-2023-10-12-start-get-involved-slack

doc/start: add Slack to get-involved

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

qa/tests: fixed typo and added `--force-priority` in missing places

Signed-off-by: Yuri Weinstein <yweinste@redhat.com>

Merge pull request #53926 from zmc/workunit-assert-client-role

 qa/tasks/workunit: improve assert on client names

doc/start: add Slack to get-involved

Add a link to the Ceph Slack to doc/start/get-involved.rst.

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

Merge pull request #53846 from cbodley/wip-62989-again

rgw: fix http error checks in keystone/barbican/vault clients

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

Merge pull request #53579 from Matan-B/wip-matanb-reset_purged_snaps_last

osd/OSD: introduce reset_purged_snaps_last

Reviewed-by: Samuel Just <sjust@redhat.com>
Reviewed-by: Ilya Dryomov <idryomov@gmail.com>
Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>

Merge pull request #53908 from zdover23/wip-doc-2023-10-10-troubleshooting-troubleshooting-memory-profiling

doc/rados: edit memory-profiling.rst

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

Merge pull request #53930 from ceph/wip-yuri-crontab-main

qa/tests: Added upgrade/reef-x to the mix

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

qa/tests: Added upgrade/reef-x to the mix

Signed-off-by: Yuri Weinstein <yweinste@redhat.com>

Merge pull request #53085 from ceph/wip-yuriw-reef-x-main

qa/tests: initial reef-x check in

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

workunit: Improve assert on client names

Instead of a bare AssertionError, we can mention the problematic value.

Signed-off-by: Zack Cerza <zack@redhat.com>

Merge pull request #53862 from adk3798/nvmeof-upgrade

mgr/cephadm: fix upgrades with nvmeof

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

Merge pull request #53900 from ljflores/wip-fix-stress-split-tests

qa/suites/upgrade: fix env indentation in stress-split upgrade tests

doc/rados: edit memory-profiling.rst

Edit doc/rados/troubleshooting/memory-profiling.rst.

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

qa/tasks/ceph_manager: thrash - add reset_purged_snaps_last

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

Merge pull request #50132 from aruniiird/add-rbd-mirror-mon-alerts

ceph-mixin: Add RBD Mirror monitoring alerts

qa/suites/rbd: drop redundant ignorelist entries

CACHE_POOL_NO_HIT_SET is retained in *api_tests*.yaml and
rbd_mirror.yaml snippets for TestLibRBD.ListChildrenTiered and
TestClusterWatcher.CachePools tests.

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

crimson/os/seastore: return ghobject_t::max as the end when list_objects
reaches the end of the listing

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

qa/suites/rbd: deduplicate (data) pool facets

With cache tiering facets gone, "pool" facets are strictly about
--data-pool option now.  Rename to "data-pool" and create symlinks
to a common directory.

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

qa/suites/rbd: drop cache tiering workload tests

Cache tiering facets have been a constant source of job timeouts
accompanied by "slow request" warnings on the OSDs for at least two
years.  Same workloads pass without pool/small-cache-pool.yaml or
thrashers/cache.yaml.

See cache tiering deprecation note added in commit 535b8db33ea0 ("doc:
deprecate the cache tiering").

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

Merge pull request #53719 from rishabh-d-dave/mon-fscmds

 mon/FSCommands: clean up FSCommands

Reviewed-by: Leonid Usov <leonid.usov@ibm.com>
Reviewed-by: Xiubo Li <xiubli@redhat.com>
Reviewed-by: Kotresh HR <khiremat@redhat.com>
Reviewed-by: Laura Flores <lflores@redhat.com>

crimson/os/seastore/onode_manager: drop unnecessary status

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

crimson/os/seastore/onode_manager: avoid unnecessary delta related
overhead

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

Merge PR #53885 into main

* refs/pull/53885/head:
Revert "mds: disable delegating inode ranges to clients"

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

Merge pull request #53517 from cbodley/wip-qa-distros-s

qa/distros: remove centos/rhel8 and ubuntu20.04 from supported distros

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
Reviewed-by: Ilya Dryomov <idryomov@redhat.com>
Reviewed-by: Ali Maredia <amaredia@redhat.com>

qa/suites/upgrade: fix env indentation in stress-split upgrade tests

This is an issue with the stress-split yaml files, as introduced in https://github.com/ceph/ceph/pull/51889.

The stress-split tests have an incorrectly-intented "env" section, which teuthology detects as an entry for "clients".

Fixes: https://tracker.ceph.com/issues/63158
Signed-off-by: Laura Flores <lflores@ibm.com>

Merge pull request #53829 from ajarr/wip-63009

librbd: kick ExclusiveLock state machine stalled waiting for lock from reacquire_lock()

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

Merge pull request #53722 from rishabh-d-dave/mon-authmon

mon/AuthMonitor: clean up AuthMonitor

Reviewed-by: Leonid Usov <leonid.usov@ibm.com>
Reviewed-by: Xiubo Li <xiubli@redhat.com>
Reviewed-by: Kotresh HR <khiremat@redhat.com>
Reviewed-by: Laura Flores <lflores@redhat.com>

Merge pull request #53721 from rishabh-d-dave/mon-mdsmon

mon/MDSMonitor: clean up MDSMonitor

Reviewed-by: Leonid Usov <leonid.usov@ibm.com>
Reviewed-by: Xiubo Li <xiubli@redhat.com>
Reviewed-by: Kotresh HR <khiremat@redhat.com>
Reviewed-by: Laura Flores <lflores@redhat.com>

Merge pull request #53405 from rishabh-d-dave/ceph-auth-caps-val-caps

mon/AuthMonitor: make "ceph auth caps" print error messsages

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

Merge PR #53099 into main

* refs/pull/53099/head:
script: update ceph-debug-docker for centos 9.stream

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

Revert "mds: disable delegating inode ranges to clients"

This isn't necessary -- the MDS handles delegating inode ranges
to clients from its preallocated inode set properly - the suspected
bug involving not persisting the sessionmap and causing asserts
during replay isn't an issue. The preallocated set is persisted
with the log event and the MDS correctly rebuild the set from
this information during replay.

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

Merge PR #53206 into main

* refs/pull/53206/head:
mds: use LogSegment dump for debugging

Reviewed-by: Leonid Usov <leonid.usov@ibm.com>
Reviewed-by: Venky Shankar <vshankar@redhat.com>

Merge pull request #52254 from vedanshbhartia/coverity_uninit

rgw: Add coverity uninitialized variable and initialize RGWBucketEntryMetadataObject

reviewed-by: yuvalif

Merge pull request #52328 from vedanshbhartia/coverity_1512267

rgw: fix potential null dereference in rgw_iam_policy.c: ParseState::do_string

reviewed-by: yuvalif

Merge pull request #52472 from vedanshbhartia/coverity_1510724

rgw: Remove unnecessary null check from valid_s3_bucket_name

reviwed-by: soumyakoduri , yuvalif

Merge pull request #52734 from vedanshbhartia/coverity_ostream

rgw: Restore ostream format state after changing it

reviwed-by: yuvalif

Merge pull request #52326 from yuvalif/wip-yuval-lua-reload

rgw/lua: support reloading lua packages on all RGWs

reviwed-by: dang, cbodle, anthonyeleven

Merge pull request #53890 from zdover23/wip-doc-2023-10-09-troubleshooting-troubleshooting-mon-4-of-x

doc/rados: edit troubleshooting-mon.rst (4 of x)

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

Merge pull request #53892 from rishabh-d-dave/fix-test_cephfs.py

src/test/pybind: don't use decorator "with_setup"

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

doc/rados: edit troubleshooting-mon.rst (4 of x)

Edit doc/rados/troubleshooting/troubleshooting-mon.rst.

Follows https://github.com/ceph/ceph/pull/53875

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

Merge pull request #53884 from leonid-s-usov/rgw-posix-test

test/rgw: don't compile POSIX test unless enabled

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

Merge PR #53873 into main

* refs/pull/53873/head:
qa: typo fix when checking for perf counter - s/md_thresh_evicted/mdthresh_evicted
qa: lower mds_session_metadata_threshold for tests

Reviewed-by: Milind Changire <mchangir@redhat.com>
Reviewed-by: Rishabh Dave <ridave@redhat.com>
Reviewed-by: Leonid Usov <leonid.usov@ibm.com>

src/test/pybind: don't use decorator "with_setup"

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

qa: typo fix when checking for perf counter - s/md_thresh_evicted/mdthresh_evicted

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

qa: lower mds_session_metadata_threshold for tests

... and increase the number of files that are created so as to
hit the threshold with a high probability.

Fixes: http://tracker.ceph.com/issues/62873
Signed-off-by: Venky Shankar <vshankar@redhat.com>

Merge pull request #53817 from cloudbehl/active-alert-filter

mgr/dashboard: Filter active alerts

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

test/rgw: don't compile POSIX test unless enabled

Signed-off-by: Leonid Usov <leonid.usov@ibm.com>

librbd/ManagedLock: kickstart ExclusiveLock state machine

... that is stalled waiting for lock. Do this when trying to reacquire
lock in the ImageWatcher's rewatch mechanism. This would enable the
ExclusiveLock state machine to propagate the blocklist error to the
caller trying to perform an image operation requiring an exclusive
lock.

Previous attempt, e66db763, to fix the hang due to exclusive lock
acquisiton (stuck waiting for lock) racing with client blocklisting
did not always work. e66db763 kickstarted the ExclusiveLock state
machine when the ImageWatcher tried to schedule a exclusive lock
request and the blocklisting was detected. However, there is a short
window between a watch getting deregistered and client blocklisting
getting detected as part of rewatching. If hit when trying to schedule
a lock request, the ExclusiveLock state machine wasn't kickstarted,
blocklist error wasn't propagated, and the hang resurfaced.

A more robust approach is taken to resume the ExclusiveLock state
machine stuck waiting for lock during client blocklisting. Whenever
a client's ImageWatcher loses connection to the cluster, as it happens
during blocklising, the ImageWatcher initiates a mechanism to rewatch
the image and tries to reacquire the lock. Piggyback on this rewatch
mechanism that gets triggered during client blocklisting. And when
trying to reacquire the lock, kickstart the ExclusiveLock state
machine stalled waiting for lock (STATE_WAITING_FOR_LOCK).

Fixes: https://tracker.ceph.com/issues/63009
Signed-off-by: Ramana Raja <rraja@redhat.com>

crimson/os/seastore/onode_manager: drop write_dirty

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

crimson/os/seastore/onode_manager: populate delta recorders for each
onode modification

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

Merge pull request #53874 from zdover23/wip-doc-2023-10-07-rados-troubleshooting-community

doc/rados: edit troubleshooting/community.rst

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

Merge pull request #53875 from zdover23/wip-doc-2023-10-07-troubleshooting-troubleshooting-mon-3-of-x

doc/rados: edit troubleshooting-mon.rst (3 of x)

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

doc/rados: edit troubleshooting/community.rst

Edit doc/rados/troubleshooting/community.rst.

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

doc/rados: edit troubleshooting-mon.rst (3 of x)

Edit doc/rados/troubleshooting/troubleshooting-mon.rst.

Follows https://github.com/ceph/ceph/pull/52827

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

Merge pull request #53876 from zdover23/wip-doc-2023-10-08-architecture-rbd-sentence-repair

doc/architecture: repair RBD sentence

doc/architecture: repair RBD sentence

Improve an ambiguous sentence in doc/architecture.rst.

The problem presented by the original sentence is that the phrasal verb
"to provide with" is implicated in one of its possible readings.
Interpreted in that way, the sentence seems to express the incorrect
idea that RBD furnishes block devices with snapshotting and cloning, as
though snapshotting and cloning are being delivered to the block
devices. In fact, snapshotting and cloning are just features of RBD, and
are features that are described on this page:
https://docs.ceph.com/en/quincy/rbd/rbd-snapshot/.

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

Merge pull request #53790 from zdover23/wip-doc-2023-10-03-architecture-17-of-x

doc/architecture: edit "Peering and Sets"

doc/architecture: edit "Peering and Sets"

Edit the English in the section "Peering and Sets" in the file
doc/architecture.rst.

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

Merge PR #53855 into main

* refs/pull/53855/head:
script: add option for debug build

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

script: add option for debug build

See: https://github.com/ceph/ceph-build/pull/2167

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

mgr/cephadm: fix upgrades with nvmeof

Currently, nvmeof was being treated as if it used
a ceph image during upgrades. This would cause logging
of messages like (I've removed the nvmeof daemon id)

log [WRN] :     Upgrade daemon: nvmeof.<id>: Cannot redeploy
nvmeof.<id> with a new image: Supported types are: mgr, mon,
crash, osd, mds, rgw, rbd-mirror, cephfs-mirror, ceph-exporter,
iscsi, nfs

and if you had set a custom image for the
mgr/cephadm/container_image_nvmeof setting, this would
be undone as part of the upgrade process.

Fixes: https://tracker.ceph.com/issues/63127
Signed-off-by: Adam King <adking@redhat.com>

Merge pull request #53417 from jrchyang/fix_mclock_scheduling_slow_main

osd: fix: slow scheduling when item_cost is large

Reviewed-by: Sridhar Seshasayee <sseshasa@redhat.com>

Merge pull request #52430 from vedanshbhartia/coverity_datarace

rgw: Add coverity annotations for missing mutex locks

reviewed-by: yuvalif, mkogan1

rgw/lua/doc: support reloading lua packages on all RGWs

without requiring a restart of the RGWs
test instructions:
https://gist.github.com/yuvalif/95b8ed9ea73ab4591c59644a050e01e2
also use capitalized "Lua" in logs/doc

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

mon/AuthMonitor: check if entity is absent before creating it

Although this code path is not used for creating entities yet, it is
better to fix the bug sooner than later. Method
AuthMonitor::_update_or_create_entity() must exit (with appropriate
error code) when entity to be created on the Ceph cluster is already
present.

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

mds/MDSAuthCaps: re-word an error message for better clarity

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

Merge pull request #53834 from dparmar18/remove-egg-fragment-from-doc

doc: remove egg fragment from dev/developer_guide/running-tests-locally

Reviewed-by: John Mulligan <jmulligan@redhat.com>
Reviewed-by: Rishabh Dave <ridave@redhat.com>
Reviewed-by: Anthony D'Atri <anthony.datri@gmail.com>
Reviewed-by: Zac Dover <zac.dover@proton.me>

rgw: fix http error checks in keystone/barbican/vault clients

when RGWHTTPManager encounters an http error, it uses
rgw_http_error_to_errno() to map that to a negative posix error code.
RGWHTTPClient::process() returns that mapped error code, and exposes the
original http error via get_http_status()

the http client code for keystone, barbican, and vault were returning
early on the errors from process(), so weren't getting to the http error
checks

these clients now check for specific http errors before testing the
result of process()

Fixes: https://tracker.ceph.com/issues/62989
Signed-off-by: Casey Bodley <cbodley@redhat.com>

Merge pull request #52296 from asm0deuz/bz2153448_ipv6

mgr/cephadm: ceph orch add fails when ipv6 address is surrounded by square brackets.

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

Merge pull request #53621 from phlogistonjohn/jjm-cephadm-dtypes-common

cephadm: introduce Daemon Forms

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

Merge PR #53836 into main

* refs/pull/53836/head:
PendingReleaseNotes: add a note about disallowing delegating inodes
qa: start testing mds_client_delegate_inos_pct config
mds: disable delegating inode ranges to clients

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

doc: remove egg fragment from dev/developer_guide/running-tests-locally

DEPRECATION: git+https://github.com/ceph/teuthology#egg=teuthology
[test] contains an egg fragment with a non-PEP 508 name pip 25.0 will enforce
this behaviour change. A possible replacement is to use the req @ url syntax,
and remove the egg fragment. Discussion can be found at
https://github.com/pypa/pip/issues/11617

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

src/MDSMonitor: make use of imported namespace symbols

Symbols imported into current namespace should be used directly; there
is no need to mention their parent namespace while using them. IOW, to use
"std::string" after it has been imported, just write "string" instead of
"std::string".

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

mon/MDSMonitor: remove unused namespace imports

Remove symbols imported into the current namespace since they are not being
used.

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

mon/FSCommands: import std::ostream into current namespace

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

rgw/lua: install lua packages in temp directory

and switch to it only once installation is done.
this is needed for cases where installation can happen
while RGW is running

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

rgw/lua/doc: fix luajson example

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

mon/FSCommands: make use of imported namespace symbols

There's no need to mention the "home" namespace of a symbol while using
it after it has been imported into the current namespace. IOW, no need to
write, for example, "std::string" after it has been imported from its
namespace; instead simply writing "string" will suffice.

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

mon/FSCommands: remove unused namespace imports

When symbols imported into the current namespace won't be used, don't
import them.

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

mon/AuthMonitor: make use of imported namespace symbols

Once a symbol has been imported into the current namespace, no need to
mention the original namespace while using it. IOW, no need to write
"std::string" after it has been imported from the namespace "std" into the
current namespace.

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

mon/AuthMonitor: remove unused namespace imports

Don't import items into current namespace when they'll be never used.

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

Merge pull request #53700 from cloudbehl/cephfs-snapshot

mgr/dashboard: Fixed cephfs snapshot listing in directories

Reviewed-by: Avan Thakkar <athakkar@redhat.com>
Reviewed-by: Nizamudeen A <nia@redhat.com>

Merge pull request #53782 from sachinpunadikar/vstart-fix

vstart: exclude default route during cluster setup

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

mgr/dashboard: Filter active alerts

- List alerts which are not suppressed

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

Signed-off-by: cloudbehl <cloudbehl@gmail.com>

Merge pull request #53819 from zdover23/wip-doc-2023-10-04-rados-troubleshooting-index

doc/rados: edit troubleshooting.rst

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

PendingReleaseNotes: add a note about disallowing delegating inodes

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

qa: start testing mds_client_delegate_inos_pct config

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

mds: disable delegating inode ranges to clients

Fixes: http://tracker.ceph.com/issues/63103
Signed-off-by: Venky Shankar <vshankar@redhat.com>

Merge pull request #52735 from vedanshbhartia/coverity_minlong

rgw: Add coverity annotation for warning about tautological comparison

reviewed-by: thotz, yuvalif