Commit Graph

117 Commits

Author SHA1 Message Date
Zack Cerza
f4321b0519 Skip the kernel stanza if the branch passed is '-'
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-06-25 12:54:24 -06:00
Zack Cerza
ba1eae988e Add more docstrings
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-06-25 12:54:24 -06:00
Zack Cerza
e1597a036f No need to explicitly return None here
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-06-25 12:54:24 -06:00
Zack Cerza
7082a3dff0 Add a few more docstrings and a debug statement
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-06-25 12:54:23 -06:00
Zack Cerza
743edd6b70 Fix a couple linter errors
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-06-25 12:54:23 -06:00
Zack Cerza
173c7b84f6 Fix bug where the base yaml wasn't being merged
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-06-25 12:54:23 -06:00
Zack Cerza
fd13a18c83 With dry_run, quote any individual args
So that individual commands can be copied and pasted
in order to execute them individually.

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-06-25 12:54:23 -06:00
Zack Cerza
91c7385988 Raise ScheduleFailError even if not sending email
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-06-25 12:54:23 -06:00
Zack Cerza
356ab7e06d Do everything that schedule_suite.sh does
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-06-25 12:54:23 -06:00
Zack Cerza
89b38dd347 Don't schedule an email job if email isn't set
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-06-25 12:54:23 -06:00
Zack Cerza
26fdef0a75 Split prepare_and_schedule() out of main()
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-06-25 12:54:23 -06:00
Zack Cerza
1be15edbcb Use the return value of schedule_suite()
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-06-25 12:54:23 -06:00
Zack Cerza
19d73071d1 Don't reread the entire yaml file...
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-06-25 12:54:22 -06:00
Zack Cerza
d92f538549 Remove unused functions
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-06-25 12:54:22 -06:00
Zack Cerza
0aabafd65f 'and' is a thing
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-06-25 12:54:22 -06:00
Zack Cerza
675f087b28 Take machine_type instead of worker
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-06-25 12:54:22 -06:00
Zack Cerza
d14c44f4e2 Add functions for querying gitbuilder repos
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-06-25 12:54:22 -06:00
Zack Cerza
43505b2ace Only allow scheduling one suite per call.
Also remove all traces of the extra and confusing term 'collection'

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-06-25 12:54:22 -06:00
Zack Cerza
f5af797ce3 Port to docopt
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-06-25 12:54:22 -06:00
Zack Cerza
1ccbff8dbf Refactor collection scheduling out of main()
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-06-25 12:54:22 -06:00
Zack Cerza
2a7de82498 Move teuthology-ls's implementation to ls.py
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2014-06-06 16:15:15 -05:00
Sage Weil
67844e0ef3 suite: allow priority to be specified when scheduling
Signed-off-by: Sage Weil <sage@inktank.com>
2014-03-26 10:09:12 -07:00
Sage Weil
94d73bd411 suite: fix build_matrix for + case
The + means we should concatenate everything in the directory.  Do that.

This was totally broken before (and unused until now).

Signed-off-by: Sage Weil <sage@inktank.com>
2014-03-06 17:56:01 -08:00
Yuri Weinstein
bd9748d562 Added --limit option to teuthology-suite.
Use --limit to limit the number of jobs being scheduled during
teuthology-suite. Also can be used with schedule_suite.sh via the
10th argument.

