Merge "Reload HAProxy on refresh"

Merge "Defaults to ceph_mon_node_ips and ceph_mon_node_names"

Merge "Handle galera_node_names being an array"

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

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>

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

Merge "Add passing of X-Forwarded-Proto to Glance API endpoint"

Merge "Add zaqar API endpoint in HAProxy"

Merge "Remove keystone PKI cert generation"

Merge "Implement aodh profiles"

Remove keystone PKI cert generation

We don't currently offer any parameter interface to enable
PKI certs, and these have all been deprecated by keystone, so
remove them.

Change-Id: I8232262b928c91dcde7bea2f23fa2a7c2660719e

Merge "Fix mistral profile steps"

Merge "Configure Ceph/OSD after Ceph/Mon when on same node"

Add passing of X-Forwarded-Proto to Glance API endpoint

Glance supports the http_proxy_to_wsgi middlware, and it was recently
enabled in the overcloud [1]. However, for it to work properly, we
need to add the X-Forwarded-Proto header which was missing from the
HAProxy configuration.

[1] I4a8f7fc079ca93c50aa0ef7b0548dc64f6c5cfa0

Change-Id: I82e2db1145b0476cec27676fdfbb97e86cbd8182

Add zaqar API endpoint in HAProxy

This will enable us to terminate SSL connections for Zaqar's API.

Change-Id: If75e2947a2dca95b3e53e1b1ffd93f36fc7fb1cc

Merge "Next generation HA architecture work"

Implement aodh profiles

Change-Id: Ie48a123cc5bc402aee635a5daf118b158c6f3b6a
Implements: blueprint refactor-puppet-manifests

Merge "Add base nova-ironic profile"

Merge "Fix Ironic dbsync ordering"

Merge "Remove unnecessary certmonger manifest"

Allow use of raw disks for Swift storage role

Adds a call to swift::storage::disks to make it partition additional
disks to use with Swift.

Depends-On: I3692b17988f8eacb7b8a7df1006a7fe24f587065
Change-Id: Iac75cf2244339fae444be5859b7583e80bafc4e2

Fix mistral profile steps

Fixing Mistral profile
when deploying the service.

Change-Id: I942d419be951651e305d01460f394870c30a9878

Remove unused parameter in sahara

Remove unused parameter in sahara

Change-Id: I46c033b410ab850289b798ee93990b6fb10c80ea

Merge "Fixup nit in manila pacemaker profile, duplicate variable"

Merge "Run local CA trust before haproxy deployment"

Fixup nit in manila pacemaker profile, duplicate variable

See discussion at https://review.openstack.org/#/c/342961/8

Change-Id: I571b65a5402c1028418476a573ebeb9450ed00c9

Remove unnecessary certmonger manifest

The original intent of this manifest was to set the defaults for the
CA and some other values when requesting certificates via certmonger.
However, this approach doesn't work and in the undercloud we ended up
doing this via hieradata instead. So this file is not used.

Change-Id: I552d504091acc0d5588e30d14bc2fc85d7b7ecea

Next generation HA architecture work

This change moves the cinder-volume/cinder-scheduler constraints in the
cinder-scheduler profile as these can't be applied by the cinder-volume
service when cinder-scheduler isn't managed by Pacemaker.

Blueprint:
https://blueprints.launchpad.net/tripleo/+spec/ha-lightweight-architecture

Change-Id: I5e7585c08675d8a4bd071523b94210d325d79b59
Implements: blueprint ha-lightweight-architecture
Co-Author: cmsj@tenshu.net

New composable service: FluentD

Change-Id: I278dfee01ed24713dec29aae55d2c7bdcb7e74bc

Run local CA trust before haproxy deployment

Before haproxy tries to use the TLS certificates it should already
trust the CA. So it's necessary for the local CA-related manifest to
notify the ::tripleo::haproxy class.

This works for newly set deployments. deployments that have already
ran the ca-trust section will already trust the CA and thus won't
need that part.

Change-Id: I32ded4e33abffd51f220fb8a7dc6263aace72acd

Move constraints to their respective services

The openstack-core-then-httpd constraint needs to live in the apache
pacemaker manifest and not in the main controller manifest file.
The same goes for those specific vsm/cisco neutron resources.

Change-Id: I2041d4d163f051427b62eec07b8345ad7006cc1d

Merge "Move nova constraints from controller manifest to each service"

Merge "Remove global openstack-core resource"

Merge "Create role for the fake openstack-core resource"

Move nova constraints from controller manifest to each service

Currently we are still creating all the pacemaker constraints for nova
in the main overcloud_controller_pacemaker.pp manifest file.

Let's move those to each role where they belong. Note that given
that a constraint depends on two separate pacemaker resources it is
a bit arbitrary in which file they end up being (the one of the first
resource or the second one).

Change-Id: I96a3a313d15fac820b020feae0568437c2cbade3

Remove global openstack-core resource

The openstack-core resource is not needed by the NG Pacemaker
architecture. It was moved into an isolated role by [1] so that
it could optionally be enabled when wanting the older architecture.

This submission removes the old openstack-core global resource.

1. I74a62973146c0261385ecf5fd3d06db51e079caa

Change-Id: I16a786ce167c57848551c7245f4344c382c55b3d

Create role for the fake openstack-core resource

Change-Id: I74a62973146c0261385ecf5fd3d06db51e079caa

profile/base/nova: declare nova class and configure cache correctly.

Nova {} workaround is not working correctly, we need to merge this patch
so we can move out ::nova from THT completely.

Also we need to use nova::cache to configure memcached parameters.

Co-Authorized-By: Giulio Fidente <gfidente@redhat.com>
Co-Authorized-By: Sven Anderson <sven@redhat.com>
Co-Authorized-By: Emilien Macchi <emilien@redhat.com>
Depends-On: I52d5badb9960124bb8fcb54983db2853c4185e77
Depends-On: I3e400a5f64b85f0d374fc02cc5e4080d19d0f2e4
Depends-On: Iee5f8015cbf40ca0e9a435a7de919ebdb74cf93f

Change-Id: Ie4e72e765f6a8ade48d4b2b766f067872554d1a2

Unit tests for HAproxy dual stack

Change-Id: I6a959609523bd7fa681cd86522a56fff7c92352b

Merge "Implement tripleo::network::os_net_config class"

Merge "Add firewall::service_rules define"

Merge "Add base constraint so gnocchi metricd is tied to core-clone"

Merge "Add swift ringbuilder profile"

Implement tripleo::network::os_net_config class

This class will be used on the undercloud to deploy os_net_config.

Change-Id: I507c237a35250b660b37ea8cfc4e8e7f97ae21e2

Merge "use parameter to lookup the step instead of hiera again"

Merge "Remove unused redis_vip parameter"

use parameter to lookup the step instead of hiera again

In some profiles, we were looking up the $step by using Hiera
again, while we already do it in the parameter definition.
When using this class outside THT, it will fail but with this patch, we
could use just set the $step parameter and the rest of the manifest will
work.

Change-Id: I7082f47204fb4e529b164e4c4f1032e7bdd88f02

Generate HAProxy certificates in base profile

This gives the option to generate the service certificate(s) that
HAProxy will use. This will be used for both the overcloud and the
undercloud.

bp tls-via-certmonger

Change-Id: I3d0b729d0bad5252c1ae8852109c3a70c0c6ba7d

Merge "Deploy Keystone and Gnocchi API with new Apache profile"

Merge "Include gnocchi config on non-bootstrap nodes"