From 4e17b938a06ea764f08efa8279f689d493c7b7b0 Mon Sep 17 00:00:00 2001 From: Julia Kreger Date: Wed, 1 Apr 2015 08:34:18 -0400 Subject: [PATCH] Updated comments in bifrost-test-vm Wrapped the previous comments in the bifrost-test-vm role and added an additional note about the future possibility of using the inventory it's self to define the nodes. --- roles/bifrost-test-vm/tasks/main.yml | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/roles/bifrost-test-vm/tasks/main.yml b/roles/bifrost-test-vm/tasks/main.yml index d2ca20c9e..9f9bc632d 100644 --- a/roles/bifrost-test-vm/tasks/main.yml +++ b/roles/bifrost-test-vm/tasks/main.yml @@ -15,8 +15,12 @@ - name: "Execute ping step to verify connectivity and login to the host. If this fails, the configdrive may not have loaded." ping: - name: 'Error if hostname is set to "ubuntu"' - # 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 + # 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 # as a host-level fact that can be verified. + # + # TODO: As time goes on, we may move to leveraging inventory information + # which will allow this role to do validation of the remote node. fail: msg='If this fails, the configdrive may not have been used as the hostname was set to the default "ubuntu"' when: ansible_hostname == "ubuntu"