Prior to this commit, if a user installed the "ceph-common" Debian
package without installing "ceph", then /usr/bin/ceph would crash
because it was missing the ceph_argparse library.
Ship the ceph_argparse library in "ceph-common" instead of "ceph". (This
was the intention of the original commit that moved argparse to "ceph",
2a23eac549)
http://tracker.ceph.com/issues/11388 Refs: #11388
Reported-by: Jens Rosenboom <j.rosenboom@x-ion.de>
Signed-off-by: Ken Dreyer <kdreyer@redhat.com>
- to avoid the scrub wave when the osd_scrub_max_interval reaches in a
high-load OSD, the scrub time is randomized.
- extract scrub_load_below_threshold() out of scrub_should_schedule()
- schedule an automatic scrub job at a time which is uniformly distributed
over [now+osd_scrub_min_interval,
now+osd_scrub_min_interval*(1+osd_scrub_time_limit]. before
this change this sort of scrubs will be performed once the hard interval
is end or system load is below the threshold, but with this change, the
jobs will be performed as long as the load is low or the interval of
the scheduled scrubs is longer than conf.osd_scrub_max_interval. all
automatic jobs should be performed in the configured time period, otherwise
they are postponed.
- the requested scrub job will be scheduled right away, before this change
it is queued with the timestamp of `now` and postponed after
osd_scrub_min_interval.
Fixes: #10973
Signed-off-by: Kefu Chai <kchai@redhat.com>
This tests RENAME_WHITEOUT, which was enabled for xfs in kernel commit
7dcf5c3e4527cfa2807567b00387cf2ed5e07f00. At first execution, it throws a BUG.
Subsequent executions appear to work correctly. This issue manifests for disks
and RBD instances.
Signed-off-by: Douglas Fuller <dfuller@redhat.com>
This was broken by 96992466 aka "mds: handle missing mydir dirfrag"
The previous code was mistakenly treating a not-yet-loaded
dirfrag as a non-existent dirfrag, resulting in
inconsistent fragstats even when no objects had
actually been lost.
Fixes: #11641
Signed-off-by: John Spray <john.spray@redhat.com>
Move check-local scripts
src/test/run-cli-tests
encode-decode-non-regression.sh
test/encoding/readable.sh
to check_SCRIPTS. Their output is captured in .log file when running
with a recent automake. This reduces the output of make check by an
order of magnitude.
Signed-off-by: Loic Dachary <ldachary@redhat.com>
Because of the | grep, the status of tox is no longer the status of
run-tox.sh and errors are not reported as they should.
Signed-off-by: Loic Dachary <ldachary@redhat.com>
Remove the helpers because they are not used any longer. They have been
deprecated by ceph-helpers.sh
Signed-off-by: Loic Dachary <ldachary@redhat.com>
Use ceph-helpers.sh instead of mon/mon-test-helpers.sh.
* modifying the .asok and .log names to match the ceph-helpers.sh
conventions
* use explicit ports 7300 and 7301 instead of +1 so that grep
will show that 7301 is used. This reduces the odds of a
port collision when looking for a port that's not already
used by an existing test.
Signed-off-by: Loic Dachary <ldachary@redhat.com>
Use ceph-helpers.sh instead of mon/mon-test-helpers.sh.
* The expect_false() helper is replaced by a ! statement which is
simpler and has the same effect.
* modifying the .asok and .log names to match the ceph-helpers.sh
conventions
* trim the --public-addr argument from the run_mon call
* add || return 1 after run_mon where missing
* add || return 1 to all commands in TEST_no_pool_delete
Signed-off-by: Loic Dachary <ldachary@redhat.com>
The ceph-helpers.sh was already included, along with
mon/mon-test-helpers.sh and osd/osd-test-helpers.sh which creates
confusion.
* modifying the .asok and .log names to match the ceph-helpers.sh
conventions
* trim the --public-addr argument from the run_mon call
Signed-off-by: Loic Dachary <ldachary@redhat.com>
The ceph-helpers.sh was already included, along with
mon/mon-test-helpers.sh and osd/osd-test-helpers.sh which creates
confusion.
Signed-off-by: Loic Dachary <ldachary@redhat.com>
Fix grep that assume the pool has id 0, because it may be different
when rbd is redefined to have less PG by the run_mon function.
Signed-off-by: Loic Dachary <ldachary@redhat.com>
The ceph-helpers.sh was already included, along with
mon/mon-test-helpers.sh and osd/osd-test-helpers.sh which creates
confusion.
* replace check_osd_status with wait_for_osd
* modifying the .asok and .log names to match the ceph-helpers.sh
conventions
Signed-off-by: Loic Dachary <ldachary@redhat.com>
It is just a conversion and the behavior is not changed. It involves:
* calling run_mon from inside each test instead of doing it in run
* modifying the .asok and .log names to match the ceph-helpers.sh
conventions
Signed-off-by: Loic Dachary <ldachary@redhat.com>
Instead of using mon-test-helpers.sh, primarily because the kill_daemon
function implemented in mon-test-helpers.sh is not as good as
ceph-helpers.sh.
Instead of having tests that share the same monitor, each test now runs
on a fresh monitor. The test writer no longer has to worry that it will
be re-using the pool or profile from a previous test. That causes
problems that are difficult to diagnose and the overhead of running a
new monitor is not so high.
Signed-off-by: Loic Dachary <ldachary@redhat.com>
Remove call_TEST_functions because it is too complex and rarely used.
Replace the two instances where it was used by a few lines doing the
same thing.
Signed-off-by: Loic Dachary <ldachary@redhat.com>
Fix hardcoding of id a in the run_mon function. The directory
in which the mon data is stored must be a sub-directory of the
directory given in argument.
If mon_initial_members is set, the rbd pool cannot be redefined, which
is ok because this is rare and it's only an optimization to reduce the
number of PG.
Signed-off-by: Loic Dachary <ldachary@redhat.com>
Instead of just pg dump and health detail, the wait_for_clean
function displays ceph report when it times out, to help figure out what
caused it.
Signed-off-by: Loic Dachary <ldachary@redhat.com>
The wait_for_osd to wait for an osd to go up and down is needed
internally, after running an osd. Move the inline snippet from run_osd
into a function so that it can be used by scripts as well.
Signed-off-by: Loic Dachary <ldachary@redhat.com>
