GitLab

Change-Id: Id88acc28dc1e7b0f1bd5811d7d0362bac958a63b

The systemd unit 'TimeoutSec' value which controls the time
between sending a SIGTERM signal and a SIGKILL signal when
stopping or restarting the service has been reduced from 300
seconds to 120 seconds. This provides 2 minutes for long-lived
sessions to drain while preventing new ones from starting
before a restart or a stop.

The 'RestartSec' value which controls the time between the
service stop and start when restarting has been reduced from
150 seconds to 2 seconds to make the restart happen faster.

These values can be adjusted by using the *_init_config_overrides
variables which use the config_template task to change template
defaults.

Change-Id: Id721b89bd37df91192f48743ea3c32a305022a9f

'ansible_become' defined as a host_var or in the inventory for a host
has a higher precedence than 'become' defined within a playbook.

In change Id5b76a87809f03951c954fc3d752419a673403f7, 'become' was
defined explicitly for each play running against localhost so
'ansible_become' should be removed from its host_vars.

Change-Id: I0a3019ed7f4e151625682d009e5359378152592d

Change-Id: Ifdb28ed5dc5709e62f96e16eda4ca453a85b7777

Updated the template for ironic.conf to enable configuration of
the swift url endpoint type within ironic.conf

Change-Id: I67f64167914d6e3e9fddb57e5e38dc03757ae5c7

This creates a specific slice which all OpenStack services will operate
from. By creating an independent slice these components will be governed
away from the system slice allowing us to better optimise resource
consumption.

See the following for more information on slices:

* https://www.freedesktop.org/software/systemd/man/systemd.slice.html

See for following for more information on resource controls:

* https://www.freedesktop.org/software/systemd/man/systemd.resource-control.html



Tools like ``systemd-cgtop`` and ``systemd-cgls`` will now give us
insight into specific processes, process groups, and resouce consumption
in ways that we've not had access to before. To enable some of this reporting
the accounting options have been added to the [Service] section of the unit
file.

Change-Id: Id82169f2f500045e0ff7dc348748400a4b116fe0
Signed-off-by: Kevin Carter <kevin.carter@rackspace.com>

The 'provisioning_network_uuid' and 'cleaning_network_uuid' options have
been deprecated for 'provisioning_network' and 'cleaning_network'
respectively.

These options now allow either a network UUID or name. The tasks to
determine a UUID from a given network name should no longer be
necessary, but if a deployer provides a UUID prioritize that over the
name within the ironic.conf template.

Change-Id: I85f197e56ced6a73dd470c0625b6d9b5958f5159

Package pyOpenSSL is obsoleted by python2-pyOpenSSL.

Change-Id: Idc2f4ee0175bad718aa99792acf9aa8634e61ed9

Users can configure the number of worker processes. However when
it's not specified the calculated number of workers can get too
large on hosts with a large number of CPUs.

Change-Id: Ieb1afbd5c872cc4a991b51287b57404b8932fafa

Change-Id: I7334675111244e174a6006ecad5bf93151503a3f

Change-Id: I6a232eb2dac43825bec73f8a7b7505142989521f

- Install python2 for Ubuntu 16.04 and CentOS 7 before executing test
  script.

Change-Id: I318c250e64b69d29bdec0157482a5c57c671cedf

Change-Id: I5eb5ea52196a0bd68da6e9060b1125a91ebef2d3

Change-Id: I73bd0a45bdb8f41119dac8d3fc05d1c96b492f6d

Trivialfix

Change-Id: Ieeaa60e79b9e3f77ca87ff53c90ad7b44670c4f0

Change-Id: I76ddba429ec37e5d87ac8003308a747ed8c05ffc

This change adds support for the oneview drivers (agent, iscsi).

Note that changes on the ironic installation will occur only when
the oneview drivers are being used (agent_pxe_oneview or
iscsi_pxe_oneview are in the ironic_openstack_driver_list). This
means that this patch should not change anything on the the default
ironic installation (using agent_ipmitool driver).

Change-Id: I969df888c6a8b68e7a1a0643b46eee4b546ec13c

Change-Id: I189fc52c52f013e48169576000c9d93002ac6aa4

