8954bebfaf
After changing from ubuntu to debian, testing was never updated to validate that the hostname was not ubuntu. Since we support simple-init, we should check for multiple default OS names and fail accordingly if they are found. Additionally, changed the default testing user to root as simple init writes the key to the root user's authorized_keys file. Change-Id: Icce62d0b9256188dbc9b49d948c99bb0c469ae6b
32 lines
1.8 KiB
YAML
32 lines
1.8 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.
|
|
---
|
|
- name: "Execute ping step to verify connectivity and login to the host. If this fails, the configdrive may not have loaded."
|
|
# NOTE(TheJulia): This is executed as a raw command to validate the remote
|
|
# hostname. This is because cirros lacks sftp support.
|
|
raw: hostname
|
|
register: instance_hostname
|
|
- 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
|
|
# 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.
|
|
# 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 or "debian" in instance_hostname.stdout or "centos" in instance_hostname.stdout'
|