osd_scrub_pgs now:
1.) Insures that all pgs are clean and active.
2.) Performs the scrub of all OSDs.
3.) Waits until all pgs have been scrubbed or no progress in scrubbing has
been made for two minutes.
These changes have been made on top of a previous check-in that was incomplete
Fixes: 7198
Signed-off-bye: Warren Usui <warren.usui@inktank.com>
With this I can run a simple job with a normal build (not
notcmalloc) and valgrind does not complain. Fingers crossed!
Signed-off-by: Sage Weil <sage@inktank.com>
When testing new tasks, it's a drag to have to keep manually unlocking
the targets. Default behavior is to keep them locked for any failure;
this allows overriding that choice for the case where the failure is
in debugging the task, where the state of the targets isn't important.
Signed-off-by: Dan Mick <dan.mick@inktank.com>
This involves moving everything in build_ceph_cluster() inside the try:
block, so if an exception fails the cleanup in the finally: block will
actually be executed.
New versions of ssh-keyscan return two keytypes if possible; this
breaks the comparison of "number of lines of output from keyscan"
to "number of hosts we request keys from". Fix by asking for only
one type of key (as older ssh-keyscans did).
Fixes: #7164
Signed-off-by: Dan Mick <dan.mick@inktank.com>
This is the main source of noise when running valgrind +
tcmalloc. Apparently there are other issues, so I think we
still need the notcmalloc gitbuilder, but this gets us part of
the way.
Signed-off-by: Sage Weil <sage@inktank.com>
See #7171. In rare cases CRUSH can't handle it when only 2/6 of
the OSDs are marked in. Avoid those situations for now.
Signed-off-by: Sage Weil <sage@inktank.com>
Currently, to see if a node has rebooted into the right kernel,
need_to_install() compares a given 40-char commit hash with a 7-char
commit hash abbreviation it pulls from the output of 'uname -r'.
gitbuilders can now export UTS_RELEASE kernel version string through
.../$SHA1/version file. Use this string instead of the 40-char commit
hash and compare it with the output of 'uname -r' directly. This saves
us the parsing exercise and, more importantly, makes it possible to
install clean tagged kernels using 'tag:' element, which wasn't
possible before because version string of such kernels doesn't have
a commit hash in it.
If version file is unavailable, fallback to the existing way of doing
things.
Signed-off-by: Ilya Dryomov <ilya.dryomov@inktank.com>
in connect(). The bug caused vm behavior to happen for a
target if any of the machines in the cluster was a vm. The code
was also changed to set the key to none only if rsa or dsa keys
were used on a vm.
Fixes: 7113
Signed-off-by: Warren Usui <warren.usui@inktank.com>
