rados.py: Add pool_snaps option for ceph_test_rados test command

Fixes: #8284

Signed-off-by: David Zafman <david.zafman@inktank.com>
rbd_fsx: expose krbd and related fsx options

Expose

-K (enable krbd mode) through 'krbd',
-Z (use direct IO) through 'direct_io',
-U (disable randomized striping) through 'randomized_striping',
-H (disable discard ops) through 'punch_holes',

-r readbdy (read alignment) through 'readbdy',
-w writebdy (write alignment) through 'writebdy',
-h holebdy (discard alignment) through 'holebdy'.

Signed-off-by: Ilya Dryomov <ilya.dryomov@inktank.com>
Merge pull request #249 from ceph/wip-8284

rados.py: Add pool_snaps option for ceph_test_rados test command
Merge pull request #252 from ceph/wip-fsx-krbd

rbd_fsx: expose krbd and related fsx options

Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
Further clarify 'too many values to unpack' error.

Many errors in yaml configurations cause ValueError to get thrown
with the message 'too many values to unpack.'  A previously reverted
change tried to handle all these situations and print an appropriate
message.

The current behavior of throwing the ValueError exception and exiting
is probably what we still want in these cases.  So instead of handling
the error, the code now checks for the exception at the top-most call
and displays appropriate messages in log.error and in the ctx.summary
data itself.

Fixes: 7510
Signed-off-by: Warren Usui <warren.usui@inktank.com>
Merge pull request #253 from ceph/wip-7510-wusui

Further clarify 'too many values to unpack' error.
Use binary flag for paramiko ChannelFiles

This works around http://tracker.ceph.com/issues/8313

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
Revert "Revert "Show hostname instead of IP in errors""

This reverts commit 10fee0e368750cf4cd953db5700df59c7f611119.

Conflicts:
	teuthology/orchestra/run.py
Use Unicode format strings

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
Consolidate log file setup into shared function

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
Use Remote.hostname in logs

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
Add Remote.ensure_online()

If the connection is alive, do nothing. If not, reconnect. Allow any
exceptions to bubble up to the caller. This is intended to fix unhelpful
'Bad File Descriptor' errors we were seeing when VMs go down.

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
Use 'true' instead of 'echo online'

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
Pass hostname to execute()

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
Fix PEP-8 issues

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
f pep-8
don't use name
use name in exc