Use get_instance helper to find keystone, glance or nova nodes
Use KeystoneHelper.keystone_settings in more places
Remove unneeded variables
Do not make tempest_path a proposal attribute

And also use it everywhere, instead of just "sometimes".
Only avoid using /etc/tempest/tempest.conf if PFS is used
Fix fetching keystone settings by always using keystone_settings
Fix fetching the keystone settings

We don't reference the keystone instance from the tempest attributes, so
instead use the data from the nova attributes on the nova node.
Remove need for search for glance nodes in config recipe
Use venv_prefix_path where needed
Use a loop to create subdirectories

This helps make the code more readable.
Make bash code to upload images a bit more solid

Fail earlier if there's an issue.
Avoid hard-coding of cirros version
Reorder searches for openstack components
Do not respect the use_gitrepo setting for installing clients

This makes no sense: we know the stable clients are working anyway with
glance/nova from git.
Install clients for all openstack components

We use some of them in the config recipe, and anyway, tempest is testing
the CLI tools.
Always run neutron tests in tempest

We require neutron to be installed to have nova installed, so we know
it's always there.
Look if there's an endpoint to know if something should be tested
Reorganize order of variables for tempest.conf template

No functional change, but more readable.
Add FIXME for searches that do not have a fully correct query

We always need to search with an environment...
Remove use of .venv/bin/activate for CLI tools

Since the tools are always installed from packages, this is not required
anymore.