Merge "profile/nova/compute: add iscsi bits"

Merge "Composable Horizon service - puppet-tripleo"

Merge "Add ironic inspector as a terminated HAProxy endpoint"

Composable Horizon service - puppet-tripleo

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

Merge "Split Sahara pacemaker roles into separate services"

Merge "Add new nuage agent profile."

Merge "Fix Ceilometer profiles"

Add new nuage agent profile.

The Nuage agent will also get used on compute nodes.

Change-Id: Ic842ebdc183918ec9a24f83ae39adfba27bc799c

Add new midonet_agent profile.

We include this from the midonet profile (which is for the controller).
Agent will also get used on compute nodes as well.

Change-Id: I2393fc3c360f5f5786411f64dbcd06f380376093

Use include for base neutron profile

Simplify the base neutron profile so that we always include
the neutron and neutron::config classes at step 3. In doing so
we can now simplify the pacemaker version to use a simple include
which will avoid duplicate class errors.

Change-Id: I95b9188607ab6c599ad4cde6faa1deb081618f3e

Split Sahara pacemaker roles into separate services

In order for each service to operate independently when used with Pacemaker,
the roles needed to be separated.

This also drops all pacemaker constraints, as they do not make much sense
in the composable realm.

Change-Id: Id61eb15b1e2366f5b73c6e7d47941651e40651b1
Partial-Bug: #1592284

Fix Ceilometer profiles

* Configure ceilometer to default mongodb backend.
* remove useless sync_db parameter in pacemaker class.
* Remove duplicated resource for ceilometer central agent service with
  pcmk.
* Remove pacemaker delay resource dependency, the resource does not
  exist anymore.

Change-Id: Ie337bfe770c5b22158dc307eb36e39c2b93b95a9

Merge "Add non-pcmk Trove API/Conductor/Taskmanager profiles"

Merge "Create kernel profile"

Merge "Add gnocchi profiles"

Merge "neutron/plugins/ml2/bigswitch: do not require agent"

Merge "nova: do not manage nova-compute with pacemaker"

Merge "Split Heat pacemaker roles into separate services"

Add gnocchi profiles

Implements: blueprint refactor-puppet-manifests

Co-Authored-By: Carlos Camacho <ccamacho@redhat.com>
Co-Authored-By: Juan Antonio Osorio Robles <jaosorior@redhat.com>
Change-Id: I38f218da118c45f3af1938eb17298017f4c70e24

Create kernel profile

It will load and configure kernel modules.

Change-Id: If4f1047ff8c193a14b821d8b826f637872cf62bd

nova: do not manage nova-compute with pacemaker

In the case where all services are installed on the same node
(All-In-One), we don't want Pacemaker to manage nova-compute. This patch
makes the Nova service more specific for overriding start/stop/restart
actions, to be applied only for services that usually run on
controllers.

Change-Id: I81a86331cff4e7988e035ada82834a45a6bb99c1

Merge "nova/compute: add NFS bits"

Merge "Set bootstrap_nodeid to undef by default"

Merge "Replace reference to cinder:: parameter with nova:: parameter"

Merge "Default ceph_osd_selinux_permissive to false"

Merge "Support deprecated enable_ceph_storage boolean"

Set bootstrap_nodeid to undef by default

bootstrap_nodeid is a parameter looked-up with Hiera but only set on
controllers. With the transition to composable roles, we don't need to
set this parameter on compute nodes, so let's just default this value by
undef so we can deploy Neutron classes on compute nodes.

Change-Id: I94034f696c163969a7489d9cb0a20423210d3b84

Replace reference to cinder:: parameter with nova:: parameter

Changes a reference to a cinder:: parameter into the nova compute
class to a nova:: parameter, which makes sure we're not trying
to access "out of bound" parameters.

Change-Id: I871ef420700e6d0ee5c1e444e019d58b3a9a45a6

Merge "Set manage_roles for all steps in Keystone/Pacemaker"

Merge "nova: manage migration bits in a composable way"

