bifrost/playbooks/roles/bifrost-deploy-nodes-dynamic/tasks/main.yml
Julia Kreger 5cf7a66de7 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
2015-06-24 19:47:18 +00:00

50 lines
2.1 KiB
YAML

# Copyright (c) 2015 Hewlett-Packard Development Company, L.P.
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# 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
auth: None
ironic_url: "{{ ironic_url }}"
uuid: "{{ uuid }}"
state: present
config_drive: "http://{{ hostvars[inventory_hostname]['ansible_' + network_interface]['ipv4']['address'] }}:{{ nginx_port }}/configdrive-{{ uuid }}.iso.gz"
instance_info: "{{ instance_info }}"
delegate_to: localhost
when: instance_info is defined
- name: "Collect the checksum of the deployment image."
local_action: stat path={{deploy_image}}
register: test_deploy_image
when: instance_info is not defined
- name: "Deploy to Hardware - Bifrost Default"
os_ironic_node:
auth_type: None
auth: None
ironic_url: "{{ ironic_url }}"
uuid: "{{ uuid }}"
state: present
config_drive: "http://{{ hostvars[inventory_hostname]['ansible_' + network_interface]['ipv4']['address'] }}:{{ nginx_port }}/configdrive-{{ uuid }}.iso.gz"
instance_info:
image_source: "http://{{ hostvars[inventory_hostname]['ansible_' + network_interface]['ipv4']['address'] }}:{{ nginx_port }}/{{deploy_image_filename}}"
image_checksum: "{{ test_deploy_image.stat.md5 }}"
image_disk_format: "raw"
root_gb: 10
delegate_to: localhost
when: instance_info is not defined