Commit Graph

2387 Commits

Author SHA1 Message Date
Zack Cerza
cadcbf3914 Remove unused import
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-06-04 16:29:11 -05:00
Zack Cerza
549b52b3b3 Update --help output
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-06-04 12:42:43 -05:00
Josh Durgin
3ae54ead8a Merge pull request #267 from ceph/wip-runxfstests
wip-runxfstests

Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
2014-06-03 15:56:44 -07:00
Zack Cerza
0f04af0799 Add a timeout for unmounting
Ten minutes seems like it might be longer than necessary, but I doubt
it's too short.

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-06-03 12:39:38 -05:00
Alfredo Deza
60e6788ddf Merge pull request #265 from ceph/wip-orchestra
Refactor teuthology.orchestra.run
2014-06-03 09:12:21 -04:00
Zack Cerza
af73e3080c Remove some duplicate code.
Well, not exactly duplicate, but functionally compatible. No sense in
maintaining two "walk the filesystem to assemble job information"
utilities.

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-06-02 19:01:09 -05:00
Zack Cerza
ae0d26ae71 Try a different approach to zombie prevention
The previous method I had committed had the unfortunate side-effect of
preventing teuthology job processes from giving their returncode to the
worker process. This method shouldn't do that.

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-06-02 09:23:27 -05:00
Yehuda Sadeh
3968f6f744 radosgw-admin: adjust bucket link interface
Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
2014-06-02 09:23:27 -05:00
Ilya Dryomov
86c60f7de0 rbd: change test_script to run_xfstests_krbd.sh
Invoke run_xfstests.sh through a wrapper, which supplies an exclusion
list, in order to skip tests we don't want to run.

Signed-off-by: Ilya Dryomov <ilya.dryomov@inktank.com>
2014-05-31 15:16:52 +04:00
Ilya Dryomov
2551c528a2 rbd: recognize 'randomize' parameter
randomize (default: false) randomizes xfstests test order.

Signed-off-by: Ilya Dryomov <ilya.dryomov@inktank.com>
2014-05-31 15:11:38 +04:00
Sage Weil
241a102568 Merge pull request #266 from ceph/wip-8489
Give daemons a five-minute timeout for stopping

Reviewed-by: Sage Weil <sage@inktank.com>
2014-05-30 16:28:40 -07:00
Zack Cerza
205c455fc1 Give daemons a five-minute timeout for stopping
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-05-30 16:39:16 -05:00
Zack Cerza
7035c17ec9 Add RemoteProcess.__repr__()
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-05-30 16:36:03 -05:00
Zack Cerza
eac2c2abbb Update users of the teuthology.orchestra.run APIs
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-05-30 16:05:23 -05:00
Zack Cerza
b386f5e5df Refactor teuthology.orchestra.run
RemoteProcess behaves more like subprocess.Popen, with some important
differences.

A summary of the API changes:
* RemoteProcess.exitstatus is either an int or None; it is never a callable
  nor a gevent.AsyncResult.
* New method: RemoteProcess.execute()
* New method: RemoteProcess.poll()
* New method: RemoteProcess.wait()
* New attribute: RemoteProcess.returncode - alias to exitstatus
* New property: RemoteProcess.finished - added because returncode can be None
  if the connection was interrupted
* run.execute() is removed.

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-05-30 16:05:23 -05:00
Zack Cerza
59ee17dc19 Avoid piles of zombies
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-05-30 09:59:36 -05:00
Zack Cerza
0fcfaee726 Add comment explaining 'preexec_fn=os.setpgrp'
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-05-30 09:56:27 -05:00
Zack Cerza
b26ba60565 Check for broken repos before updating
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-05-29 13:23:23 -05:00
Zack Cerza
abc722c8be More verbose logging for daemon restarts
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-05-29 00:37:27 -05:00
Zack Cerza
2363a17caa Bump log level to info
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-05-28 23:40:48 -05:00
Zack Cerza
c4783543e3 Don't block on teuthology-results processes
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-05-28 14:56:59 -05:00
Zack Cerza
a4a9a1f22f Correct logging when running teuthology-results
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-05-28 11:43:20 -05:00
Zack Cerza
baeeddbe76 Merge pull request #263 from qakapil/patch-two
updated the get_system_type function
2014-05-28 09:34:47 -05:00
ksharma
0b97eba364 updated the get_system_type function
updated the get_system_type function
to add the rpm condition for SUSE.

Signed-off-by: ksharma@suse.com
2014-05-28 16:31:56 +02:00
John Spray
3bf5338cc0 Merge pull request #259 from ceph/wip-journal-tool
mds_journal_migration task
2014-05-28 14:43:34 +01:00
John Spray
6f8015135c tasks/mds_journal_migration: create.
This is for testing the changes to CephFS
journal format, especially the migration
of journals from old to new format.

