Remove setting of unused 's3_branch' value

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
Add new job_status module

Provides methods for getting and setting job status. This is necessary
because we used to just use a boolean called 'success'. In order to stop
reporting infrastructure errors as failures, we need more than just a
boolean.

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
Use new functions provided by job_status

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
Instead of setting status='pass', set success=True

Keeping this old behavior will mean that any tasks that are not updated
to use set_status() and still simply set success=False will still be
able to properly report their failures.

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
lock.lock_many(): use passed os_{type,version}, not ctx

Affects teuthology --lock with os_{type,version} in the yaml.

Fixes: #9811
Signed-off-by: Dan Mick <dan.mick@inktank.com>
task.ssh_keys(): manage authorized_keys, not keys2

See the issue for a description, but authorized_keys2 is dead
on RHEL7 and dying elsewhere.

Fixes: #9812
Signed-off-by: Dan Mick <dan.mick@inktank.com>
Merge pull request #343 from ceph/wip-dmick

Fixes for #9811 and #9812
Fix linter errors

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
Better error handling in list_locks()

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
task/nfs: stop nfs server after test finishes more thoroughly

In addition to "exportfs -au", we take a sledgehammer to it
by shutting down all the worker threads and flushing out
the client export table.

Signed-off-by: Yan, Zheng <zyan@redhat.com>
Signed-off-by: Greg Farnum <Greg@inktank.com>