Fix typo

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Merge pull request #134 from ceph/wip-6546

Fix race condition between tests starting and teuthology-results being run

Make check for running jobs part of the loop

Use a new, tested implementation to check for running jobs.

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

ResultsSerializer.running_jobs_for_run() and test

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Merge pull request #133 from ceph/wip-store-results

Fixes for reporting and verbosity

Put machine_type in config.yaml

Reason being, if --worker is passed to teuthology-schedule, we are
already placing the job in the tube for that machine type. If the job
config doesn't specify a machine_type, it will later be assigned to
'plana' by teuthology-run without actually modifying the config. This
can result in jobs running on hardware other than what was requested.

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Make verbosity propagate correctly to modules

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Debug output for all http requests

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Fix broken lambda

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Be more verbose when waiting for free machines

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Give reporting a verbose flag

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Lack of a job_id should be a warning

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Merge pull request #132 from ceph/wip-refactor-scripts

Fix imports; fix keyscan_check()

Exempt teuthology-updatekeys from test_invalid

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Ignore the exit code of ssh-keyscan

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Fix a circular import

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Move imports to top-level

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Merge pull request #131 from ceph/wip-store-results

Reporting tweakage

Move part of report_all_runs() into report_runs()

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Set a default timeout of 20s for HTTP connections

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Move monkey patching to __init__.py

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Don't attempt to report if there's no job_id

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Add basic tests for scripts

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Properly express conflicting options

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Move teuthology's arg parsing to scripts/

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

PEP-8

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

PEP-8

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Move teuthology-coverage's arg parsing to scripts/

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

PEP-8

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Move teuthology-results' arg parsing to scripts/

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

PEP-8

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Move teuthology-report's arg parsing to scripts/

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Move teuthology-updatekeys' arg parsing to scripts/

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Move schedule() to new schedule module

Move teuthology-schedule's arg parsing to scripts/

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

PEP-8

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Move teuthology-lock's arg parsing to scripts/

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

PEP-8

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Move teuthology-worker's arg parsing to scripts/

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Move teuthology-ls's arg parsing to scripts/

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

More PEP-8 cleanup

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Move teuthology-suite's arg parsing to scripts/

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Move teuthology-nuke's arg parsing to scripts/

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Add scripts/ module

PEP-8 cleanup

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

ignoring -X option in apache as it was causing s3test failures on the rpms.

Signed-off-by: tamil <tamil.muthamizhan@inktank.com>

Merge pull request #126 from ceph/wip-store-results

Reflect paddles API change: jobs in jobs/

Reflect paddles API change: jobs in jobs/

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Merge pull request #125 from ceph/wip-store-results

Add more tests for ReportSerializer

Add more tests for ReportSerializer

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Merge pull request #124 from ceph/wip-store-results

Various fixes related to reporting

Tweak logic of try_push_job_info()

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Make most of lock_machines() PEP-8 clean.

Also, use some better variable names.

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Make teuthology-schedule also print the job's name

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Store the job_id as a str, not an int.

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Move teuthology.report import; add large warning

Because of the monkey patching, if we get the import order wrong bad
things happen.

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Merge pull request #123 from ceph/wip-store-results

Fix issue with monkey patching

Add (and use) try_push_job_info() for easy pushin'

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Move new imports to inside main()

The monkey patch stuff was causing problems again.

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

schedule_suite.sh: include default machine type in job name

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

Merge pull request #121 from ceph/wip-store-results

Submit teuthology results to a paddles instance

Add test for test_all_runs

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Clarify job vs. run

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Create a FakeArchive class to forge test results

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Escape reserved characters in URIs

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Report results synchronously.

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Add more documentation.

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Add create_run() and push_job_info() with docs.

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

s/submit_/report_/g

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Rename ResultsPoster to ResultsReporter

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Add create_run; allow passing json to submit_job

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Overhaul posting logic to avoid some corner cases

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Add flag for refreshing already-pushed runs

Also tweak implementation of save_last_run

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Remove old method used for testing

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Add --server flag

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Reuse connection objects.

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Add argument parsing to make this a proper script.

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

A first pass at integration with paddles

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Merge pull request #120 from ceph/wip-6243

Wip 6243

rest-api.py: test that ceph-rest-api is up and servicing requests

Wait for the daemon to actually be ready before continuing with other
tasks.  Loop 12 times, waiting 5s, before deciding something's wrong.

Fixes: #6243
Signed-off-by: Dan Mick <dan.mick@inktank.com>

rest-api.py: fix up client identification

The code to extract which clients to start the API on was confused;
fix it up.  Also, strip off 'client.' instead of 'clients' (since it
was done with len(), the result was the same, but confusing)

Signed-off-by: Dan Mick <dan.mick@inktank.com>

Revert "ceph.conf: filestore sloppy crc = true"

This reverts commit 631476b9796f817e5f1654affb6b2a5aacfe9a80.

This fails on ext4 because of xattr size limits.

nuke: s/run_name/name/

This matches an existing argument (with the same meaning) and
avoids an error like

2013-10-01T17:20:35.395 CRITICAL:root:  File "/var/lib/teuthworker/teuthology-master/virtualenv/bin/teuthology", line 9, in <module>
    load_entry_point('teuthology==0.0.1', 'console_scripts', 'teuthology')()
  File "/home/teuthworker/teuthology-master/teuthology/run.py", line 235, in main
    nuke(ctx, log, ctx.lock)
  File "/home/teuthworker/teuthology-master/teuthology/nuke.py", line 391, in nuke
    if ctx.run_name:

2013-10-01T17:20:35.395 CRITICAL:root:AttributeError: 'Namespace' object has no attribute 'run_name'

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

Merge pull request #118 from ceph/wip-nukeskip

Check description of machines before nuking when -a is passed

Merge pull request #95 from ceph/wip-empty

don't summarize an empty run

Check description of machines before nuking when -a is passed

When teuthology-nuke is passed with --archive/-a to kill and nuke
machines from an archive folder it blindly will nuke all the
targets it grabs from the config.yaml in the archive dir. This
change will check the description of locked machines and make sure
the run name is in the description. if not it removes the target
from the list passed to nuke().

Signed-off-by: Sandon Van Ness <sandon@inktank.com>

suite: don't schedule follow-on summary job for an empty run

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

suite: fix num_jobs count

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

Merge pull request #116 from ceph/wip-sudoers-sync

Reviewed-by: Josh Durgin <josh.durgin@inktank.com>

Merge pull request #115 from ceph/wip-priority

add priorities to scheduling

nuke: make half-hearted attempt to sync before reboot

We don't want to block on sync for fear of a hung kernel
mount.  However, we can give it a try and wait a few seconds
to get what we can.

This fixes a problem where our recent modifications to the
sudoers file are lost, with a 0 byte file left in its place,
because the task fails and we do a reboot -f -n.

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

Merge pull request #117 from ceph/wip-tests

Port tests to py.test; add a couple tests

Fix test

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Add test for skipping host key verification

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Port from nosetests to py.test

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Add another config test

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

Don't use properties

They're quite inconvenient for this use-case.

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>

ceph.conf: filestore sloppy crc = true

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

schedule: show priority on -s

Make output a bit nicer too.

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

schedule: priority option

Default to 1000.  Note that beanstalk priority is 2^31, so this
won't mix well with older code scheduling things at the
beanstalk default priority.

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