This is a teuthology task rather than being
a workunit script, because it involves
manipulating services and config during the
procedure.

Signed-off-by: John Spray <john.spray@inktank.com>
2014-05-28 14:39:41 +01:00
John Spray
554f8c81e0 task/ceph: separate ceph.conf into function
This is so that other tasks can modify ceph.conf
as needed.

Signed-off-by: John Spray <john.spray@inktank.com>
2014-05-28 14:39:41 +01:00
John Spray
79dfe2cbef tasks: Use '_' instead of '-' in names
Python modules are not allowed to contain hyphens
in the name.  Using hyphens only works here because
we're using low level __import__.

Change run_tasks to replace '-' with '_' when reading
configs, and rename the task modules to have valid
python module names.

Aside from general hygiene, the motivation to do this
is to allow one task to build on code in another
task by importing it.

Signed-off-by: John Spray <john.spray@inktank.com>
2014-05-28 14:39:41 +01:00
Zack Cerza
d3c230071a Don't crash when an invalid branch is passed
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-05-22 12:26:50 -05:00
Zack Cerza
a6aebe2bbc Set results_server by default
So that queued jobs get pushed to paddles

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-05-22 08:49:26 -05:00
Zack Cerza
3b382b7011 Process queued jobs synchronously
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-05-21 16:18:00 -05:00
Warren Usui
9a376d71e0 Merge pull request #260 from ceph/wip-8422
Add exception hook to teuthology-worker
2014-05-21 10:24:56 -07:00
Zack Cerza
08ace5465e Add exception hook to teuthology-worker
Workers processes are dying occasionally, and this should cause the
exceptions to be logged.

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-05-21 11:34:11 -05:00
Zack Cerza
d58d2c152e Merge pull request #258 from ceph/wip-fix-format-samba
Fixed formatting and added a better variable for time out
2014-05-19 18:33:22 -05:00
Yuri Weinstein
c8881bb7fa Fixed broken quotations
Signed-off-by: Yuri Weinstein <yuri.weinstein@inktank.com>
2014-05-19 16:18:48 -07:00
Yuri Weinstein
6e893ad219 Fixed formatting and added a better variable for time out
Signed-off-by: Yuri Weinstein <yuri.weinstein@inktank.com>
2014-05-19 16:03:30 -07:00
Samuel Just
216e5c9a80 task/: add populate_rbd_pool for quickly spinning up a pool with rbd stuff
Signed-off-by: Samuel Just <sam.just@inktank.com>
2014-05-19 15:04:54 -07:00
Samuel Just
8d9d724b7f ceph_manager: reset timeout if we are making progress in wait_for_recovery
Signed-off-by: Samuel Just <sam.just@inktank.com>
2014-05-19 14:10:20 -07:00
Zack Cerza
33dbffff1d Avoid calling yaml.safe_load() twice on each job
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-05-19 11:12:07 -05:00
Zack Cerza
debf84ed88 Split progress indicator out to separate functions
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-05-19 11:12:07 -05:00
Yuri Weinstein
a24218463d increased sleep to 100 as 90 seemed insufficient
Signed-off-by: Yuri Weinstein <yuri.weinstein@inktank.com>
2014-05-19 08:11:20 -07:00
Yuri Weinstein
6eac0baf88 Added more logging
Signed-off-by: Yuri Weinstein <yuri.weinstein@inktank.com>
2014-05-17 10:15:57 -07:00
Yuri Weinstein
686c01b58b Added sleep log info line
Signed-off-by: Yuri Weinstein <yuri.weinstein@inktank.com>
2014-05-17 10:07:52 -07:00
Zack Cerza
333c7ad7d0 Merge pull request #257 from ceph/wip-samba-fix
increased time wait to 90 sec so tests does not fail
2014-05-15 11:48:29 -05:00
Yuri Weinstein
1a503afff4 increased time wait to 90 sec so tests does not fail 2014-05-15 09:33:44 -07:00
Zack Cerza
d8c74392cd Add Pulpito links to teuthology emails
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-05-15 11:08:10 -05:00
Zack Cerza
c3dccbf29a Downgrade paramiko to work around an SFTP bug
See http://tracker.ceph.com/issues/8327

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-05-14 22:43:29 -05:00
Zack Cerza
c7d7abae2b Merge pull request #256 from ceph/wip-6542-wusui
Add missng docstrings to repair_test.py
2014-05-13 16:26:12 -05:00
Warren Usui
5dfc570004 Add missng docstrings to repair_test.py
Fixes: 6542
Signed-off-by: Warren Usui <warren.usui@inktank.com>
2014-05-13 14:02:13 -07:00
Zack Cerza
010f83f19b Fix unit tests under Jenkins
os.getlogin() was throwing:
  OSError: [Errno 25] Inappropriate ioctl for device

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-05-13 13:09:04 -05:00