radosgw-admin: fix errors found by pyflakes

Using the wrong variable, and true/false are capitalized in python.

Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
Fix unused vars, unused imports, and aliasing

Found by pyflakes

Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
ceph: fix log, /var/run/ceph stupid

Signed-off-by: Sage Weil <sage@inktank.com>
install: ignore apt-get update failures on package removal

At that point we don't really care.

Signed-off-by: Sage Weil <sage@inktank.com>
ceph.conf: log file, not log dir

Signed-off-by: Sage Weil <sage@inktank.com>
task: mon_thrash: Add 'seed' and 'store-thrash' options

This patch introduces an option to thrash a monitor store when we thrash
the monitors, as well as a 'store-thrash-probability' option (defaulting
to 50%).

We also took this opportunity to introduce a new 'seed' option, that ought
to allow a given run of this task to be reproducible.  This might come in
hand when attempting to reproduce a given behavior that would otherwise
be randomly triggered.

You should note that while the 'seed' option will indeed mimic past
behaviors, this only applies to a past behavior of this task: other tasks
are not affected by this value, nor are any workunits or even ceph daemons.

Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
task: mon_thrash: Thrash multiple monitors and 'maintain-quorum' option

We now add a new option 'thrash-many' that by being set to true will break
the default behaviour of killing only one monitor at a time.  Instead,
this option will select up to the maximum number of killable monitors to
kill in each round.

We also add a new 'maintain-quorum' option that will limit the amount of
monitors that can be killed in each thrashing round.  If set to true, this
option will limit the amount of killable monitors up to (n/2-1).  This
means that if we are running a configuration that only has up to two
configured monitors, if 'maintain-quorum' is set to true, this task won't
run as there are no killable monitors -- in such a scenario, this option
should be set to false.

Furthermore, if 'store-thrash' is set to true, then 'maintain-quorum' must
also be set to true, as we cannot let the task to thrash all the monitor
stores, or we wouldn't be able to sync from other monitors, nor can we
let quorum be dropped, or we won't be able to resync our way into quorum.

Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
task: ceph: create monitor data directories prior to --mkfs

The new monitor store does not create the data directory on --mkfs. We
must create it instead, much like what happens with the osds.

Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
nuke: sudo for killall

Signed-off-by: Sage Weil <sage@inktank.com>
rbd: set TESTDIR when running xfstests

[elder@inktank.com: fixed typo]

Signed-off-by: Sage Weil <sage@inktank.com>
ceph.conf: debug ms = 1 for monitors

Signed-off-by: Sage Weil <sage@inktank.com>
exec, pexec: set TESTDIR

Signed-off-by: Sage Weil <sage@inktank.com>
rbd: drop udev

The packages have these rules now.

Signed-off-by: Sage Weil <sage@inktank.com>
Add timer.py and display summary info in run.py.

Signed-off-by: Warren Usui <warren.usui@inktank.com>
nuke: blow away /home/ubuntu/cephtest too

(along with /tmp/cephtest)

Signed-off-by: Sage Weil <sage@inktank.com>
radosgw-admin: Bug fixes for issue 4251.

Fixes errors introduced by d44fb14762c3d7e2853a8e4be012cd8e4557124e.

Signed-off-by caleb.miles <caleb.miles@inktank.com>
Add timer.py and display summary info in run.py.

Signed-off-by: Warren Usui <warren.usui@inktank.com>
nuke: blow away /home/ubuntu/cephtest too

(along with /tmp/cephtest)

Signed-off-by: Sage Weil <sage@inktank.com>
teuthology: add an extra_packages flag to install

Some tests require additional packages
(e.g., java bindings, hadoop bindings).
Extend the install task to allow for those
packages to be specified in the yaml files.

Signed-off-by: Joe Buck <jbbuck@gmail.com>
Reviewed-by: Sam Lang <sam.lang@inktank.com>
teuthology: remove CEPH_JAVA_PATH

Since teuthology now uses debian packages,
we do not need to set this in workunit.
The one test that uses this now tests for
it locally.

Signed-off-by: Joe Buck <jbbuck@gmail.com>