adds 50% test coverage to teuthology.worker without any changes to teuthology.worker itself

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

Merge pull request #375 from ceph/wip-dmick

A bunch of little fixes found during addition of calamari_nosetest task

Merge pull request #376 from ceph/wip-sha-compare-fix

kernel: fix commit hash matching in need_to_install()

Merge pull request #377 from ceph/wip-kill-fixes

Two minor teuthology-kill fixes

No need to invoke teuthology-nuke if there is nothing to nuke

Slipped in with 26803da18d1b ("Rewrite portions of teuthology.kill").

Signed-off-by: Ilya Dryomov <idryomov@redhat.com>

Don't fetch owner info from job_info when --owner given

Commit 293c2bd49c0d ("Be more informative if owner info is missing")
introduced --owner but it is still pulled from job_info no matter what.
Fix it.

Signed-off-by: Ilya Dryomov <idryomov@redhat.com>

kernel: fix commit hash matching in need_to_install()

We don't need to install a new kernel if the "want" (version) commit
hash and the "have" (cur_sha1) commit hash match but the former is
shorter than the latter.

Signed-off-by: Ilya Dryomov <idryomov@redhat.com>

misc.append_lines_to_file: respect sudo

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

misc.create_file: if sudo, must sudo for chmod as well.

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

misc.get_system_type(): fix distro=False version=True, add comments

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

orchestra/run.py: document use of Raw()

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

packaging.py: Use misc.get_system_type()

Dump _get_relmap() and _RELEASE_MAP in favor of common code.
Also remove the repo management stuff; it was only used by the
now-obsolete calamari task.

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

Remove obsolete calamari task

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

packaging.py: import misc as misc, not teuthology

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

Merge pull request #374 from andrewschoen/orchestra-test

test improvements

moved all the cluster.write_file tests into their own class; rewrote the existing test to have one assertion per test case

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

Merge pull request #373 from andrewschoen/orchestra-test

added a test for teuthology.orchestra.cluster.write_file

Refuse to schedule against missing suite branches

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

removed fudge and unneeded asserts

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

added a test for teuthology.orchestra.cluster.write_file

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

Merge pull request #372 from ceph/wip-10034-wusui

Added calamari-server

Merge pull request #366 from dachary/wip-10170-write-file

fix Cluster::write_file extra argument & support owner

Added calamari-server

Calamari-server has been added to the list of installed packages
to be removed.  Also added salt-common and salt-minion to the list
of rpms to be removed.

Fixes #10034
Signed-off-by: Warren Usui <warren.usui@inktank.com>

Merge pull request #371 from ceph/docs

Add autogenerated API docs to Sphinx output

Add autogenerated API docs to Sphinx output

Also drop -W for now. We'll have to focus on PEP-257 stuff later.

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

Merge pull request #369 from ceph/docs

Add lab setup docs

Reorganize documentation and Sphinx files

This should allow a more consistent experience between reading
Sphinx-generated HTML docs and just using GitHub's renderer.

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

Add docs to tox envlist

So PRs don't break documentation

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

Remove fancy source file handling

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

Add stub source files linking to canonical documents

Now we can drop the Makefile magic.

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

Merge pull request #360 from andrewschoen/docopt-ls

port teuthology-ls to use docopt instead of argparse

adding tests for teuthology.ls, coverage now at 88%

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

Add LAB_SETUP.rst to Sphinx build

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

Add reference configs

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

Add lab setup doc

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

port teuthology-ls to use docopt instead of argparse

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

Check individual jobs for packages

Previously we were just checking one time per run to see if packages had
been built. Now, let's check each job since os_type values can differ.
Fail if any are missing. Results are cached to avoid hugging the
archive to death.

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

Revert "ceph.conf: filestore fiemap = true"

This reverts commit 0f97481ce44e0487ac6cffa051a05590f907a081.

Until we fix #10166

Merge pull request #367 from ceph/wip-rgw-swift

swift: set full access to subusers creation

swift: set full access to subusers creation

Default subuser permissions are 'none'.

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>

cluster: add owner argument to write_file

The misc.sudo_write_file now supports the optional owner argument, add
support for it.

Signed-off-by: Loic Dachary <ldachary@redhat.com>

cluster: remove extra write_file argument

http://tracker.ceph.com/issues/10170 Fixes: #10170

Signed-off-by: Loic Dachary <ldachary@redhat.com>

Merge pull request #365 from ceph/wip-kill-pid-check

Verify running pid before passing it to be killed.

