when aborting uncommitted transaction, abort_allocation
should be called to cancel in-flight allocations in
in-memory allocator.
Signed-off-by: Myoungwon Oh <myoungwon.oh@samsung.com>
alloc_extent() will allocates continous blocks (e.g., block 1 ~ 4).
To deliver such informations to journal efficiently,
this commit adds a member variable regarding only modified range in the deltas
Signed-off-by: Myoungwon Oh <myoungwon.oh@samsung.com>
NormalNBD - NVMe Block device with seastar file
1. Support preffered write granularity/alignment which is reported
from NVMe SSD. User should follow this write guidance for enhancing write
performance.
2. Support multi-stream IO in NVMe SSD.
Signed-off-by: Jinyong Ha <jy200.ha@samsung.com>
one transaction got an extent whose state is MUTATION_PENDINGat that time.
but another transaction do split and set the extent state to RETIRED.
when the first transaction resume and do continuation, the state of the extent
has been changed to RETIRED. So need eagain to try again.
Signed-off-by: chunmei-liu <chunmei.liu@intel.com>
This was using an obscure syntax that worked at one time and wasn't
documented (AFAIK).
Fixes: https://tracker.ceph.com/issues/51182
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
The print statement originally suggested to the user to run "make && ctest -j12", but that is outdated. The user should execute "./run-make-check.sh" instead.
Signed-off-by: Laura Flores <lflores@redhat.com>
Updating the initial pull request. The smoke test line should read "./run-make-check.sh" instead of "make && ctest -j12".
Signed-off-by: Laura Flores <lflores@redhat.com>
The current version of Ceph uses "ninja" rather than "make," so the testing documentation needs to be updated.
Signed-off-by: Laura Flores <lflores@redhat.com>
This PR is solving an error: Directory not empty.
This was encountered during removal of the downloaded keycloak package from keycloak.py
Signed-off-by: Kalpesh Pandya <kapandya@redhat.com>
When connecting to remote cluster via mon address, the cluster
confuration file is not required. However, the mirror daemon
fills up the ceph context with the cluster name resulting in
the following warning in (systemctl) logs:
auth: unable to find a keyring on ...
These warning messages need to be silence as they can mislead
the user.
Fixes: http://tracker.ceph.com/issues/51204
Signed-off-by: Venky Shankar <vshankar@redhat.com>
* refs/pull/41431/head:
mds: progress the recover queue immediately after the inode is enqueued
mds: switch to mdr->get_client()
Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
to ensure the support for the new syntax of "prefix --name bon,jour"
does not break existing behavior of "prefix hello cruel,world" where value2
contains reads "hello,world", and the parsed result should be
prefix="prefix"
value=["hello", "cruel,world"]
instead of something like
prefix="prefix"
value=["hello", "cruel", "world"]
or
prefix="prefix"
value=["cruel", "world"]
the above test only applies to the test where "value" is a CephString.
if "value" is a CephChoices, the parsed argument should be
prefix="prefix"
value=["hello", "cruel", "world"]
Signed-off-by: Kefu Chai <kchai@redhat.com>
instead of comparing the validated argument with empty dict, check for
its content for better coverage of testing.
Signed-off-by: Kefu Chai <kchai@redhat.com>
otherwise nose tries to run it as a test as its name contains "test", like
E.
======================================================================
ERROR: Runs validate in different arg/kargs ways.
----------------------------------------------------------------------
TypeError: arg_kwarg_test() missing 3 required positional arguments: 'prefix', 'args', and 'sig'
Signed-off-by: Kefu Chai <kchai@redhat.com>
"scrub" command was marked obsoleted in
e9a5ce0897, so the test_ceph_argparse
cannot retrieve its command description using "get_command_descriptions"
cli tool anymore, let's drop the related test accordingly.
Signed-off-by: Kefu Chai <kchai@redhat.com>
if desc.N is not None, we should take the argument as a comma separated
values, and validate the values individually.
restructure the validate() function and its helpers to pass the
validated args if desc.N explicitly, as desc.instance.val should only
hold a single value of desc.instance type, otherwise we need to reset
it after collecting all the argument in a CSV string value is parsed.
this change would be less consistent and hackish.
and update a test to verify this behavior.
Fixes: https://tracker.ceph.com/issues/51145
Signed-off-by: Kefu Chai <kchai@redhat.com>
