729 Commits

Author SHA1 Message Date
Zuul
311c2de714 Merge "Add OVS-DPDK parameter as part of roles file" 2018-07-31 12:21:09 +00:00
Zuul
3632e22569 Merge "Add support for containerized networking-ansible ML2 plugin" 2018-07-31 07:16:48 +00:00
Zuul
f5e6294830 Merge "Fix parameter name used to create the Manila CephX keyring" 2018-07-31 01:18:18 +00:00
rabi
26e0ce7702 Add support for containerized networking-ansible ML2 plugin
Co-Authored-By: Derek Higgins <derekh@redhat.com>
Change-Id: I8500de7ec71a8efa567a0b3aa29725ef235795bc
2018-07-30 20:48:22 +00:00
Zuul
bef70cb6de Merge "NUMA aware vswitches" 2018-07-30 19:54:05 +00:00
Giulio Fidente
787e7910a8 Fix parameter name used to create the Manila CephX keyring
We obsoleted the CephFS native parameter but missed to update it
in the Ceph profile.

Change-Id: I12a91da7df6079d391fa9583c4f2c926d123044f
Closes-Bug: #1783341
2018-07-30 12:05:48 +00:00
Zuul
fac21c7ed1 Merge "Allow setting physical network MTU via heat template" 2018-07-26 18:16:14 +00:00
Rajesh Tailor
1b8e78af03 NUMA aware vswitches
Expose configuration parameteres for NUMA
aware vswitches.
The related nova changes are done in change:
https://review.openstack.org/#/c/564440/

Depends-On: Ia8fd65a75bc732087128e9e32ba72b2491423898
Change-Id: I318ba9c262f64c0d416a017ed836ae0729acedb4
2018-07-25 16:32:47 +05:30
Zuul
3e875145cd Merge "DnsServers using get_attr" 2018-07-25 00:53:28 +00:00
Zuul
7ce066eba3 Merge "EC2MetadatIp using get_attr" 2018-07-25 00:36:06 +00:00
Zuul
b0a08fa572 Merge "Update manila environment file names" 2018-07-24 04:49:31 +00:00
Zuul
ad5d4d1db1 Merge "ControlPlaneDefaultRoute using get_attr" 2018-07-24 01:34:22 +00:00
Zuul
dfc09b6ff1 Merge "ControlPlaneSubnetCidr using get_attr" 2018-07-24 01:34:20 +00:00
Zuul
36118fceb4 Merge "Add release note for vnx and unity template changes" 2018-07-24 00:21:48 +00:00
Kamil Sambor
29bd1a9d87 Allow setting physical network MTU via heat template
Add 'neutron::plugins::ml2::physical_network_mtus' as a
NeutronML2PhysicalNetworkMtus in heat template to allow
set MTU in ml2 plugin

Change-Id: I2f9666a5422a1272d53c869e4dad8581ab605b37
Closes-Bug: 1782123
2018-07-23 15:31:29 +02:00
Zuul
959e1d7f75 Merge "Enable deep_compare of pcmk resources by default" 2018-07-20 15:38:20 +00:00
Zuul
ce2aa5b4dc Merge "Add AllNodesExtraMapData parameter" 2018-07-20 13:28:09 +00:00
Zuul
a9fda38818 Merge "Fix HostnameMap lookup - replace str_replace with yaql" 2018-07-20 04:31:45 +00:00
Zuul
6f59e1bdc5 Merge "Improve nova statedir ownership logic" 2018-07-19 22:08:20 +00:00
Tom Barron
08e67b8ff0 Update manila environment file names
to eliminate special 'docker' names.

Containerized manila is now the standard way to deploy
so we can avoid confusion by just using the regular
manila environment file names for these rather than having
some manila environment files with 'docker' in their names.

Note that we are using the content of
environments/manila-cephfsnative-config-docker.yaml
for
environments/manila-cephfsnative-config.yaml.  This resolves an
issue wherein the parameters in the latter file were not updated
when support for the CephFS-NFS back end was added [1].

Closes-Bug: #1782622

[1] Ib408c7827e5fba0c1b01388db26363806fc64370

Change-Id: I08431d848158846a434fecc98d3bee838c7fad68
2018-07-19 18:46:10 +00:00
Michele Baldessari
60d75f19e5 Enable deep_compare of pcmk resources by default
By settings these hiera keys we will actually do a deep comparison
of all pacemaker resources and update them accordingly.

This means that rerunning a deploy against an existing overcloud
will actually detect if a pacemaker resource needs updating and
will update the definition and restart of said resource. The code
will wait for the resource to be up (aka the cluster to settle
after an update).

Depends-On: I62d5662327333e17e1c32a029695b3d3a904ca10
Change-Id: I0b9ed429e6b1bd4a62b4462bf6312782f29b7d6b
2018-07-19 15:52:36 +02:00
Harald Jensås
aeb783fdcf Fix HostnameMap lookup - replace str_replace with yaql
Using str_replace when looking up hostnames in
HostnameMap may return unpredictable results.
For example if the map contain:

HostnameMap:
  overcloud-compute-12: cmp-12-london
  overcloud-compute-123: cmp-123-berlin

When we do string replace for a hostname that is
not in the map (example: 'overcloud-compute-124')
it will match on the substring. The result would
be 'cmp-12-london4'.

This change replaces the str_replace with a yaql
expression that will return the value of they key
matching the hostname provided. If the key is not
in the map, the hostname is returned.

Closes-Bug: #1781560
Change-Id: Ia3a6a3539cd566131cd81b4d755190aa5b61a573
2018-07-19 07:24:29 +00:00
Yong Huang
3474e4b15d Add release note for vnx and unity template changes
Add release note for vnx and unity recently template changes:
https://review.openstack.org/#/c/572260/

Change-Id: I5202cd6cd44d9c45315cffc3805b4c4854900005
Closes-bug: #1782056
2018-07-17 13:41:26 +08:00
Zuul
c17fbfd823 Merge "Log more details for container images prepare" 2018-07-16 17:29:07 +00:00
Saravanan KR
e7d39847bc Add OVS-DPDK parameter as part of roles file
Adding mandatory OVS-DPDK parameter to the roles file instead of
environment file, as it requires to be added for each custom role.

Change-Id: I33fa198228e3e3adcb8e93146d9a9caa46a28895
2018-07-16 09:58:55 +05:30
Zuul
58cb630f75 Merge "Reduce default services for Standalone" 2018-07-15 19:47:12 +00:00
Harald Jensås
52ec1a018b DnsServers using get_attr
Nameservers are configured on the ctlplane subnets by the
undercloud installer, the nameservers are used early during
the deployment, prior to running os-net-config.

Remove the default DnsServer's in THT, replacing it with
an empty list and use get_attr to get the values for
DnsServers for the overcloud from the ctlplane subnet(s).

A conditinal is used in  puppet/role.role.j2.yaml so that
the parameter value is used whenever it is not [] (default)
to provide backwards compatibilityi and in case the user
want to use different DnsServers for the overcloud and
undercloud.

Partial: blueprint tripleo-routed-networks-templates
Change-Id: I5f33e06ca3f4b13cc355e02156edd9d8a1f773cd
2018-07-14 09:19:12 +02:00
Harald Jensås
19381ecffb EC2MetadatIp using get_attr
The route to metadata service is set up in host_routes
of ctlplane subnets by extraconf post deploy::
  extraconfig/post_deploy/undercloud_ctlplane_network.py

