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
Often it is beneficial to use the functionality provided by `use_cirros`
and `cirros_deploy_image_upstream_url` to download and test with images
of other OS distributions. This patch changes the variable names to
reflect that use case.
`use_cirros` -> `download_custom_deploy_image`
`cirros_deploy_image_upstream_url` -> `custom_deploy_image_upstream_url`
The functionality of the old variables is retained for backwards
compatability.
Co-Authored-By: Alex Welsh <alex@stackhpc.com>
Change-Id: I8b3a973bd5ef19333af563c170b2a28fbbfb5c76
Also reduce verbosity of ansible-lint as we don't really need
so much DEBUG output, errors may get lost in there.
Change-Id: I5f5a3ffd7f69275685695992732ed98549c6e5fa
It's dangerous to allow a regular user to write into the source code of
the services. The repositories are still writable, so that the
development workflow can still use the ``--develop`` flag to make
modifications.
While here, remove the horribly outdated and broken ANSIBLE_INSTALL_ROOT.
Change-Id: Id2e25dd57668d24a11dc2cd16eea2b607b7a3f16
No code changes required, but the linters job needed fixing since now
it tries to automagically load the collections in playbooks/collections
and fails with "relative resource paths not supported".
Change-Id: Ifa99a4bccc52f095c057a082fbe3d15a1633de9f
This change removes support for RabbitMQ. It's no longer needed by any
services, we've agreed to find an alternative for multi-node inspector,
and neutron will likely stop needing it with the switch to OVN (and
that is assuming we ever get to supporting neutron, which is unlikely).
Change-Id: I2dbefe3482addfb9cd65c34d8d1007a501531399
The implementation is copy-pasted with some tweaks from project-config
CI jobs. A list of exclusions is currently huge and should be reduced.
Change-Id: I7811884caa5d6c46e42bc4d1638ccaa4d17910b7
The VENV env variable is defined but not used, if we don't specify
the correct path for the ansible-playbook binary, the task will
fail.
Change-Id: I425445fc3c716d085ba9c93fa253b649ee7c0438
Under some failure conditions, the status code is not returned.
Also skip the agent/os build steps for install inside of vagrant
which can add a substantial amount of time to the build process.
And minor revision to the test-bifrost.yaml conditional as the
pattern was traditionally "is defined", but when running the test
scripts, most things are defined, so it was just a side effect of
changes being purely tested in CI over time.
Change-Id: Icf132b71b243f9fce9d1c2afca656299befcfabc
Things get a little confusing for ansible and we changed the
main scripting to somewhat expect this when explicitly
executing the install playbook, as is in test-bifrost.sh.
Change-Id: I2bfea0e383387bf3c78359ffa8b98e4965ad3709
Updated vagrant steps to install vagrant with a generic image
and the libvirt provider and Ubuntu Bionic.
Also attempts to update the configuration such that systemd does
not try to continually re-assert the eth0 network IP address which
allows vagrant users to reach the same failure points in setup as
OpenStack CI.
Change-Id: Id8705dc80137ec27870017b400fd3a6ee5007ebf
The ssh drivers are gone, long live the ssh drivers~
Removing from the optional drivers list entry for pxe
drivers, removed from tests, examples, role documentation
and the helper script that is not used for testing, since
it is completely outdated at this point since we've been
using the role for quite a long time.
Change-Id: Ibafc4db4b326aafef80660be35ecd1dbad57cb51
It sets the correct working dir when calling scripts/env-setup.sh.
It also stops sourcing /opt/stack/ansible/hacking/env-setup as it is
no longer needed.
It currently conforms with README.rst.
Change-Id: If921b6b0df98d9e4e725b79168fbb3f500939362
Signed-off-by: Cédric Ollivier <ollivier.cedric@gmail.com>
Due to the differing performance profile of gate machines,
and that the settings often need to be chnaged in order
to mirror gate testing, it may be good to change the VM
defaults.
Change-Id: I6fc42cb279c0e930804c4f07cc820858f606cc6c
This change helps with the remote problem where ansible is installed
on a machine using the same script by User A and User B is also
trying to use ansible using the same method. The change lets each
user have his own copy of ansible installed in a directory of
his/her choosing.
Change-Id: I8fddaaa8cad291da840ac44c4b1e9a7a93f92aee
Signed-off-by: Ganesh Maharaj Mahalingam <ganesh.mahalingam@intel.com>
Partial-Bug: #1589672
Add two README files to avoid any confusion about the intent of the
scripts/ and tools/ directories.
Change-Id: I50d94088649ec6b6d7f0799f40760e03b5aad371
Fixes syntax for get_url module in ansible playbook vagrant.yml
which was causing the vagrant up command to fail.
Change-Id: Iffb0b17c8f808b33538b06d21c52ca603f018a9c
Closes-Bug: 1521750
Ansible is scheduled to remove the 'sudo' module
and replace it with 'become'. Switch to using this
now since it is already supported in 1.9.
Closes-Bug: #1504535
Change-Id: Ie63816e36a95c86d54fea5f50325522b41597047
distro setuptools is old and should never be touched if you're going
to install things from pip. easy_install is insecure and open to
MITM attacks.
Downloading get-pip.py over https and running it will install modern
pip and setuptools on the system and is the preferred method for
installing pip in all cases.
Change-Id: I1bf88f018806142ca639f3c183598b318636e61d
Added columns 13 thru 16 which defines the ipmi_target*, ipmi_transit*
and ironic driver fields which in this case default to empty and agent_ssh for
the driver.
Change-Id: I32feb85ae5dc2343aa1b0a3cca92ee4697fd2c05