Merge "Minor testing clean-up"

playbooks/inventory/group_vars/all

@@ -9,10 +9,10 @@ mysql_password:
 # virtual machines for the hardware instead of real hardware.
 # testing: true
 #
-# Normally this user should be ubuntu, however if cirros is used,
+# Normally this user should be root, however if cirros is used,
 # a user may wish to define a specific user for testing VM
-# connectivity during  atest sequence
-testing_user: ubuntu
+# connectivity during a test sequence
+testing_user: root
 http_boot_folder: /httpboot
 nginx_port: 8080
 ssh_public_key_path: "{{ ansible_env.HOME }}/.ssh/id_rsa.pub"

playbooks/roles/bifrost-test-vm/README.md

@@ -25,7 +25,7 @@ hosts: testvm
   name: "Tests connectivity to the VM"
   sudo: no
   gather_facts: yes
-  remote_user: ubuntu
+  remote_user: root
   roles:
     - role: bifrost-test-vm
 

playbooks/roles/bifrost-test-vm/tasks/main.yml

@@ -17,7 +17,7 @@
   # hostname. This is because cirros lacks sftp support.
   raw: hostname
   register: instance_hostname
-- name: 'Error if hostname is set to "ubuntu"'
+- name: 'Error if hostname is set to "ubuntu", "cirros", "debian", or "centos"'
   # TODO: Presently this step is unable to cycle through each host and verify
   # it's hostname is properly set. Perhaps if there was some way to extract
   # the data on each host from Ironic's DB and then verify that information
@@ -28,4 +28,4 @@
   # NOTE(TheJulia): If we go down the path of additional validation, we need
   # to keep things like Cirros in mind.
   fail: msg='If this fails, the configdrive may not have been used as the hostname was set to the default.'
-  when: '"ubuntu" in instance_hostname.stdout or "cirros" in instance_hostname.stdout'
+  when: '"ubuntu" in instance_hostname.stdout or "cirros" in instance_hostname.stdout or "debian" in instance_hostname.stdout or "centos" in instance_hostname.stdout'