Verify running pid before passing it to be killed.

So we don't kill other processes that have taken jobs pid since
job has been no longer running.

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

ceph.conf: filestore fiemap = true

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

Merge pull request #364 from ceph/wip-scheduling

Fix / work around gitbuilder problems

Choose SHA1 sums more intelligently

By checking the actual gitbuilder archive that we care about, if
--distro is passed. If it is not, tweak the default to 'rhel' instead of
'ubuntu' since that seems to produce better results. To be clear, this
is a workaround I'm putting in place since our gitbuilders never seem to
be in sync.

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

More useful logging when unlocking single nodes

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

Mention test name when failing to schedule

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

Add a default results_sending_email

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

Merge pull request #363 from ceph/wip-10089

drop default os_type of 'ubuntu' in teuthology-suite

Drop null values in placeholders

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

Merge pull request #358 from andrewschoen/docopt-porting

porting teuthology-coverage to docopt

added a simple test for teuthology.coverage.main

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

added a simple test for teuthology.coverage.main

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

refactored teuthology.coverage to use an dict from docopt instead of a namespace from argparse

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

Drop ubuntu as default distro for scheduled runs

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

added a test to make sure docopt was parsing things correctly for teuthology-coverage

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

cleaning up usage for teuthology-coverage

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

porting teuthology-coverage to docopt

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

ceph.conf: mon pg warn max object skew = 0 (disable warning)

Make the 'too few pgs' warnings go away from qa runs.

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

Merge pull request #357 from andrewschoen/tox-integration-tests

Moves all integration tests to an integration folder and provides a new ...

Merge pull request #356 from ceph/wip-default-machine-type

Allow setting default machine_type per-lab

Moves all integration tests to an integration folder and provides a new tox env to run only those tests

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

Allow setting default machine_type per-lab

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

Merge pull request #355 from ceph/wip-email

teuthology_results: add --dry-run

If timeout is None or zero, don't log

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

Add dry-run option

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

Use docopt instead of argparse

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

Use new config API

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

Merge pull request #354 from andrewschoen/tox-cov-report

Adds a coverage report when running the tox tests

Adds a coverage report when running the tox tests

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

Merge pull request #351 from ceph/wip-9925-wusui

Allow more strings in filters

Mention hostnames in ConnectionLostErrors

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

More suite filter changes.

Comment change, and update help message to reflect the ability to
use a comma separated list as a filter.

Merge pull request #352 from ceph/wip-update-vm-hostkeys

Continue fix for 9871

Merge pull request #353 from ceph/wip-9975

remove configuration files when nuking

use ctx instead of remotes

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

Also look for /etc/teuthology.yaml

First we look for ~/.teuthology.yaml - if that is not found, we look for
a system-wide configuration in /etc/teuthology.yaml

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

Fix duplicate test name

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

call the new helper in nuke_helper

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

create a configuration removal function in nuke.py

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

Don't explode if results_sending_email isn't set

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

Suppress tracebacks when coverage isn't configured

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

Install virtualenv if it isn't present

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

Also install mysql on MacOS

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

Fix typo

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

Drop default watchdog_interval to 120s

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

Explicitly pass json header

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

Allow omitting 'tasks' stanza in job definitions

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

Abstract the lab domain (e.g. front.sepia.ceph.com)

This should make usage in other labs easier.

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

If lock_server is not set, check_lock() no-ops

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

ceph.conf: set mon pg warn max per osd = 10000

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

Continue fix for 9871

The existing logic was too tortured, and the previous fix was bad.
Rewrite to extract "get_statuses()" to a worker, and make it
extremely clear which VM hostkeys are being updated and why.

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

Add libmysqlclient-dev to ubuntu requirements

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

Revert "Remove MySQL requirements"

This reverts commit c69051367069eab4dd608753e14ffc8cee75b50b.

Remove MySQL requirements

These are only needed for teuthology-coverage, which requires a good
amount of extra setup (if it even works).

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

Allow more strings in filters

A comma separated list can now be used in filter and filter-out.

Fixes: #9925
Signed-off-by: Warren Usui <warren.usui@inktank.com>

Fix for 9871 was wrong: original <= should always have been <

...but now it matters.  lock_list returns exactly the length needed,
and so the 'failure' case is strict less-than.

Suggested-by: Sandon VanNess <svanness@redhat.com>
Signed-off-by: Dan Mick <dan.mick@redhat.com>

Merge pull request #348 from ceph/wip-9871

lock.py: update keys for any owned VMs, even if none mentioned on CLI