-
Notifications
You must be signed in to change notification settings - Fork 20
Add script to prepare bare metal nodes information #19
Comments
https://github.com/schmaustech/kni-preflight is an example of an existing script to help with this The script uses this ansible module - https://docs.ansible.com/ansible/latest/modules/redfish_facts_module.html Note that check_dhcp is an ELF binary - not clear where it comes from |
It appears to be a build of this: https://github.com/nagios-plugins/nagios-plugins/blob/master/plugins-root/check_dhcp.c |
That script feels more complicated than I would have expected. I thought we didn't need the bootMACAddress for real hardware because the inspection process would discover it. Maybe that's only if we use the auto-discovery feature, though? |
The kni-preflight script is doing more then just grabbing MAC addresses as it is also validating that the DHCP server is going to pass out a IP address for that MAC address via check check_dhcp. It also checks DNS and validates that the api,ns1 and *.apps VIPS are setup in DNS. The complexity of it arises because I was told to use Ansible. Personally I can grab the MACs via Python as that was what I used in kni-wipe: https://github.com/schmaustech/kni-wipe |
So I went ahead and added the ability for my kni-preflight to generate a install-config.yaml file based on the sample I found here: https://github.com/openshift/installer/blob/master/docs/user/metal/install_ipi.md. I have tested it on a hardware setup in Westford and produced the following output which is derived from data kni-preflight was already discovering:
|
You are correct we used the check_dhcp.c code. |
check_dhcp has been replaced by check_dhcp.py now |
preflight does this for the appliance. Preflight was merged with #92 non-appliance will have to stay manual for now |
As per https://github.com/openshift/installer/blob/master/docs/user/metal/install_ipi.md we need information like the following in
install-config.yml
with information about the bare metal nodes:The text was updated successfully, but these errors were encountered: