Merge "Fill DNS name for haproxy certificates"

Merge "Fix value of ctlplane fqdn fact"

Fix value of ctlplane fqdn fact

This fact was being retrieving the value of the hostname for the management
network. We should instead be using a value set explicitly in t-h-t.

Depends-On: Idb3ca22ac136691b0bff6f94524d133a4fa10617
Change-Id: I6fcf7c7853071a9f3377aec475308bc8d10d5b33
Related-Bug: #1621742

Fill DNS name for haproxy certificates

This sets the subject alt name field for the certificates we
auto-generate, which will remove the security warnings we constantly
see in the undercloud. This is the proper way to set certificates,
since the usage of the CN as a replacement for the subjectAltName is
being deprecated (very slowly).

Change-Id: I475cbffd47425e850902838eec06bf461df2acd0
Closes-Bug: #1622446

Merge "Added Contrail Composable Services"

Merge "Add Ceph RGW profile"

Merge "Add class to write overcloud VIPs into /etc/hosts"

gnocchi: move statsd and metricd at step5

Gnocchi statsd and metricd require Keystone resources to be in place
when using Swift backend, because those services will try to access to
Swift containers.

To do so, we want to move the service start at step 5 instead of 4 and
also require Keystone resource to be managed before starting the
services.

Change-Id: Ie5bc1481a8700c7cd080a76d0978146a84825767
Closes-Bug: #1621164

Merge "Manage Redis VIP when deploying with keepalived"

Merge "Include heat base profile outside of step"

Merge "Add Ceph RGW listener to HAProxy"

Merge "Convert ringbuilder to build devices array"

Add class to write overcloud VIPs into /etc/hosts

This will be useful to start using FQDNs instead of IPs if we don't
have a DNS set-up. This will effectively grab a map of hosts to add
and create them with puppet's host resource.

Change-Id: I1bdb2701dfb3e7ef072e674c9882d3be5af7296c

Merge "Map ctlplane network to management fqdn"

Merge "Make fqdn fact names match service net map"

Merge "Ironic: update included drivers"

Merge "Ironic: manage PXE environment for conductors"

Map ctlplane network to management fqdn

We don't really expose a management network, but a ctlplane one. So
when someone requests the ctlplane fqdn, we'll give the management
one. Since that's what we actually fill in the hosts.

Change-Id: Iede5a06d73206f98f283729f1879484f09d4012a

Make fqdn fact names match service net map

This will make it easier to use and consume from t-h-t, making their
use more dynamic. This is still safe to do, since these are not used
by anything yet.

Change-Id: I4d603938c568320d4ae1cc7a396070ad4ac62132

Merge "updates to fluentd support"

Ironic: update included drivers

Configure drac, ilo and ssh drivers out of box.
Remove deprecated ironic::drivers::deploy.

This change brings the default driver list closer to one of undercloud.

Change-Id: I8b9a136a0ff22916d7c468bbb0df7248bc35a5c2
Partially-Implements: blueprint ironic-integration

Merge "Make service profiles default to rabbitmq_node_ips"

Merge "Plumgrid helper"

Merge "Opencontrail vrouter profile"

Plumgrid helper

Extra settings that need to be applied for plumgrid where compute
nodes are running.

Change-Id: Ided5483f0f36f0efd5a09112832d07f028a2a7f9

Manage Redis VIP when deploying with keepalived

Previously we weren't creating Redis VIP in keepalived, causing Redis to
be unusable in non-HA deployments.

Change-Id: I0bb37f6fb3eed022288b2dcfc7a88e8ff88a7ace
Related-Bug: #1618510

Convert ringbuilder to build devices array

Currently we have some hard-coded mangling in t-h-t but we
instead need to build the array based on the nodes running swift
storage, combined with the SwiftRawDisks parameter.

This will enable running SwiftStorage on nodes other than Controller
and SwiftStorage roles, and is required for custom-roles due to
the hard-coded stuff in the role templates and overcloud.yaml

Change-Id: I11deed1df712ecccf85d36a75b3bd2e9d226af36
Partially-Implements: blueprint custom-roles

Merge "Create memcache_servers list in swift proxy profile"

Make service profiles default to rabbitmq_node_ips

Instead of hard-coded yaml aliases in t-h-t, make each service
profile that requires rabbit default to the list of rabbit ips.

Note this could still be extended in future to e.g enable per
service rabbit clusters, but the default is to lookup the
hiera which should be logically equivalent to current t-h-t.

Change-Id: Ie53c93456529420588eb1927703ea91b54095d87
Partially-Implements: blueprint custom-roles

Release puppet-tripleo 5.0.0

This is the first tag for newton and OpenStack Infra will create a
tarball from it.

Change-Id: I7a98743bbbd1879426790a9821cadfa7ecbd5d22

Merge "midonet: update neutron parameter"

Merge "Write restart flags to restart services only when necessary"

Create memcache_servers list in swift proxy profile

Instead of mangling this list in t-h-t, generate the list derived
from memcached_node_ips, which is now always set when memcached
is deployed, regardless of the role.

Note the port default is hard-coded as this is already hard-coded
(in two places) in t-h-t, but we can override it if this changes
in future.

We need this to remove the swift specific stuff out of overcloud.yaml
to enable custom-roles.

Change-Id: Ic8872e5e51732874ca5b93bff5efd3e7ed75bc31
Partially-Implements: blueprint custom-roles

Merge "Default haproxy listeners activation on hiera service _enabled"

Add Ceph RGW profile

This patch adds tripleo::profile::base::ceph::rgw

Co-Authored-By: Giulio Fidente <gfidente@redhat.com>
Change-Id: Ib69b8171321284bb64e348d45b7eea2f00f8d506

Ensure iscsi_ip_address is in brackets if IPv6

Change-Id: I787becce343b38e6c27c9a1b937b47c0aefb034d
Related-Bug: 1618930

midonet: update neutron parameter

Update neutron parameter to lookup the right variable in Hiera.

Change-Id: I7b93e8d308f2ff2fb3a2083af75140dfa62b3ad8

Merge "Reload HAProxy on refresh"

Add Ceph RGW listener to HAProxy

Shares the same (ssl)port with Swift Proxy

Change-Id: I2e1de1a3fa6ad62895a1e972e43858f23c08bbea

Default haproxy listeners activation on hiera service _enabled

Change-Id: I5c620ba717f782b39c599aff24b4ac56fb695a04

Added Contrail Composable Services

changed parameter order

for tripleo heat templates see https://review.openstack.org/#/c/346799/

Implements: blueprint contrail-services

Change-Id: I8de63b6e21f8fdf3c2fd13bf5475cce4a85311d6

Ironic: manage PXE environment for conductors

Change-Id: Icb9633134114041bbd497e7652482dd5d34c9327
Depends-On: I8b83eff694316755e4dd2dbcde7b569472893bc5
Partially-Implements: blueprint ironic-integration

Merge "Defaults to ceph_mon_node_ips and ceph_mon_node_names"

Merge "Handle galera_node_names being an array"

Write restart flags to restart services only when necessary

Write restart flag file for services managed by Pacemaker into
/var/lib/tripleo/pacemaker-restarts directory. The name of the file must
match the name of the clone resource defined in pacemaker. The
post-puppet restart script will restart each service having a restart
flag file and remove those files.

This approach focuses on $pacemaker_master only (we don't want to
restart the pacemaker services 3 times when we have 3 controllers), so
it relies on the assumption that we're making the matching config
changes across the pacemaker nodes.

Change-Id: I6369ab0c82dbf3c8f21043f8aa9ab810744ddc12

Defaults to ceph_mon_node_ips and ceph_mon_node_names

Uses the ceph_mon_node_ips and _names from all_nodes.yaml which
are formatted as arrays.

Also deprecates ceph_ipv6 parameter as will determine if ipv6
using is_ipv6_address.

Change-Id: I481dd2cd2cde7f1491080e6d9c7dcb7047c22de1

Merge "Add ML2/OVN support for neutron profile"

Merge "Configure the numvfs for SRIOV interfaces"

Merge "Removing WARNING: line has more than 140 characters in puppet-tripleo profiles"

Merge "Configure keystone endpoints in service profile"

Configure the numvfs for SRIOV interfaces

This patch shall create VFs via the PCI SYS interface.
Default value : $::os_service_default
Sample Format : ['eth0:4','eth2:128']
For values as in sample format, the sriov_numvfs config files
for eth0 and eth2 will have the values 4 and 128 respectively
The SR-IOV numvfs configuration shall be persisted in /sbin/ifup-local
so that, during the bootup of the compute nodes, the numvfs
configuration will be restored.

Change-Id: I7450b904475bdf46498d9af633416b3eba12f761
Implements: blueprint tripleo-sriov
Signed-off-by: karthik s <ksundara@redhat.com>

Reload HAProxy on refresh

If the configuration for the HAProxy class or any of the frontend
endpoints has changed, we explicitly execute a reload on HA setups.
This is useful since on updates of HA setups we set pacemaker on
maintenance mode, and thus we are unable to refresh HAProxy's
configuration; The aforementioned detail is problematic, since some
puppet configurations rely on HAProxy taking into account the
configuration.  An example of this is changing a port in the endpoint
map or enabling SSL.

Change-Id: I7f26257fb43146afebca928f5498ee2174178063

Add ML2/OVN support for neutron profile

Partially-Implements: blueprint refactor-puppet-manifests
Depends-on: https://review.openstack.org/#/c/340343

Add OVN plugin for non-ha profiles of neutron

Change-Id: I3850a8b5584eaf14a656b547f6a47008a2d1289e

Handle galera_node_names being an array

Prepare the pacemaker mysql manifest that galera_node_names will be an
array. Stay backwards compatible to handle comma-delimited-string too
and avoid a chicken-and-egg patch problem between t-h-t and
puppet-tripleo.

Change-Id: Ia0d9d59728c8771974bfbc486f4929b99a38e4fb
Partially-Implements: blueprint custom-roles

Merge "Add Manila CephFS backend to manila class"

Merge "Remove the ceilo wsgi transient flag"

Merge "UI profile for tripleo"

Remove the ceilo wsgi transient flag

This was a temporary flag until the tht changes got merged.
Now that the tht changes are merged we can remove this as
params are set for wsgi classes by tht.

Depends-On: I728d0980f7742aa390f11a0f0b8598d1299e2886

Change-Id: I93ea8ebb9053f269de8974880bd33d1b3e487f96

Merge "Adds OpenDaylight"

Merge "Expose enable_combination_alarm setting"

UI profile for tripleo

The new tripleo::profile::base::ui profile installs the
openstack-tripleo-ui RPM package and setup CORS for required services.

Change-Id: Ib9d8643da3f51171495fcb8b64d1a7ca86cd66bc

Configure keystone endpoints in service profile

This commit enables the configuration of the service users and
keystone endpoints in the keystone profile. Since with the
composable services work, we can't assure that the APIs will be in
the same node as keystone, this needs to be done from the keystone
profile.

Depends-On: I62273f403838893602816204d9bc50d516c0057f
Change-Id: I36e1c478e7c92be61da6a0d710e9025d4d354072

Merge "SR-IOV nic agent changes in line with composable roles."

Add Manila CephFS backend to manila class

Change-Id: Idaad75238a2884fe82b1e5fce3ed910d866b98a2

updates to fluentd support

This change contains modifications to the fluentd module to support
https://review.openstack.org/353506.

bp tripleo-opstools-centralized-logging
Change-Id: I2e1abe4d8c8359e56ff626255ee50c9cacca1940

Merge "Add manifests to inject and trust CA certificates"

Revert "Pin puppetlabs_spec_helper to 1.1.1"

This reverts commit 8f8860c0343b49b50b3d0259b9c4a859f5d81cc3.

Change-Id: I456c3a46a16a904042b5af6c5fbe347912d1bf30

Pin puppetlabs_spec_helper to 1.1.1

Version 1.2.0 of puppetlabs_spec_helper pulled in a version gem with
ruby 2.2 requirements.
TripleO CI has ruby 2.0 so we need to pin the gem.

Change-Id: Idc3959e16aa329c1dfc335b766f52cd9dbde5de5

Expose enable_combination_alarm setting

Combination alarms are deprecated in Newton and hence turned off
by default. This allows users still using this type of alarms to
flip the flag and use until they migrate themselves to composite
alarms.

Change-Id: I9220b7d020dc8ed45dd6ca83ea9647efd67ea648

Merge "Move ceilometer api to run under apache wsgi"

Add manifests to inject and trust CA certificates

Change-Id: I84273b4cd6576a63fa78dc93ad6b077dd2a780c7

Merge "Clean up unused params from aodh profiles"

Merge "Configure galera-monitor on all controller nodes"

Merge "Allow use of raw disks for Swift storage role"

Move ceilometer api to run under apache wsgi

Change-Id: If3feb859b527d08e10c124b5ad2f7f4b1f19156a

Clean up unused params from aodh profiles

Change-Id: I7c89fcef8cdd0e73ef8c5d615bb55db177abd375

Merge "Fix deployment steps for AODH profiles"

Configure galera-monitor on all controller nodes

When we implemented the galera composable role we accidentally moved the
xinetd.d monitor service on the bootstrap node only. This meant that
haproxy believed that galera was down on the non-bootstrap nodes. A
shutdown of the bootstrap node meant that galera was effectively down
because haproxy would refuse to redirect the traffic to the
non-bootstrap node. Fix this by creating the
/etc/xinetd.d/galera-monitor on all controller nodes.

Change-Id: Ib5a06b3abbc32182476c2b0c81eb77a12821ad6b

Fix deployment steps for AODH profiles

The services in the AODH profiles were
limited to be executed only in the
bootstrap master node.

