these are leftovers from radosgw-agent. we kept them around for the
conversion logic, but that was only necessary to support upgrades to
jewel and kraken
Signed-off-by: Casey Bodley <cbodley@redhat.com>
the encoding of `cls_rgw_bucket_instance_entry` changed in
bd0b0e5d98 such that pacific/quincy
OSDs cannot read the version written by reef OSDs
two fields (bucket_instance_id and num_shards) were removed in the v2
encoder, but the v1 decoder still expects them to be present
since we haven't released reef yet, we can fix this with a v3 that adds
the fields back
Fixes: https://tracker.ceph.com/issues/57232
Signed-off-by: Casey Bodley <cbodley@redhat.com>
rgw_allow_notification_secrets_in_cleartext conf parameter was added for that
Fixes: https://tracker.ceph.com/issues/50611
Signed-off-by: Yuval Lifshitz <ylifshit@redhat.com>
prefer the facilities provided by standard library over the homebrew
ones for better readability and maintainability.
Signed-off-by: Kefu Chai <tchaikov@gmail.com>
prefer the facilities provided by standard library over the homebrew
ones for better readability and maintainability.
Signed-off-by: Kefu Chai <tchaikov@gmail.com>
prefer the facilities provided by standard library over the homebrew
ones for better readability and maintainability.
Signed-off-by: Kefu Chai <tchaikov@gmail.com>
This was introduce by commit e2a67f2a65,
for the third part applications they may still use the old macro.
Add it back and marked it as deprecated.
Fixes: https://tracker.ceph.com/issues/56638
Signed-off-by: Xiubo Li <xiubli@redhat.com>
test_client_metrics_and_metadataand other tests has been
updated as earlier it was checking according to the old structure
of perf stats o/p, which has been changed in this PR.
Fixes: https://tracker.ceph.com/issues/56162
Signed-off-by: Neeraj Pratap Singh <neesingh@redhat.com>
* add optional "headers" parameter to _request() and _get()
* correct the test of test_force_no_gzip, as the header in response
should be "application/vnd.ceph.api.v1.0+json", not "application/json"
with this change, we are able to pass headers to _get() in dashboard api
tests.
without this change, we'd have following error when testing
`test_force_no_gzip` defined by `tasks.mgr.dashboard.test_requests.RequestsTest`:
raceback (most recent call last):
File "/home/jenkins-build/build/workspace/ceph-api/src/pybind/mgr/dashboard/services/exception.py", line 47, in dashboard_exception_handler
return handler(*args, **kwargs)
File "/usr/lib/python3/dist-packages/cherrypy/_cpdispatch.py", line 54, in __call__
return self.callable(*self.args, **self.kwargs)
File "/home/jenkins-build/build/workspace/ceph-api/src/pybind/mgr/dashboard/controllers/_base_controller.py", line 263, in inner
ret = func(*args, **kwargs)
TypeError: Summary.__call__() got an unexpected keyword argument 'headers'
Signed-off-by: Kefu Chai <tchaikov@gmail.com>
osd: fix ScrubMap::swap() to include all ScrubMap data members
Reviewed-by: Samuel Just <sjust@redhat.com>
Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
to just check for the return type of `it.get_pos_add(n)`, this
is exactly what we expect from a buffer::ptr::const_iterator.
it is more specific and less confusing.
Signed-off-by: Kefu Chai <tchaikov@gmail.com>
