[codespell] Fixing Spelling Mistakes
This is the first in a series of commits to add support for codespell. This is continuning the process completed in ironic-python-agent. Future Commits will add a Tox Target, CI support and potentially a git-blame-ignore-revs file if their are lots of spelling mistakes that could clutter git blame. Change-Id: I41b2db35cdf21917ac20ed0866af3b9c94789f0b
This commit is contained in:
parent
3e19ff80eb
commit
4529d48afe
@ -12,7 +12,7 @@ with as few operational requirements as reasonably possible.
|
|||||||
|
|
||||||
The mission of bifrost is to provide an easy path to deploy ironic in
|
The mission of bifrost is to provide an easy path to deploy ironic in
|
||||||
a stand-alone fashion, in order to help facilitate the deployment of
|
a stand-alone fashion, in order to help facilitate the deployment of
|
||||||
infrastucture, while also being a configurable project that can consume
|
infrastructure, while also being a configurable project that can consume
|
||||||
other OpenStack components to allow users to easily customize the
|
other OpenStack components to allow users to easily customize the
|
||||||
environment to fit their needs, and drive forward the stand-alone
|
environment to fit their needs, and drive forward the stand-alone
|
||||||
perspective.
|
perspective.
|
||||||
|
@ -243,7 +243,7 @@ def _process_sdk(groups, hostvars):
|
|||||||
machine = cloud.get_machine(machine['uuid'])
|
machine = cloud.get_machine(machine['uuid'])
|
||||||
new_machine = {}
|
new_machine = {}
|
||||||
for key, value in machine.items():
|
for key, value in machine.items():
|
||||||
# NOTE(TheJulia): We don't want to pass infomrational links
|
# NOTE(TheJulia): We don't want to pass informational links
|
||||||
# nor do we want to pass links about the ports since they
|
# nor do we want to pass links about the ports since they
|
||||||
# are API endpoint URLs.
|
# are API endpoint URLs.
|
||||||
if key not in ['links', 'ports']:
|
if key not in ['links', 'ports']:
|
||||||
|
@ -300,7 +300,7 @@ See the built-in documentation for more details:
|
|||||||
|
|
||||||
The Ansible variables generated for installation are stored in a JSON file
|
The Ansible variables generated for installation are stored in a JSON file
|
||||||
(``baremetal-install-env.json`` by default) that should be passed via the
|
(``baremetal-install-env.json`` by default) that should be passed via the
|
||||||
``-e`` flag to subsequent playbook or command invokations.
|
``-e`` flag to subsequent playbook or command invocations.
|
||||||
|
|
||||||
.. _custom-ipa-images:
|
.. _custom-ipa-images:
|
||||||
|
|
||||||
|
@ -35,7 +35,7 @@ systems not managed by ironic. For example, create a file
|
|||||||
dhcp-match=set:<tag>,<match criteria>
|
dhcp-match=set:<tag>,<match criteria>
|
||||||
dhcp-boot=tag:<tag>,<boot options>
|
dhcp-boot=tag:<tag>,<boot options>
|
||||||
|
|
||||||
The tag, match critera and boot options should be modified for your
|
The tag, match criteria and boot options should be modified for your
|
||||||
environment. Here we use dnsmasq tags to match against hosts that we want to
|
environment. Here we use dnsmasq tags to match against hosts that we want to
|
||||||
manage. dnsmasq will use the last matching tagged ``dhcp-boot`` option for a
|
manage. dnsmasq will use the last matching tagged ``dhcp-boot`` option for a
|
||||||
host or an untagged default ``dhcp-boot`` option if there were no matches.
|
host or an untagged default ``dhcp-boot`` option if there were no matches.
|
||||||
|
@ -20,7 +20,7 @@
|
|||||||
args:
|
args:
|
||||||
creates: "{{ bifrost_venv_dir }}/bin/python3"
|
creates: "{{ bifrost_venv_dir }}/bin/python3"
|
||||||
|
|
||||||
# NOTE(ebbex): (incase it's missing, which it shouldn't normally be)
|
# NOTE(ebbex): (in case it's missing, which it shouldn't normally be)
|
||||||
- name: install setuptools in venv
|
- name: install setuptools in venv
|
||||||
command: "{{ bifrost_venv_dir }}/bin/pip3 install setuptools"
|
command: "{{ bifrost_venv_dir }}/bin/pip3 install setuptools"
|
||||||
args:
|
args:
|
||||||
|
@ -45,7 +45,7 @@
|
|||||||
- when: vm_name not in existing_vms.list_vms
|
- when: vm_name not in existing_vms.list_vms
|
||||||
block:
|
block:
|
||||||
# NOTE(pas-ha) Ansible still lacks modules to operate on libvirt volumes
|
# NOTE(pas-ha) Ansible still lacks modules to operate on libvirt volumes
|
||||||
# NOTE(pas-ha) adding extra 1G for disk size to accomodate for partition table / configdrive
|
# NOTE(pas-ha) adding extra 1G for disk size to accommodate for partition table / configdrive
|
||||||
- name: create volume for vm
|
- name: create volume for vm
|
||||||
command: >
|
command: >
|
||||||
virsh --connect {{ test_vm_libvirt_uri }}
|
virsh --connect {{ test_vm_libvirt_uri }}
|
||||||
|
@ -81,7 +81,7 @@
|
|||||||
# NOTE(TheJulia): In order to prevent conflicts, stop
|
# NOTE(TheJulia): In order to prevent conflicts, stop
|
||||||
# dnsmasq to prevent conflicts with libvirt restarting.
|
# dnsmasq to prevent conflicts with libvirt restarting.
|
||||||
# TODO(TheJulia): We shouldn't need to do this, but the
|
# TODO(TheJulia): We shouldn't need to do this, but the
|
||||||
# libvirt dhcp instance conflicts withour specific config
|
# libvirt dhcp instance conflicts with our specific config
|
||||||
# and taking this path allows us to not refactor dhcp at
|
# and taking this path allows us to not refactor dhcp at
|
||||||
# this moment. Our DHCP serving should be refactored
|
# this moment. Our DHCP serving should be refactored
|
||||||
# so we don't need to do this.
|
# so we don't need to do this.
|
||||||
|
@ -302,11 +302,11 @@ ipa_download_headers: HTTP headers to use when downloading IPA.
|
|||||||
|
|
||||||
### Changing Database Configuration
|
### Changing Database Configuration
|
||||||
|
|
||||||
Bifrost utilizes a nested data stucture for the configuration of database.
|
Bifrost utilizes a nested data structure for the configuration of database.
|
||||||
Simply put:
|
Simply put:
|
||||||
|
|
||||||
- Values cannot be overrriden via set_fact.
|
- Values cannot be overridden via set_fact.
|
||||||
- Values cannot be overrriden via the command line with ``-e``.
|
- Values cannot be overridden via the command line with ``-e``.
|
||||||
- The entire data structure must be defined if is modified.
|
- The entire data structure must be defined if is modified.
|
||||||
|
|
||||||
Please see defaults/main.yml file for the structure named ``ironic``.
|
Please see defaults/main.yml file for the structure named ``ironic``.
|
||||||
|
@ -116,7 +116,7 @@ update_ipa: "{{ update_repos }}"
|
|||||||
ipa_add_ssh_key: false
|
ipa_add_ssh_key: false
|
||||||
# NOTE(Alex-Welsh): cirros_deploy_image_upstream_url has been deprecated in
|
# NOTE(Alex-Welsh): cirros_deploy_image_upstream_url has been deprecated in
|
||||||
# favor of custom_deploy_image_upstream_url but is included for backwards
|
# favor of custom_deploy_image_upstream_url but is included for backwards
|
||||||
# compatability. It should be swapped permanently to
|
# compatibility. It should be swapped permanently to
|
||||||
# custom_deploy_image_upstream_url in the next major release.
|
# custom_deploy_image_upstream_url in the next major release.
|
||||||
custom_deploy_image_upstream_url: "{{ cirros_deploy_image_upstream_url | default('https://download.cirros-cloud.net/0.5.3/cirros-0.5.3-x86_64-disk.img') }}"
|
custom_deploy_image_upstream_url: "{{ cirros_deploy_image_upstream_url | default('https://download.cirros-cloud.net/0.5.3/cirros-0.5.3-x86_64-disk.img') }}"
|
||||||
# By default bifrost will deploy dnsmasq to utilize as an integrated DHCP
|
# By default bifrost will deploy dnsmasq to utilize as an integrated DHCP
|
||||||
@ -227,7 +227,7 @@ inventory_dns: False
|
|||||||
|
|
||||||
# Set this to true to configure dnsmasq to populate the NTP servers
|
# Set this to true to configure dnsmasq to populate the NTP servers
|
||||||
# setting of dnsmasq.
|
# setting of dnsmasq.
|
||||||
# Several NTP servers can be specified, seperated by commas.
|
# Several NTP servers can be specified, separated by commas.
|
||||||
# dnsmasq_ntp_servers:
|
# dnsmasq_ntp_servers:
|
||||||
|
|
||||||
# Settings to enable the use of inspector
|
# Settings to enable the use of inspector
|
||||||
|
@ -296,12 +296,12 @@
|
|||||||
group: "ironic"
|
group: "ironic"
|
||||||
mode: "0775"
|
mode: "0775"
|
||||||
|
|
||||||
- name: "Retrieve interface IP informations"
|
- name: "Retrieve interface IP information"
|
||||||
set_fact:
|
set_fact:
|
||||||
itf_infos: "{{ internal_interface }}"
|
itf_infos: "{{ internal_interface }}"
|
||||||
dhcp_netaddr: "{{ dhcp_pool_start }}/{{ dhcp_static_mask }}"
|
dhcp_netaddr: "{{ dhcp_pool_start }}/{{ dhcp_static_mask }}"
|
||||||
when: enable_dhcp | bool
|
when: enable_dhcp | bool
|
||||||
- name: "Compute interface and DHCP network informations"
|
- name: "Compute interface and DHCP network information"
|
||||||
set_fact:
|
set_fact:
|
||||||
itf_netaddr1: "{{ itf_infos['address'] }}/{{ itf_infos['netmask'] }}"
|
itf_netaddr1: "{{ itf_infos['address'] }}/{{ itf_infos['netmask'] }}"
|
||||||
itf_netaddr2: "{{ itf_infos['network'] }}/{{ itf_infos['netmask'] }}"
|
itf_netaddr2: "{{ itf_infos['network'] }}/{{ itf_infos['netmask'] }}"
|
||||||
@ -336,7 +336,7 @@
|
|||||||
when:
|
when:
|
||||||
- enable_dhcp | bool
|
- enable_dhcp | bool
|
||||||
- itf_netaddr2 | ansible.utils.ipaddr('network') != dhcp_netaddr | ansible.utils.ipaddr('network')
|
- itf_netaddr2 | ansible.utils.ipaddr('network') != dhcp_netaddr | ansible.utils.ipaddr('network')
|
||||||
- name: "Computing new DHCP informations"
|
- name: "Computing new DHCP information"
|
||||||
set_fact:
|
set_fact:
|
||||||
dhcp_start_ip: "{{ dhcp_pool_start.split('.')[-1] }}"
|
dhcp_start_ip: "{{ dhcp_pool_start.split('.')[-1] }}"
|
||||||
dhcp_end_ip: "{{ dhcp_pool_end.split('.')[-1] }}"
|
dhcp_end_ip: "{{ dhcp_pool_end.split('.')[-1] }}"
|
||||||
|
@ -72,7 +72,7 @@
|
|||||||
name: bifrost-nginx-install
|
name: bifrost-nginx-install
|
||||||
tasks_from: install
|
tasks_from: install
|
||||||
|
|
||||||
# NOTE(TheJulia) While we don't necessarilly require /opt/stack any longer
|
# NOTE(TheJulia) While we don't necessarily require /opt/stack any longer
|
||||||
# and it should already be created by the Ansible setup, we will leave this
|
# and it should already be created by the Ansible setup, we will leave this
|
||||||
# here for the time being.
|
# here for the time being.
|
||||||
- name: "Ensure /opt/stack is present"
|
- name: "Ensure /opt/stack is present"
|
||||||
|
@ -24,7 +24,7 @@ Example Playbook
|
|||||||
----------------
|
----------------
|
||||||
|
|
||||||
hosts: localhost
|
hosts: localhost
|
||||||
name: "Tests DCHP settings"
|
name: "Tests DHCP settings"
|
||||||
become: no
|
become: no
|
||||||
gather_facts: yes
|
gather_facts: yes
|
||||||
remote_user: root
|
remote_user: root
|
||||||
|
@ -1,6 +1,6 @@
|
|||||||
---
|
---
|
||||||
fixes:
|
fixes:
|
||||||
- |
|
- |
|
||||||
The CI test playbook previously looked for the requirements respository in
|
The CI test playbook previously looked for the requirements repository in
|
||||||
``/opt/git/openstack/``. This has been changed to use the ``WORKSPACE``
|
``/opt/git/openstack/``. This has been changed to use the ``WORKSPACE``
|
||||||
environment variable when being executed.
|
environment variable when being executed.
|
||||||
|
@ -58,7 +58,7 @@ features:
|
|||||||
options: "rw,nodev,relatime"
|
options: "rw,nodev,relatime"
|
||||||
|
|
||||||
|
|
||||||
For more informations please refer to the following links:
|
For more information please refer to the following links:
|
||||||
`Disk Image Layout Section <https://docs.openstack.org/diskimage-builder/latest/user_guide/building_an_image.html#disk-image-layout>`_
|
`Disk Image Layout Section <https://docs.openstack.org/diskimage-builder/latest/user_guide/building_an_image.html#disk-image-layout>`_
|
||||||
`Standard Partitioning <http://teknoarticles.blogspot.fr/2017/07/build-and-use-security-hardened-images.html>`_
|
`Standard Partitioning <http://teknoarticles.blogspot.fr/2017/07/build-and-use-security-hardened-images.html>`_
|
||||||
`LVM Partitioning <http://teknoarticles.blogspot.fr/2017/11/security-hardened-images-with-volumes.html>`_
|
`LVM Partitioning <http://teknoarticles.blogspot.fr/2017/11/security-hardened-images-with-volumes.html>`_
|
||||||
|
@ -10,8 +10,8 @@ deprecations:
|
|||||||
- |
|
- |
|
||||||
Ironic has deprecated support for classic drivers. These were the drivers
|
Ironic has deprecated support for classic drivers. These were the drivers
|
||||||
that were prepended with ``agent`` or ``pxe``. The new default hardware type,
|
that were prepended with ``agent`` or ``pxe``. The new default hardware type,
|
||||||
which superceeds drivers, is ``ipmi``. The default deployment interface is
|
which supersedes drivers, is ``ipmi``. The default deployment interface is
|
||||||
``direct``, which superceeds the ``agent`` driver type. Support for classic
|
``direct``, which supersedes the ``agent`` driver type. Support for classic
|
||||||
Drivers, will be removed from Bifrost in the Rocky release cycle.
|
Drivers, will be removed from Bifrost in the Rocky release cycle.
|
||||||
issues:
|
issues:
|
||||||
- |
|
- |
|
||||||
|
@ -1,6 +1,6 @@
|
|||||||
---
|
---
|
||||||
deprecations:
|
deprecations:
|
||||||
- The ``node_network_info`` parameter has been deperecated
|
- The ``node_network_info`` parameter has been deprecated
|
||||||
in favor of the ``node_network_data`` parameter as the
|
in favor of the ``node_network_data`` parameter as the
|
||||||
related configuration drive ``network_info.json`` file was
|
related configuration drive ``network_info.json`` file was
|
||||||
misnamed originally, and should have been named
|
misnamed originally, and should have been named
|
||||||
@ -10,7 +10,7 @@ deprecations:
|
|||||||
drives will be removed in the Queens cycle.
|
drives will be removed in the Queens cycle.
|
||||||
fixes:
|
fixes:
|
||||||
- When support for passing configuration drive data to the
|
- When support for passing configuration drive data to the
|
||||||
nodes was originally created, the file was accidently named
|
nodes was originally created, the file was accidentally named
|
||||||
incorrectly as ``network_info.json``. The correct filename
|
incorrectly as ``network_info.json``. The correct filename
|
||||||
is ``network_data.json``. Both files will now be written
|
is ``network_data.json``. Both files will now be written
|
||||||
until support for writing ``network_info.json`` is removed
|
until support for writing ``network_info.json`` is removed
|
||||||
|
@ -1,6 +1,6 @@
|
|||||||
---
|
---
|
||||||
other:
|
other:
|
||||||
- A new install_dib varible has been introduced to the ironic install role
|
- A new install_dib variable has been introduced to the ironic install role
|
||||||
to control installation of disk image builder and dib-utils.
|
to control installation of disk image builder and dib-utils.
|
||||||
To maintain the previous behavior install_dib will default to
|
To maintain the previous behavior install_dib will default to
|
||||||
the value of create_image_via_dib.
|
the value of create_image_via_dib.
|
||||||
|
@ -1,5 +1,5 @@
|
|||||||
---
|
---
|
||||||
fixes:
|
fixes:
|
||||||
- |
|
- |
|
||||||
Bifrost no longers installs GRUB2 and shim on the host system, avoding
|
Bifrost no longer installs GRUB2 and shim on the host system, avoiding
|
||||||
potential issues with a local bootloader.
|
potential issues with a local bootloader.
|
||||||
|
@ -1,7 +1,7 @@
|
|||||||
---
|
---
|
||||||
features:
|
features:
|
||||||
- |
|
- |
|
||||||
Adds the use of the ``openstacksdk`` library which is superceeding the
|
Adds the use of the ``openstacksdk`` library which is superseding the
|
||||||
``shade`` library for communicating with OpenStack services using the
|
``shade`` library for communicating with OpenStack services using the
|
||||||
Ansible modules. By default this will attempted to be installed
|
Ansible modules. By default this will attempted to be installed
|
||||||
from source. Set the ``openstacksdk_source_install`` option to false
|
from source. Set the ``openstacksdk_source_install`` option to false
|
||||||
|
@ -2,5 +2,5 @@
|
|||||||
fixes:
|
fixes:
|
||||||
- |
|
- |
|
||||||
Fixes an issue where the proliantutils library version was unconstrained,
|
Fixes an issue where the proliantutils library version was unconstrained,
|
||||||
potentially resulting in installation of an incompatibile library. The
|
potentially resulting in installation of an incompatible library. The
|
||||||
version has been pinned to 2.8.x.
|
version has been pinned to 2.8.x.
|
||||||
|
@ -22,5 +22,5 @@ fixes:
|
|||||||
This has been tested on Ubuntu 14.04 LTS.
|
This has been tested on Ubuntu 14.04 LTS.
|
||||||
issues:
|
issues:
|
||||||
- Basic testing has revealed that PXE drivers may not
|
- Basic testing has revealed that PXE drivers may not
|
||||||
result in the configurtion drive being written out
|
result in the configuration drive being written out
|
||||||
with the current configuration of bifrost.
|
with the current configuration of bifrost.
|
||||||
|
@ -35,8 +35,8 @@ Usage:
|
|||||||
where:
|
where:
|
||||||
N - max number of dict elements to be defined in the <first> JSON file
|
N - max number of dict elements to be defined in the <first> JSON file
|
||||||
input - path to input JSON file
|
input - path to input JSON file
|
||||||
<first> - path to first output JSON file containig at most N entries
|
<first> - path to first output JSON file containing at most N entries
|
||||||
<second> - path to the second output JSON file containig the rest
|
<second> - path to the second output JSON file containing the rest
|
||||||
""" % sys.argv[0]
|
""" % sys.argv[0]
|
||||||
|
|
||||||
|
|
||||||
|
2
tools/vagrant_dev_env/Vagrantfile
vendored
2
tools/vagrant_dev_env/Vagrantfile
vendored
@ -45,7 +45,7 @@ Vagrant.configure(VAGRANTFILE_API_VERSION) do |config|
|
|||||||
bifrost.vm.network "public_network", bridge: 'enp3s0', dev: 'enp3s0'
|
bifrost.vm.network "public_network", bridge: 'enp3s0', dev: 'enp3s0'
|
||||||
|
|
||||||
|
|
||||||
# besure we get the entire bifrost directory tree
|
# be sure we get the entire bifrost directory tree
|
||||||
bifrost.vm.synced_folder "../../.", "/home/vagrant/bifrost", type: "rsync"
|
bifrost.vm.synced_folder "../../.", "/home/vagrant/bifrost", type: "rsync"
|
||||||
|
|
||||||
bifrost.vm.provision 'ansible' do |ansible|
|
bifrost.vm.provision 'ansible' do |ansible|
|
||||||
|
@ -6,7 +6,7 @@
|
|||||||
become: yes
|
become: yes
|
||||||
tasks:
|
tasks:
|
||||||
############################################################################
|
############################################################################
|
||||||
# Install some reqired bits into the vm
|
# Install some required bits into the vm
|
||||||
############################################################################
|
############################################################################
|
||||||
- name: Remove private default route
|
- name: Remove private default route
|
||||||
command: ip route del default dev eth0
|
command: ip route del default dev eth0
|
||||||
|
Loading…
x
Reference in New Issue
Block a user