Default ceph_osd_selinux_permissive to false

We don't set selinux in 'permissive' mode by default anymore.

Change-Id: I1aa46086f69e7c3efd2782da62fd18ade8343fde
Partial-Bug: 1595518

Support deprecated enable_ceph_storage boolean

This was missing in the initial revision of the Ceph modules but we
want to support usage of the old parameter for a cycle.

Change-Id: I7ac096feb9f5655003becd79d2eea355a047c90b

nova/compute: add NFS bits

Deploy NFS bits that were in THT before.

Change-Id: I75d68cc766ad274b16b22f43b7d34d02ab26de13

Merge "Nova API and Neutron server should run database creation in step 3"

Merge "neutron plugins: include neutron base"

Merge "Remove base/neutron/ml2.pp"

Resolve duplicate base::ceilometer issues

We are seeing CI failures in Ia0a59121b9ffd5e07647f66137ce53870bc6b5d6
due to duplicate Tripleo::Profile::Base::Ceilometer.

Also, this patch gets the pacemaker_master variable out of the
'base' tree.

Change-Id: I2cf3362528235fcdeaef1491f86c05f4995938da

Nova API and Neutron server should run database creation in step 3

We perform the Galera setup in step 2 so there is no guarantee that the
database will be available in that same step [1].

We used to implement a dependency in puppet using the 'galera-ready'
resource (clustercheck) but this is not possible with roles because we
also don't have any guarantee about clustercheck being installed on the
same node.

Because of the above all services must create/sync their databases
in a later step. This patch fixes Nova API and Neutron Server, the other
services use step 3 already.

1. https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/services/README.rst

Change-Id: I22750ffb64afbe40b5560a6a0d0dabc5b8927d32

nova: manage migration bits in a composable way

Allow to enable/disable migration bits from a single place, and select
which services are running on a node.
The use case here is to allow container deployements where libvirt &
nova-compute are separated.
Also support collocation for backward compatibility.

Change-Id: I0b765f8cb08633005c1fc5a5a2a8e5658ff44302

profile/nova/compute: add iscsi bits

Add iscsi bits allowing live migration.

Change-Id: I495a7af4220f0b35e9fd0a6cfc146610d8f46d10

neutron/plugins/ml2/bigswitch: do not require agent

A change in puppet-neutron now makes sure what agent has
python-networking-bigswitch package which was required on the agent, so
we don't need to load the plugin on the agent node anymore.

Change-Id: Ibde20ddb74da10e6eb0637a9ae37a550d33c038f
Depends-On: I651515b865802e59fa1234b87f82a0020e3bb9ca

Merge "Add puppet profile for swift storage"

Set manage_roles for all steps in Keystone/Pacemaker

We the base::keystone profile is trying to create users and roles
before step 5, this change will set the a value for manage_roles
from the pacemaker profile for all steps.

Change-Id: I7aca8d6418988160641b29f9396a0fbe33b64e51

neutron plugins: include neutron base

neutron base is required by plugins to work correctly. Let's add it here
so we don't have to deal with it in THT.

Change-Id: Icaf6b0479297983220874d7ee9ea2c3336706332

Remove base/neutron/ml2.pp

Not used anymore.

Change-Id: Id3f943132fce235a5a4888da30332b8fc2434a78
Depends-On: I6436c5c36ac0081a720cb8a737a573b1246127a1

Merge "Add base ironic profiles"

Revert "Pin puppet-lint-absolute_classname-check to 0.1.3"

This reverts commit b546ad26fbdc98e6ae09d41b2ab2982db4c55135.
Bug was fixed, see:
https://github.com/voxpupuli/puppet-lint-absolute_classname-check/releases/tag/0.2.2

Change-Id: I95727fa7763064ab06bbccead9bf80d913863b5d

Merge "Add Ceph profiles"

Merge "Move neutron ml2 plugins in plugins dir"

Pin puppet-lint-absolute_classname-check to 0.1.3

https://github.com/voxpupuli/puppet-lint-absolute_classname-check/issues/6