Use get_attr on the server resource to resolve attribute
value from the subnet(s) and pass it to the parameter
'EC2MetadatIp' used in the THT/network/config/* templates.

Changes the default for 'EC2MetadatIp' to ''.
Removes the comment that the value should be overriden in
parameters_defaults. It also removes the parameter from
network-environment templates.

A conditinal is used in  puppet/role.role.j2.yaml so that
the parameter value is used whenever it is not '' (the
default) to provide backwards compatibility in case the
user set a different value for this parameter in
network-environment.yaml.

When deploying a routed control plane the network config
templates would previously need to be updated to carry
'EC2MetadatIpLeafX' parameters for each leaf.  By getting
the value to pass from the server resource this change
reduces the required nic-config template customisation.
(Reduces the risk of user error.)

Partial: blueprint tripleo-routed-networks-templates
Change-Id: I9c019ec840a44ca8c5f98be55daea365bc6554ec
2018-07-14 09:18:58 +02:00
Harald Jensås
c649cf0545 ControlPlaneDefaultRoute using get_attr
Use get_attr on the server resource to resolve attribute
value from the subnet(s) and pass it to the parameter
'ControlPlaneDefaultRoute' used in the THT/network/config/*
templates.

Changes the default for 'ControlPlaneDefaultRoute' to ''
as well as the comment that the value should be overriden
in parameters_defaults. It also removes the parameter from
network-environment templates.

A conditinal is used in  puppet/role.role.j2.yaml so that
the parameter value is used whenever it is not '' (the
default) to provide backwards compatibility in case the
user set a different value (different from the one used in
undercloud.conf) for this parameter in
network-environment.yaml.

When deploying a routed control plane the network config
templates would previously need to be updated to carry
'ControlPlaneXDefaultRoute' parameters for each leaf. With
8 Leafs in addition to the network local to the undercloud
that is 8 parameters less to place in the configuration.
By getting the value to pass from the server resource this
change reduces the required nic-config template
customisation (reduces the risk of user error).

Partial: blueprint tripleo-routed-networks-templates
Change-Id: I5139249d55e9ac01761c270b8c0f31ef35595940
2018-07-14 09:11:39 +02:00
Harald Jensås
6ab86a3ebe ControlPlaneSubnetCidr using get_attr
Use get_attr on the server resource to resolve attribute
value from the subnet(s) and pass it to the parameter
'ControlPlaneSubnetCidr' used in the THT/network/config/*
templates.

As the value is now resolved from resource attributes,
this changes the default for 'ControlPlaneSubnetCidr' to ''
as well as the comment that these value should be overriden
in parameters_defaults. It also removes the parameter from
network-environment templates.

A conditinal is used in  puppet/role.role.j2.yaml so that
the parameter value is used whenever it is not '' (the
default) to provide backwards compatibility in case the user
set a different value (different from the one used in
undercloud.conf) for this parameter in
network-environment.yaml.

When deploying a routed control plane the network config
templates would previously need to be updated to carry
'ControlPlaneXSubnetCidr' parameter (in case the subnet
mask is not the same for all the routed network leafs).
With 8 Leafs in addition to the network local to the
undercloud that is 8 parameters less to place in the
configuration. By getting the value to pass from the
server resource this change reduces the required nic-config
template customisation (reduces the risk of user error).

Partial: blueprint tripleo-routed-networks-templates
Change-Id: I92ee0f9a2107cdf1ca5903d3756a235a79c36c73
2018-07-14 09:11:28 +02:00
Zuul
e120a5e42b Merge "ceilometer: deprecation cleanup" 2018-07-13 14:51:43 +00:00
Bogdan Dobrelya
e489e58db4 Log more details for container images prepare
Add a heat param to log tripleo container images prepare
verbose details into a file.

Depends-On: I1c72b5ef0d7acbc4eded422d569f1383d92ad3c2
Change-Id: I000ffeb9b111c4a2a8919fe448dcead922ef03c3
Signed-off-by: Bogdan Dobrelya <bdobreli@redhat.com>
2018-07-13 14:38:51 +03:00
Alex Schultz
72b2417e95 Reduce default services for Standalone
With the standalone deployment mechanism, the default list of enabled
OpenStack services is Keystone, Nova (and related), Neutron (and
related), Glance, Cinder, Swift and Horizon. The default list of
disabled of OpenStack services is Aodh, Barbican, Ceilomter,
Congress, Designate, Gnocchi, Heat, Ironic, Manila, Mistral, Panko,
Sahara, Tacker and Zaqar. Disabled services can be enabled by
passing the appropriate environment files to re-enable them during
the deployment.

Change-Id: Ibf49ac61f4a50b429e72ebeb5b35f6524ae8f24d
Related-Blueprint: all-in-one
2018-07-11 16:46:02 -06:00
Zuul
a356b336fe Merge "Add support for {{role.name}}IPs in Composable Role" 2018-07-10 01:57:46 +00:00
Zuul
8666a73d68 Merge "Remove the OVS-DPDK deprecated parameters" 2018-07-09 23:03:32 +00:00
James Slagle
7f42272024 Add AllNodesExtraMapData parameter
The AllNodesExtraMapData parameter is used to inject additional
hieradata into the all_nodes hierdata file on each node. The injected
data will be deeploy merged with the calculated all_nodes data for the
stack.

The parameter can be taken advantage of for split-controlplane use cases
where the hieradata from the control stack needs to be populated into
the separate compute stacks.

To easily get the hieradata out of the control stack, a new stack output
is added, AllNodesConfig.

Partially Implements: blueprint split-controlplane

Change-Id: I7b865bf82520006eef3ac2f36df34b1f3c34e642
2018-07-09 12:22:32 -04:00
Oliver Walsh
58624abf5f Improve nova statedir ownership logic
The nova_compute container kolla config is currently set to recursively change
the ownership of /var/lib/nova to nova:nova on startup.

This is necessary when upgrading from an non-container deployment to a
containerized deployment as the nova uids are not consistent across the host
and container image.

If the nova instancedir is an NFS mount then open filehandles are
broken and every VM using that NFS export fails with I/O errors.

This change re-implements the nova statedir ownership logic to target only the
files/directories controlled by nova.

Requires dist-git change: https://review.rdoproject.org/r/14734

Change-Id: I57d421feb6356d28002e77fb9bfa50a397758cbf
Closes-bug: 1778465
2018-07-09 17:07:30 +01:00
Zuul
66872cc8d9 Merge "Add subnet host_routes to output of OS::TripleO::*::Ports::Port" 2018-07-09 11:24:33 +00:00
Harald Jensås
7817fccc5b Add support for {{role.name}}IPs in Composable Role
Use the {{role.name}}IPs for network port resources in
Composable Role to enable setting predictable ip address
on network ports.

We currently support predictable ip addressing using the
from_pool templates. The from_pool creates a 'fake' neutron
port. Because of this we loose access to properties of
the ports subnet, properties that is useful to simplyfy
advanced configurations such as routed spine-and-leaf.

Creating a neutron port also open the possibility to use ML2
plugins in the future (Neutron Ansible ML2) to do port
binding that configures network devices.

The parameter (interface) to use this is the same as with
from_pool. Example:

parameter_defaults:
  {role.name}IPs:
    {network.name_lower}:
    - 10.0.0.1
    {network.name_lower}:
    - 172.16.1.1

We just don't need to override the resource registry. I.e
the following is not used/required.
resource_registry:
  ``OS::TripleO::{{role.name}}::Ports::{{network.name}}Port``

NOTE: This does not remove the from_pool templates. We may
      want to deprecate them, but in that case we need to
      ensure it is possible to upgrade an environment that
      used them by changeing the resource registry to use
      the default port templates in this change.

Related: blueprint tripleo-routed-networks-templates
Change-Id: I3f9b55ffbf6b6626b4d0dfbec3867811cb74de14
2018-07-06 21:54:43 +02:00
Zuul
995fa8451a Merge "Set configure_qemu to true to get qemu port range applied" 2018-07-06 18:06:52 +00:00
Zuul
1842528f14 Merge "Store ODL logs to file" 2018-07-06 13:34:01 +00:00
Harald Jensås
f0bc5e460a Add subnet host_routes to output of OS::TripleO::*::Ports::Port
Adds the host_routes on the neutron ports subnet to the output
of the Tripleo Ports template. This will allow use of get_attr
in the role to pass routes for a subnet to NetworkConfig.

Partial: blueprint tripleo-routed-networks-templates
Change-Id: I46896153ea59ae0a6a8dff0208de2b0d168927ae
2018-07-06 12:03:08 +02:00
Zuul
7ae479f416 Merge "Undercloud specific volumes for mistral-executor container" 2018-07-06 08:14:37 +00:00
Zuul
cf77ea0ae0 Merge "Add networking-ansible ML2 plugin support" 2018-07-05 17:44:05 +00:00
Zuul
c8bbe1c3a0 Merge "Handle blank lines followed by comment in nic conversion script" 2018-07-05 10:54:39 +00:00
Martin Schuppert
547552fc7b Set configure_qemu to true to get qemu port range applied
With https://review.openstack.org/#/c/561784 we change the default
migration port range to '61152-61215'.
nova::migration::qemu::configure_qemu needs to be set to true that
the config gets applied via puppet-nova.

Change-Id: Iad4b392c9fe7426f2ce10a02fadd8b1aeee34ef6
Closes-bug: 1779820
Depends-On: Idadfc7b3507977f1385e846a48a734ed0e5f0a32
2018-07-03 11:24:39 +02:00
Zuul
9f1d855627 Merge "OVN: Remove environment files which deploy OVN dbs in non HA" 2018-07-03 03:42:20 +00:00
Zuul
794b9d527e Merge "Add SELinux management to containerized undercloud" 2018-07-02 12:35:34 +00:00
Janki Chhatbar
223b9428f1 Store ODL logs to file
ODL is configured to log to console and then access the logs
via docker logs. Older logs are being deleted too fast. So
moving back to logging to file which is default setting in ODL
THT.
https://github.com/openstack/tripleo-heat-templates/blob/
master/puppet/services/opendaylight-api.yaml#L60

Change-Id: I29724cfb23e13c8293d1b7b4341cbd409ae1b7a6
Closes-Bug: 1779629
2018-07-02 10:35:25 +00:00
rabi
6d5b9ab421 Add networking-ansible ML2 plugin support
Depends-On: https://review.openstack.org/577074
Depends-On: https://review.openstack.org/576093

Change-Id: Ib0c2395fbabc654378101527ca065b5c69d3222d
2018-07-02 13:22:04 +05:30