| Commit message (Collapse) | Author | Files | Lines |
|
There seems to be change in 'nvme list' json output
which caused failures in asserts after 'nvme connect-all'
command.
Signed-off-by: Vallari Agrawal <vallari.agrawal@ibm.com>
|
|
Suggested by Casey.
Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
|
|
Suggested by Casey. This is a relatively small value, but let's
be conservative.
Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
|
|
Signed-off-by: Casey Bodley <cbodley@redhat.com>
|
|
Signed-off-by: Casey Bodley <cbodley@redhat.com>
|
|
the part head objects don't have a RGW_ATTR_MANIFEST attribute, so
get_obj_state_impl() isn't able to set the correct
RGWObjState::accounted_size unless RGW_ATTR_COMPRESSION provides one
get_part_obj_state() builds a fake manifest in memory to represent the
part and updates state.size accordingly, but it hadn't corrected the
value of state.accounted_size
Signed-off-by: Casey Bodley <cbodley@redhat.com>
|
|
RGWGetObj::execute() relies on the virtual get_decrypt_filter() which is
overridden by RGWGetObj_ObjStore_S3
but when called from RGWGetObjAttrs_ObjStore_S3, there is no override.
add one that just calls rgw_s3_prepare_decrypt()
Signed-off-by: Casey Bodley <cbodley@redhat.com>
|
|
The version_id member in RGWGetObjAttrs shadows the one in
RGWGetObj.
Just assign it before conditionally returning the version-id header
in RGWGetObjAttrs_ObjStore_S3::send_response().
Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
|
|
upload-part
There appear to be workloads that provide a checksum algorithm in
create-multipart-upload, but do not provide (what must be) the
corresponding algorithm when uploading the parts. (complete-multipart-upload
has no checksum argument, the value is implicit.)
This behavior is inconsistent with at least some SDKs, but it is
possibly accepted behavior in AWS S3, and is not logically necessary,
since the originally supplied checksum type is already known.
Therefore, change the behavior of upload-part to fall back to a
checksum type that was sent with the corresponding create-multipart-upload
request, if one is present, rather than failing with InvalidRequest.
Fixes: https://tracker.ceph.com/issues/68513
Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
(cherry picked from commit 6b487a4c6dbadf3f470c8b12ddd5f2521c6920c6)
|
|
If a create-multipart-upload requested additional checksum but a
part upload omits it, fail gracefully.
Possibly we can fixup the checksum uploads and remove this scenario,
but coding defensively would still be correct.
Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
|
|
Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
|
|
Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
|
|
(code review)
Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
|
|
Implements the corresponding S3 operation, and
introduces a new Object::list_parts SAL interface to support it.
Includes Casey Bodley <cbodley@redhat.com>:
* use uncompressed part size
* local variable shadowed a member variable and broke handling of
PartNumberMarker in request and response
Fixes: https://tracker.ceph.com/issues/64109
Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
|
|
This commit updates the `is_device` function to correctly handle
loop devices.
The function now validates loop devices when they are explicitly
allowed, by checking their type (`loop`) in addition to `disk`
and `mpath`.
Changes include:
- Extending the type check to include `loop` in the list of
supported device types.
- Enhancing the docstring for better documentation of the
function's purpose and behavior.
These changes ensure that loop devices are properly recognized
and handled when configuring OSDs in ceph-volume.
Fixes: https://tracker.ceph.com/issues/69432
Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
|
|
random values are generated thereafter
Signed-off-by: Jon Bailey <jonathan.bailey1@ibm.com>
|
|
running
Signed-off-by: Jon Bailey <jonathan.bailey1@ibm.com>
|
|
ceph_json.h fully.
Also moved and renamed JSONStructures files so they structures are more easily identifiable and usable by others if desired.
Signed-off-by: Jon Bailey <jonathan.bailey1@ibm.com>
|
|
data_generation::SeededRandomGenerator
Signed-off-by: Jon Bailey <jonathan.bailey1@ibm.com>
|
|
Signed-off-by: Jon Bailey <jonathan.bailey1@ibm.com>
|
|
Signed-off-by: Jon Bailey <jonathan.bailey1@ibm.com>
|
|
Add sequences to test IOs with simple error injects, along with some small fixes for previous error inject implementation.
Signed-off-by: Jon Bailey <jonathan.bailey1@ibm.com>
|
|
interactive mode
Add injecterror commands that can be used in interactive mode to inject read and write errors as well as clear them
Signed-off-by: Jon Bailey <jonathan.bailey1@ibm.com>
|
|
Error inject interfaces for EC reads and writes using
ceph tell osd.<n> interface
Signed-off-by: Bill Scales <bill_scales@uk.ibm.com>
|
|
Implement inheritence for ceph::io_exerciser::IoOp to allow better differentiation between the different Op types and allow more complex Operations to be implemented
Signed-off-by: Jon Bailey <jonathan.bailey1@ibm.com>
|
|
This reverts commit 41327dcf0f733777f0d022d1bab7bf72b9807b3c.
Signed-off-by: Matan Breizman <mbreizma@redhat.com>
|
|
As Seastore/trash was removed from the non-exp suite, it is
moved here until fully supported.
Follow-up to: 5150dae471c
Signed-off-by: Matan Breizman <mbreizma@redhat.com>
|
|
"basic" directory is included in the non-exp suite and is fully supported
Signed-off-by: Matan Breizman <mbreizma@redhat.com>
|
|
Seastore supports thrash_simple only until https://tracker.ceph.com/issues/69405
is resolved.
Signed-off-by: Matan Breizman <mbreizma@redhat.com>
|
|
This partially reverts f4dee79c3309e5ac1e1a142b85f492851e6757e1
Until https://tracker.ceph.com/issues/69402 is resolved.
Signed-off-by: Matan Breizman <mbreizma@redhat.com>
|
|
This commit introduces type annotations to the `create_id` function in `ceph_volume.util.prepare`.
The parameters and return value are now typed as follows:
- `fsid` is a `str`.
- `json_secrets` is a `str`.
- `osd_id` is an optional `str` (`Optional[str]`).
- The function returns a `str`.
Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
|
|
Instead of immediately triggering, have this healthcheck trigger
after some time has elasped. This delay can be configured by
mon_nvmeofgw_delete_grace.
Track the time when gateways go into DELETING state in a new
member var (of NVMeofGwMon) 'gws_deleting_time'.
Signed-off-by: Vallari Agrawal <vallari.agrawal@ibm.com>
|
|
tab in yaml file
Fixes: https://tracker.ceph.com/issues/69192
Signed-off-by: Shweta Bhosale <Shweta.Bhosale1@ibm.com>
|
|
Some C++ fixes, macro fixes, etc.. Gathers other core
files, makes names more closely match executable.
Signed-off-by: Jesse F. Williamson <jfw@ibm.com>
|
|
CEPH_VERSION used to be present in the env to hold the ref. Other projects
relied on this. Make life easier for them.
Signed-off-by: Dan Mick <dan.mick@redhat.com>
|
|
https://github.com/ceph/ceph/pull/56583 recently fixed
merge_and_store_attrs() to preserve existing attrs, but this broke the
swift api's ability to remove container metadata. RGWCreateBucket
handles this merging itself with prepare_add_del_attrs(), so we should
just assign createparams.attrs to the bucket and store it with
bucket->put_info()
make the same change for RGWPutMetadataBucket which swift uses to
add/remove existing metadata
Fixes: https://tracker.ceph.com/issues/69301
Signed-off-by: Casey Bodley <cbodley@redhat.com>
|
|
When an OSD creation fails, ceph-volume can zaps unrelated
existing raw based OSD as part of the 'rollback step'.
Fixes: https://tracker.ceph.com/issues/69430
Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
|
|
Signed-off-by: Anthony D'Atri <anthonyeleven@users.noreply.github.com>
|
|
This config allows to configure the delay in triggering
NVMEOF_GATEWAY_DELETING healthcheck warning, which is
triggered when NVMeoF gateways are in DELETEING state
for too long (indicating a problem in namespace
load-balacing).
The default value for this config is 15 mins.
Signed-off-by: Vallari Agrawal <vallari.agrawal@ibm.com>
|
|
Fixes: https://tracker.ceph.com/issues/68974
Signed-off-by: Neeraj Pratap Singh <neesingh@redhat.com>
|
|
Fixes: https://tracker.ceph.com/issues/68346
Signed-off-by: Mark Kogan <mkogan@redhat.com>
|
|
polluting the cache of the underlying futurized store
Signed-off-by: Xuehan Xu <xuxuehan@qianxin.com>
|
|
To be used by exclusively by Seastore, See `seastore::cache_hint_t`
Signed-off-by: Xuehan Xu <xuxuehan@qianxin.com>
|
|
Layers above Cache can use cache_hint_t to notify Cache whether to put
the extents to Cache::lru.
CEPH_OSD_OP_FLAG_FADVISE_DONTNEED and CEPH_OSD_OP_FLAG_FADVISE_NOCACHE
will be treated as "no need to put into Cache::lru"
Signed-off-by: Xuehan Xu <xuxuehan@qianxin.com>
|
|
ops to replica osds and the pg log append happens in the same
continuation
Since backfill relies on the pg log to discover new modifications, we
need to make sure backfill always discover the modification that's not
sent to replica osds.
Signed-off-by: Xuehan Xu <xuxuehan@qianxin.com>
|
|
objects
Signed-off-by: Xuehan Xu <xuxuehan@qianxin.com>
|
|
if the object is deleted while it's being backfilled
Fixes: https://tracker.ceph.com/issues/69027
Signed-off-by: Xuehan Xu <xuxuehan@qianxin.com>
|
|
recoveries
Signed-off-by: Xuehan Xu <xuxuehan@qianxin.com>
|
|
the end of Enqueuing::Enqueuing
Fixes: https://tracker.ceph.com/issues/69120
Signed-off-by: Xuehan Xu <xuxuehan@qianxin.com>
|
|
When cephadm writes client files to hosts, the new files are created in
/tmp/cephadm-<fsid> and then moved to their target directory. This makes
the client files receive the usr_tmp_t context, which is different then
their original context.
mv -Z will set the SELinux security context type of the target file
based on the SELinux policy rules for the target directory.
Fixes: https://tracker.ceph.com/issues/69395
Signed-off-by: zaken <yonatan.zaken.ext@nokia.com>
|