Change-Id: Ie6b69aef23c2028ad8632038a9a8283f4c1b8d40

Move neutron ml2 plugins in plugins dir

Move neutron ml2 plugins in plugins directory and make it compatible
with current THT.

The old class will be removed soon.

Change-Id: I35332df5800083f3373f5c37411534d11e9b4401

Merge "Drop galera_bootstrapped fact"

Add puppet profile for swift storage

Breaks out the swift storage configuration from t-h-t to a
composable profile

Change-Id: Ie68d7eccf4938bdbdea93327af0638b3fd002b3e
Partially-Implements: blueprint refactor-puppet-manifests

Add ironic inspector as a terminated HAProxy endpoint

This is needed for the undercloud, as it's in HAProxy where we make
the SSL terminations.

Change-Id: Ie4d652b4e5a95849c2fa32a5ce5ecec09ccb6bd9
Related-Bug: #1595047

Split Heat pacemaker roles into separate services

In order for each service to operate independently when used with Pacemaker,
the roles needed to be separated.

This also drops all pacemaker constraints, as they do not make much sense
in the composable realm.

Change-Id: I3fa76562c13191884a53848ad5adcbc9f5a24f61

Merge "Create libvirt micro-service"

Add Ceph profiles

Add Ceph base, monitor, osd and client profiles. Does not add
pacemaker specific versions as ceph is not managed by pacemaker.

Change-Id: Ie92b25a9c68a76b6d92abedef31e8039b16d9863

Merge "Implement Libvirt profile"

Merge "Add Neutron opencontrail plugin profiles"

Create libvirt micro-service

Create a new profile where we only manage Nova libvirt service.

Change-Id: I2ff3cf6a31bc10aa46a089fcbfc654eda55e6ef8
Implements: blueprint refactor-puppet-manifests
Depends-On: Ib0d3111560af5af451e522c6dc3b3918d0463e7d

Merge "Add ceilometer profiles"

Implement Libvirt profile

Deploy bits when running nova-compute + libvirt.

Note: some work will be done in puppet-nova to decouple nova-compute
from libvirt in nova::compute::libvirt class.

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

Merge "Add fact to get the fqdn for a host in the different networks"

Drop galera_bootstrapped fact

It's not in use.

Change-Id: I4f5a82e439699967b0c10e1d01dd7b3aaa52b5d3

Add Neutron opencontrail plugin profiles

Change-Id: Ie5ac07e6dd447216f17481ea4acf3bc204926bc8

Merge "Add pacemaker nuage profile."

Add ceilometer profiles

Implements: blueprint refactor-puppet-manifests

Change-Id: I4b5e93a108e80e91af26ffee454130ee18c0042e

Merge "Deploy nova::db classes in api profile"

Merge "nova/api: include ::nova::network::neutron"

Add fact to get the fqdn for a host in the different networks

This will be useful for getting a certificate for a specific
hostname via certmonger.

bp tls-via-certmonger

Change-Id: I3d563061154533f4dc3a0ee48394afbfa954282a

keystone: fix a race condition in bootstrap

Before, we had bootstrap_master set to 'undef' by default that was
used to whether or not run keystone bootstrap exec during deployment.

Setting the value to undef was a mistake, because enable_bootstrap in
puppet-keystone is set to true by default, so bootstrap was running
on all controllers at step 4 for HA scenario, while we want it to run on
a single controller (pacemaker_master) at step 4, like we do for
db-sync.

This patch:

* removes bootstrap_master.
* re-use sync_db to whether or not run keystone bootstrap.
  so it will only run on a single node when database is ready and
  db-sync done (orchestration dones by anchors in puppet-keystone).

Change-Id: I1042862f7c346d1c358b908c33eae0f33afd5e9f

Add puppet profile for snmp

Breaks out the snmp configuration from t-h-t to a composable profile

Change-Id: I7e6ab28a99b759594187e8571af6e22d836ae7bf
Partially-Implements: blueprint refactor-puppet-manifests

Deploy nova::db classes in api profile