This patch ensures that all directories under `/openstack/venvs`
are created with `0755` permissions by default. This prevents
permission denied errors when running certain commands from the
virtual environment.

Change-Id: Id93d49a4bd6ce2a056d41aea54ec9299d9d32b5c

Some tags were not applied on the highest level and forced duplication.
We apply here our practices to have role_name-(config|install) [1].

[1]: http://docs.openstack.org/developer/openstack-ansible/developer-docs/contribute.html#ansible-style-guide

Change-Id: I23c36c5ea888a75ce7deddf272bd222af3b9e9f5

This patch fixes a few minor issues:
* The ironic_service_user_name was not being passed to Nova
  - this value was fixed in Nova #1647253
* The br-flat network seems to already existing on some gate nodes, this
has been changed to named br-vlan

Change-Id: I12f68a1cda39a3d615fbcf6ad62694faca6d09f4

Fixes the ability to deploy a venv in cases where:
1) developer_mode is not enabled
2) A cached venv is not downloaded from the repo server

Additional cleanup to the developer_mode venv deployment
logic is implemented by adding a *_venv_download var
which is used to decouple developer_mode from the
cached venv extraction process so that a deployer
can force venv builds in-place (disable cached
venv usage) without enabling developer mode
constraints.

Change-Id: Ie16f861b9abdc5887365bb58c9eec427f55cb177

The run_tests.sh script fails when it is run multiple times on CentOS.
The `bindep` run returns an empty list of packages and then `yum`
exits with an error since no packages were provided to install.

This patch checks the length of the `bindep` output and skips the `yum`
installation when the package list is empty.

The patch also cleans up some of the old cruft left over from previous
scripts and avoids repetition.

Change-Id: Idddbe91d736198fa6c98b81a125a818e68657b18
Signed-off-by: Major Hayden <major@mhtx.net>

The ironic_openstack_driver_list key is being defined twice. This
results in a warning during task execution and the adoption of the
last defined value.

Removing the first defined value here to maintain current behavior.
If the desired behavior is the opposite, I will reinstate the first
and delete the second.

Change-Id: I0f7526230c5192611daefb1934b627aef463f053

Change-Id: I12359451e4116d1ff2b522f7fd0c43dd6fc64053
Implements: blueprint remove-pki

Change-Id: I0c923830ce1d1d20c7da483be8166873de1f839e

The api_url value is wrong and causes gate failures now that the value
is tested by the ironic server when it starts up.

Change-Id: I7d63135a74b6472a284495f04d7c04dc553f53ad

Change-Id: I35f90d708cda58c5b09f17e5596cf226d1b16605
Implements: blueprint trusty-removal

Change-Id: I161bd21339332e22d702ae0ca727d313d15a3c67

This patch adds a task to copy ldlinux.c32 to the tftp directory
when on ubuntu 16.04. Also, changed the copy tasks to a unique
task making use of the copy module with 'remote_src: True'.

Change-Id: I4a71c198ad11d293e15f8a80bc1dcd0c6e6aa479

This patch adds the ability to specify a cleaning and provisioning
network for Ironic. If none are specified then Ironic continues to
function as it does now.

Ironic role will calculate the UUID of the neutron network assuming a
network name is provided.

Additionally, this is added to testing by configuring a network to add
with the network-name.

Change-Id: I9be6f351c0da292ac8b861d2168e73d1861e1603

Some hacking have been removed,so we can delete them.
More details:
    https://github.com/openstack-dev/hacking/blob/master/setup.cfg

Change-Id: Ic8685a4f595d981f4b60a6f6a0ac66f3a91a08be

Post install tasks in os_ironic role don't quote password
values, having shell special characters in password is
resulting in failed runs.

Closes-Bug: 1647248
Change-Id: If7fefb112ffdee8d9ee162b1f0bc0bbe10528e3e

Change-Id: I1d3ce3530f679348cc31f3b64cf65cba581b94ad

Currently when clicking on the small bug in the roles'
docs, we land on openstack-manuals launchpad.

This should solve it.

Change-Id: I99d8c4a1dc7d74342d5a64e90b767c96833a7b26