This changes 'ansible_ssh_host' to 'ansible_host'. The 'ansible_ssh_host'
variable has been deprecated as noted here: [0].
[0] - http://docs.ansible.com/ansible/intro_inventory.html#hosts-and-groups
Change-Id: I7f405b30a90b2671ff01b49747ad9f3a9486445a
Related-Bug: #1636606
Signed-off-by: Kevin Carter <kevin.carter@rackspace.com>
With the implementation of https://review.openstack.org/388087 all
tox targets may now use upper constraints.
Change-Id: If6c0cc075bc495d259b2e344cfa1050f092770b7
Added a water mark to the role documentation
to make clear to deployers which release they're
working with.
Modified conf.py that adds watermark to the documentation according
to particular branch. If the current branch is master than
watermark is 'Pre-release' and if the current branch is
stable/<release-name> than watermark is <release-name>.
This is a combined implementation based on the following
patches:
- https://review.openstack.org/372565
- https://review.openstack.org/382637
- https://review.openstack.org/384140
Change-Id: Id0bab5c90d119bec40d1c745203d0c89ad7096f3
This change removes the use of 'ignore_errors: true' because it causes deployers
to see red output and a stacktrace, which traditionally means something is broken,
even when the failure is known to have a fall back option or be intentional. This
conversion will provide a generally cleaner interface.
It should be noted that the 'failed' filter will still function normally. Tasks
with the 'failed_when: false' option will still be marked as 'failed' in any
registered variable. This change simply makes the output look cleaner.
Change-Id: I7067a541362c826579fc7d17abf86fe9ddc89bee
Closes-Bug: #1633438
Signed-off-by: Kevin Carter <kevin.carter@rackspace.com>
Releasenote translation publishing is being prepared. 'locale_dirs'
needs to be defined in conf.py to generate translated version of the
release notes.
Note that this repository might not get translated release notes - or
no translations at all - but we add the entry here nevertheless to
prepare for it.
Change-Id: Ib9c0bf013bc56a770c63ca49e280abd7c255539d
This patch consumes the centralised Ansible test scripts
implemented in https://review.openstack.org/381853
Change-Id: I2623678bc73068a64178fafc94db124c8c62ba83
With https://review.openstack.org/381479 merged, setting this
variable in tox.ini via an extra var is unnecessary.
Change-Id: Ie2cc718173015a681209d73737a4a7b60af7216c
We want to add virtual bmc testing and a full stack test of Ironic, as a
first step lets set it up with all the integrated components.
Change-Id: If4db7ada81e3ce54a6f28db6a67e7e0b4d679cce
This patch consumes the test scripts implemented by
https://review.openstack.org/375061 to ensure that
the tests and test preparation is consistent and
more maintainable.
Change-Id: Ie01392e33b86b9b61019cc733f9f3db15f36c19d
There's no real cost to having IPA debug output on, and it helps
operators work out from a KVM what went wrong when things get
exciting. So let's just turn it on by default.
Co-Authored-By: Michael Carden <mike.carden@gmail.com>
Change-Id: I2ac525570088ea4494968666213d6f583bd5ede9
Ansible 2.1.1 introduces a regression in the way conditional
includes are handled which results in every task in the
included file being evaluated even if the condition for the
include is not met. This extends the run time significantly
for a deployment.
This patch forces all conditional includes to be dynamic.
Change-Id: I634d7ba7c7d59b21cffd3cddfdc28717bbe91a86
Related-Bug: https://github.com/ansible/ansible/issues/17687
Update the rootwrap filter config file copy task to handle
looking up rootwrap filter files using 'with_fileglob' to avoid
having to maintain the task with each addition or removal of these
files.
Change-Id: I768ba7415408c1c5dbd66e9fe57788d41587742c
Ironic in OSA is currently broken as the ironic database
user isn't created, and consequently the ironic services can't
connect to the database. It broke back in patch 91deb13.
This patch corrects the os_ironic side of the problem. Another
patch will fix the openstack-ansible side.
Change-Id: I30b04756258a3241bb2444fd98f3a6624c0d2309
Partial-Bug: #1625081
Signed-off-by: Michael Davies <michael@the-davies.net>
The volume of logs we collect in CI jobs are extensive. This
patch ensures that the logs collected are compressed to reduce
the space taken in CI systems.
PYTHONUNBUFFERED is also set to ensure that the console log
from the CI jobs is recorded in the exact order of execution.
Change-Id: I00a05dfd4774036822bc2b19f010c00bd50b31f2
Related-Bug: #1620849
In order to make it easier to differentiate between the lists of
python packages, distribution packages, downloaded packages,
package pins and other similar variables the variable names are
being changed to ensure that they have a more explicit suffix
that defines the purpose and makes the naming more consistent.
This is to facilitate a lookup plugin which will be able to look
up all the package lists and present them as a consolidated piece
of data which may be used for artifact preparation.
Change-Id: I6f61216d318ea1adbcbf544066e9c71205fa5daf
This change enables log collection within the gate so that further analysis
on gate tasks can be performed post build. This is very useful when
debugging problems.
Change-Id: I9ddf80f1c8cc22e170bb32c17ecacf79cc99120a
Signed-off-by: Kevin Carter <kevin.carter@rackspace.com>
Some Linux distributions, such as CentOS 7 and Xenial, have trouble
validating SSL certificates when using get_url with servers
that use Server Name Indication (SNI).
This patch adds those packages to the list of required packages and
uses bindep to install them in developer test environments the same
way that the gate tests install them.
Add SNI support to ops role via OS packages
Change-Id: Id98a1db916427209842c13bdaab89f4dfe2d40c2
Finish moving to new format, remove tags, fix RST titles, fix line
lengths, add link, indent example file, fresh coat of wax.
Change-Id: Ibcf3d5743c9b70d4f29ea8181d54a6c5c1974580
As per [1] all linting tests can now use upper-constraints. This patch
removes all instances of the install_command override relating to lint
testing which were needed to negate the use of upper-constraints.
[1] http://lists.openstack.org/pipermail/openstack-dev/2016-August/101474.html
Change-Id: Ic4e51f4db3933d9b6d814274930cc8580ea3ae00
This adds python packages required for SNI support to ensure that tox
installs the packages into the venv when running tests.
Change-Id: Iadfe2b027c6d9cbd89d89d9ef0d6c514aaf2d1da
The numerous tags within the role have been condensed
to two tags: ironic-install and ironic-config
These tags have been chosen as they are namespaced
and cover the two major functions of the role.
Change-Id: I1ddbf229f4a3665c2c6fc798b39882d137c536d1
The plugin repo needs to be cloned for ansible-lint to
understand ansible plugins. The commands currently
reside in tox.ini under the ansible section and are not
currently included. This commit fixes that error.
Change-Id: I75e24492d91203d9c07a57658aac917c76564994
OSLO logging currently defaults the 'use_stderr' option to True
which results duplicate logs in service daemon logs for both
upstart and systemd. To correct this issue the use_stderr
option has been set to false.
Change-Id: I601c764bcb731baeedba784787b721cccfa487e4
Closes-Bug: 1588051
Signed-off-by: Kevin Carter <kevin.carter@rackspace.com>
This patch adds a var used by the repo_build process to determine
which host group to check for members of before building the pip
packages required by this role. This provides the ability to
optimise the repo build process to only building the packages
that are required for an environment.
Depends-On: Idda16b4f382eee57c7469af898859d6d81d4eb30
Change-Id: Iedb4c0c91daa291f881a9fc00c5b8c40bb312ce3
The current method of installing the distribution packages required is
set in the tasks and cannot be changed by a deployer.
Currently the apt task always installs the latest package. This results
in unexpected binary changes when a deployer may simply be trying to
execute a configuration change.
This patch adds the ability for a deployer to change the desired state
so that the results are predictable.
Change-Id: I16a81da2b61cd69bafc59ac113fe546a748e8675
When a task is included, contains variables within the
include, and loads other tasks with conditionals that
iterate (using a lookup plugin) Ansible will spinlock for
~30 seconds per item per task. To resolve this the iteration
using the lookup plugin was removed and the task broken out
into individual tasks.
Change-Id: I2f979c950217f85340fbb22a8c9151b5605753f7
The 'docs' tox target executes the doc8 lint test which may result in
failures when testing documentation builds, but OpenStack-CI does not
execute that tox target.
In order to ensure that we catch all standard documentation syntax
errors and prevent them from merging, this patch includes the docs
target in the 'linters' chain of tests.
Fixes for any failures which result from executing this test are also
included in the patch.
Change-Id: I33eaae97d9e2bc6da6197db67d82e9aa8737e766
If multiple regions exist, keystone_authtoken should contain region_name.
This prevents the service from authenticating across regions.
Change-Id: I3e08385d91f5f913209dc944875c843828da3c2b
The openstack_hosts role is not used in any of the test playbooks
or required by dependent roles and can be removed from the test
role requirements.
Change-Id: I4182c169c1fa01651bf2187182961187f23d27d4