Move nova::db classes from THT to puppet-tripleo in Nova API profile.

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

nova/api: include ::nova::network::neutron

It was included in THT before, but it's now in nova/api role.
It will also be added in nova/compute role later.

Change-Id: I6b5857d3d4740c0bf3f748719f30a05f1c62cb59

Add base ironic profiles

Co-Authored-By: Dmitry Tantsur <dtantsur@redhat.com>
Partially-implements: blueprint ironic-integration
Change-Id: I4831789b31cf5df1e61fe3ec05994156e0afc054

Merge "add plumgrid neutron profile"

Implement Nova Compute profile

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

Implement Nova vncproxy profile

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

Implement Nova Consoleauth profile

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

Implement Nova Scheduler profile

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

Implement Nova API profile

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

Add mongodb profiles

Implements: blueprint refactor-puppet-manifests

Co-Authored-By: Carlos Camacho <ccamacho@redhat.com>
Co-Authored-By: Juan Antonio Osorio Robles <jaosorior@redhat.com>
Change-Id: Idb1e78ebec7682fe68ca5902a22cfb6030498091

Pacemaker: only create DB on pacemaker master

This fixes a bug introduced in 72fb6727951af2cf5b16e686b3a36f5f3a7c71c7
where non-master nodes would try to create the database (and potentially
fail due to a race).

Change-Id: Ic0c7b05618b69caca0d4019b5153163f2405bbe5

Merge "Import ::nova class from THT"

Merge "Make neutron server/ml2 profiles match t-h-t"

Merge "Enable bulk delete in swift for gnocchi"

Import ::nova class from THT

Import ::nova class with memcached parameter computed from Hiera, that
was previously in THT, now in nova-base role.

Use step 3 for ::nova since we need it for database resources.

Also make sure nova base profile is included for conductor role and any
nova pacemaker role.

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

Merge "Include neutron in step 3"

add plumgrid neutron profile

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

Add pacemaker nuage profile.

The pacemaker version of nuage has its DB sync at a different
step so we need a separate profile to ensure plugin.ini exists.

Change-Id: Ia4a297008b259a8e238bd1ad511e82f4e5ab6d63

Make neutron server/ml2 profiles match t-h-t

This patch brings the neutron profiles and the
associated steps in line with what already happens in
t-h-t. Specifically:

 -we want to create the db $step >= 2 and $sync_db
 -we want to make sure plugin.ini exists before the neutron dbsync
 -we want to make sure the db sync runs before neutron::server starts
  when using pacemaker
 -split the neutron server profiles. They are quite different across
  pacemaker and base.

Change-Id: I52815f45a04bf3e39940b9cb116261730580a3e2

Enable bulk delete in swift for gnocchi

Closes-bug: #1590495

Change-Id: I912c09584d9b4cae530a247c91b6fd331943a8c5

Remove loadbalancer profile

We don't need loadbalancer profile anymore, we now have haproxy &
keepalived profiles that replace it.

Change-Id: I5bf57f88a85fa8180392e9dde7ab39f4eda63113

Include neutron in step 3

neutron::server requires the neutron's init.pp to be ran beforehand
because init.pp installs the 'neutron' package. Else the relationship
with neutron::policy will fail [1].So to address this, we move
neutron's init.pp to be ran on step 3 if db_sync is enabled.

[1] https://github.com/openstack/puppet-neutron/blob/master/manifests/server.pp#L498

Change-Id: I8eb497c2b483e11d1f640a7ba9fae5a9d5bcef4d

Merge "Drop duplicate neutron pacemaker constraint"

Merge "Deprecate loabalancer profiles"

Merge "Implement Nova conductor profile"

Merge "Implement Nova base profile"

Merge "Adds the puppet-tripleo manifests for manila"

Drop duplicate neutron pacemaker constraint

Drop a duplicate keystone-to-neutron-server-constraint constraint
for the neutron server profile.

Change-Id: I928f9f9f3cf665ac80696f66e1d64ceb02129b8e