Change-Id: I85cfa97f0e5e9a70e86e51d8188a2abf86e2ec1c

Merge "Add cinder-backup profiles"

Merge "Align hiera keys with service names"

Defaults empty hash to kernel_modules and sysctl_settings

Set empty hash by default to kernel_modules and sysctl_settings so catalog
won't fail if the parameters are not set in Hiera.

Change-Id: I24ab535b01e2724af457d39c03cd990c574ef0aa

Adds OpenDaylight

Partially-Implements: blueprint opendaylight-integration

Note this patch only adds support for a single ODL instance.

 - neutron/opendaylight.pp handles installing ODL to control nodes
 - ml2/opendaylight.pp handles configuring ML2 to work with ODL
 - ovs/opendaylight.pp handles configuring OVS to connect to ODL

Change-Id: I666dc0874f1d11a72a62d796f4f6d41f7aa87a3f
Signed-off-by: Tim Rozet <trozet@redhat.com>

Include heat base profile outside of step

The base profile can already take the steps into account. so it's not
necessary to include it inside the if statement.

Change-Id: I7b22063db7ff4807242cc0e353dccc34be848b8d

Removing WARNING: line has more than 140 characters in puppet-tripleo profiles

Some lint checks are returning:
WARNING: line has more than 140 characters in puppet-tripleo profiles

This patch will remove those warnings by adding \'s

Change-Id: I19b56c93db82948fb0498a4c9851b522c81946f8

Merge "Handle redirects for keystone"

Align hiera keys with service names

These hiera keys aren't aligned with the service names, which
will be required for composable generation of the ip lists
per service.

Change-Id: I423b544df174254ac511b906b0c570e701678022
Depends-On: I7febf28bf409e25e8e5961ab551b6d56bb11e0c6
Partially-Implements: blueprint custom-roles

Add cinder-backup profiles

Adds a Cinder backup profile for Cinder backup service activation
(to be used in https://review.openstack.org/#/c/304563).

Cinder backup uses Swift as a default.

Change-Id: Ib1dfe52b83ab01819fc669312967950e75d8ddf1
Co-Authored-By: Jon Bernard <jobernar@redhat.com>
Co-Authored-By: Boris Kreitchman <bkreitch@gmail.com>

Handle redirects for keystone

If keystone sends a redirect and we have TLS enabled, we need to
modify the response in order to indicate https.

Change-Id: Icd61f527473bfe5153e058e94f9ed141cf13812d

Merge "Fix use of bootstrap_node in cinder base profile"

Merge "Fix mistral sync_db profile steps"

Merge "Only db-sync sahara is included in the bootstrapnode"

Fix mistral sync_db profile steps

The services in the mistral profiles were
limited to be executed only in the
bootstrap master node.

Change-Id: Iae42ffa37c4c9b1e070b7c3753e04c45bb97703f

Only db-sync sahara is included in the bootstrapnode

Currently, sahara's db-sync is run on every node. We should instead
run it if we're on the bootstrap node as race conditions may occur
and these will cause issues.

Change-Id: I5db281caae17402ff198cb4cd46304338d98c52e

Use modern auth options for Neutron

auth plugins should be used instead of deprecated admin_* options

Change-Id: I80da7ce0a3d11859dd9f2ae02cd9761b9b28ac27
Depends-on: I3018932a106df562e94067e037b3bc862be97b51

Fix use of bootstrap_node in cinder base profile

By inspecting bootstrap_nodeid in cinder base profile we can
set sync_db appropriately and not always default to true.

Change-Id: I2484b1d70a17436c0d8eab9ea8df927d57783784

Fix parameters and headers inconsistency in the puppet manifests.

As we are staring to manually check overcloud services
the first step is to check that the puppet profiles
are all aligned.

Changes applied:

  No logic added or removed in this submission.

  Removed unused parameters.

  Align header comments structure.

  All profiles parameters sorted following:
  "Mandatory params first sorted alphabetically
  then optional params sorted alphabetically."

Note: Following submissions will check pacemaker,
cinder, mistral and redis services in the base profiles
as some of them has the $pacemaker_master parameter
defaulted to true.

Change-Id: I2f91c3f6baa33f74b5625789eec83233179a9655

Merge "Convert ceph_pools into a hash type"

Convert ceph_pools into a hash type

By using create_resources we can create ceph::pool resources from
a hash. This will make it possible to define custom attributes
for the single pool from the heat templates.

Change-Id: I38978f0f3119e4ab7dd45021e598253cb066cb5a

New composable service: Sensu

Depends-On: Ic095cd1248cc2ffcef363893b20ff57f7befd6fc
Change-Id: Ia09d14fa9db07595d2e904bd9918e5645ea590c7