Remove un-necessary checks from the dynamic test

The direction bifrost roles are heading it may not make sense
to have secondary validation steps outside of the initial steps.

i.e. We should be attempting to validate we've completed x action
before leaving x action's steps.  Separating them makes no sense.

As such, we're removing the un-necessary test step from the dynamic
test.  Additionally noted the un-necessary nature and possibilites
in the node deployment given we can support users passing in their
own pre-formatted instance_info.

Change-Id: Id2bb814809e5f6c632c085a5eb75107a7ea126bb

playbooks/roles/bifrost-deploy-nodes-dynamic/tasks/main.yml

@@ -12,6 +12,11 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 ---
+# TODO(TheJulia) It would make sense to perform basic validation prior
+# to deployment, although direct definition of instance info on
+# the pass-through could mean that the user could deploy
+# things that are not directly accessible or reasonable
+# to be inspected.
 - name: "Deploy to Hardware - Using custom instance_info."
   os_ironic_node:
     auth_type: None

playbooks/test-bifrost-dynamic.yaml

@@ -36,12 +36,6 @@
   connection: local
   roles:
     - role: ironic-enroll-dynamic
-- hosts: localhost
-  connection: local
-  name: "Validates prior to deployment"
-  sudo: yes
-  roles:
-    - role: bifrost-validate-host-for-deploy
 - hosts: baremetal
   name: "Creat configuration drive files and deploy machines."
   sudo: no