Signed-off-by: Sandon Van Ness <sandon@inktank.com>
Signed-off-by: Yuri Weinstein <yuri.weinstein@inktank.com>
2014-02-28 00:34:58 +00:00
Zack Cerza
ded5c219fc Remove needless arg from list_locks()
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2013-10-16 16:20:24 -05:00
Zack Cerza
494c3b1fbe Make verbosity propagate correctly to modules
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2013-10-11 17:10:57 -05:00
Zack Cerza
193b3112a3 Move teuthology-results' arg parsing to scripts/
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2013-10-10 19:09:34 -05:00
Zack Cerza
477e4ae2ce PEP-8
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2013-10-10 19:09:34 -05:00
Zack Cerza
5fd1cd0bed Move teuthology-ls's arg parsing to scripts/
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2013-10-10 19:09:34 -05:00
Zack Cerza
e847b1075a Move teuthology-suite's arg parsing to scripts/
Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
2013-10-10 19:09:34 -05:00
Sage Weil
431d264447 suite: don't schedule follow-on summary job for an empty run
Signed-off-by: Sage Weil <sage@inktank.com>
2013-10-01 16:09:54 -07:00
Sage Weil
504195a0d9 suite: fix num_jobs count
Signed-off-by: Sage Weil <sage@inktank.com>
2013-10-01 16:09:54 -07:00
Zack Cerza
8497437003 Remove unused imports
(cherry picked from commit 586817481119c4fc4a39f8804e7871a43491e01f)
2013-09-26 14:01:37 -05:00
Zack Cerza
1685de6741 Fix typo 2013-09-18 22:57:20 -05:00
Zack Cerza
d83912e0bf If duration is missing from summary.yaml, use 0 2013-09-18 14:08:23 -05:00
Zack Cerza
61ddeb6a03 Revert "suite: don't schedule follow-on summary job for an empty run"
This reverts commit a2b7c72896.
2013-09-17 13:47:42 -05:00
Sage Weil
4520c5f6f5 suite: do not calculate product on an empty list (take 2)
The original attempt to fix this with dda7954090
failed because I screwed up the indentation for the out.append(...) line.

Signed-off-by: Sage Weil <sage@inktank.com>
2013-09-16 20:22:09 -07:00
Zack Cerza
e65d6e2a00 Revert "suite: do not calculate product on an empty list"
This reverts commit dda7954090ccc9ffd8ecd961d6b530a0e6d24964.

This commit was causing the number of jobs scheduled per suite to be
multiplied by anywhere from 2 to 7 or more.
2013-09-16 18:33:21 -05:00
Alfredo Deza
568eccf86e Merge pull request #91 from ceph/wip-suite
suite: do not calculate product on an empty list
2013-09-13 08:41:35 -07:00
Sage Weil
a2b7c72896 suite: don't schedule follow-on summary job for an empty run
Signed-off-by: Sage Weil <sage@inktank.com>
2013-09-13 08:39:30 -07:00
Sage Weil
5a083894e0 suite: be verbose about how many jobs we generate
Signed-off-by: Sage Weil <sage@inktank.com>
2013-09-13 08:39:13 -07:00
Sage Weil
dda7954090 suite: do not calculate product on an empty list
This fixes a bug where a directory with just % generates a
bogus job item with no actual content.  (e.g.,

$ find basic
basic
basic/%
$ teuthology-suite --dry-run --base . --collections basic --name foo
INFO:teuthology.suite:Collection basic in ./basic
INFO:teuthology.suite:configs [('basic/{}', [])]

...which then blows up because the job yaml is empty.  With
the fix, there are no generated jobs:

$ teuthology-suite --dry-run --base . --collections basic --name foo
INFO:teuthology.suite:Collection basic in ./basic
INFO:teuthology.suite:configs []

(The configs print was temporarily added for debugging
purposes.)

Signed-off-by: Sage Weil <sage@inktank.com>
2013-09-13 08:22:12 -07:00
Zack Cerza
31f0f7ad7e Fix get_http_log_path(), update callers, add tests 2013-09-12 17:28:50 -05:00
Zack Cerza
4699ad1c11 Don't "import misc as teuthology" 2013-09-11 10:50:42 -05:00
Zack Cerza
c1ba276321 Move get_http_log_path 2013-09-11 10:50:42 -05:00
Zack Cerza
89f1139d83 Append '/' to the end of the log directory path 2013-09-04 15:16:33 -05:00
Zack Cerza
f36f16a669 Logs and Sentry links are public now. 2013-09-03 13:46:31 -05:00
Zack Cerza
3981a8f1af Never use 'except:' without specifying an Exception. 2013-08-30 11:10:05 -05:00
Zack Cerza
24991c4ae0 Add hung job description to email 2013-08-29 17:08:51 -05:00
Sage Weil
4fbec83490 suite: return [] for build_matrix base case
This gives us an empty list instead of a crash on an empty dir
or bad symlink

Signed-off-by: Sage Weil <sage@inktank.com>
2013-08-28 21:53:41 -07:00