Commit Graph

22 Commits

Author SHA1 Message Date
Alan Bishop
fce83f534b Support deploying multiple Cinder Netapp Storage backends
CinderNetappBackendName is enhanced to support a list of backend names,
and a new CinderNetappMultiConfig parameter provides a way to specify
parameter values for each backend.

The CinderNetappVolumeList parameter is properly marked as deprecated.
The corresponding parameter no longer exists in cinder and puppet-cinder,
and it was made obsolete when the CinderNetappPoolNameSearchPattern
was introduced several cycles ago.

Depends-On: I15f4828beb9db81583146fa9afdd21999ad5b078
Change-Id: Iecc11782372097bb7f24b6cbc342c38148b9c469
2020-11-12 13:44:33 -08:00
Giulio Fidente
c7ec726276 Use container_file_t for Cinder*NfsMountOptions by default
This is so that share content is usable by cinder-volume and other
containers mounting the same share.

Closes-Bug: 1890291
Change-Id: Iacf7c9c368b26106e9921b35996c134aacb9acd7
2020-08-26 13:04:10 +02:00
John Fulton
161071f2d8 Include copy-image for GlanceEnabledImportMethods for dcn-hci
If you are using environments/dcn-hci.yaml, then you very likely
have more than one Glance server and will want to use the copy-image
feature. Thus, enable it by default for deployments which use this
environment file.

Also because GlanceCacheEnabled defaults to False and because
GlanceImageCacheMaxSize defaults to 10737418240 we don't need
to explicitly set them in environments/dcn{,-hci}.yaml.

Change-Id: If745aa0824098950367525170eaf6cb4e3804482
2020-05-20 14:41:05 -04:00
Cédric Jeanneret
0875895553 Replace svirt_sandbox_file_t by container_file_t
While they are, at SELinux level, exactly the same (one is an alias to
the other), the "container_file_t" name is easier to understand (and
shorter to write).

A second pass in a couple of days or weeks will be needed in order to
change files that were merged after this first pass.

Change-Id: Ib4b3e65dbaeb5894403301251866b9817240a9d5
2020-02-07 13:33:20 +01:00
Alan Bishop
aa1f4bf621 Fix selinux context for glance-api
Remove the z flag from glance-api's service directory. The service
directory does not need to be shared with other containers, and
podman fails to apply setting with glance is using NFS (i.e.
/var/lib/glance/images is a mount point).

Also update the NFS mount options to use svirt_sandbox_file_t, which
is consistent with the parent service directory.

Closes-Bug: #1834857
Closes-Bug: #1844465
Change-Id: I7e135615fb53815ce14a3bcfec42b28f86d6dbae
2019-09-18 05:47:56 -07:00
Martin Schuppert
1c241362ff fix storage.yaml to write environments/storage/nova-nfs.yaml
753a350418 removed ceph entries
from sample-env-generator/storage.yaml which resulted in nova-nfs
and cinder-netapp-config to be in the same section. Therefore
environment_generator.py did not update the nova-nfs.yaml .

Also add quotes to the nfs version as they are required in
the resulting nova-nfs.yaml .

Closes-Bug: #1825374

Change-Id: I209480f209e3a8841362ea619b1c9b073c89f338
2019-04-18 15:03:19 +00:00
Alan Bishop
639285f091 Update parameters for cinder's Netapp backend
Update THT to align with puppet-tripleo changes made in [1]
- Add new CinderNetappPoolNameSearchPattern parameter
- Deprecate CinderNetappStoragePools parameter
- Remove previously deprecated CinderNetappEseriesHostType parameter

[1] https://review.openstack.org/570406

Fix relative path in file the sample-env-generator uses to generate
environments/storage/cinder-netapp-config.yaml.

Change-Id: I813ca60eb5ce9e008e1b72e88d83709d3125676f
2019-01-24 15:39:00 -05:00
David J Peacock
123f40a565 flatten cinder service configuration
This change combines the previous puppet and docker files into a single
file that performs the docker service installation and configuration.

With this patch the baremetal version of cinder services has been removed.

Change-Id: I88f047a8ee9c3eed80e4c48ed9cabdb3035d518b
Related-Blueprint: services-yaml-flattening
2019-01-18 08:55:26 -05:00
Alan Bishop
dc2d0de810 Add parameters for cinder storage availability zones
Add CinderStorageAvailabilityZone parameter that configures
cinder's DEFAULT/storage_availability_zone. The default value
of 'nova' matches cinder's own default value.

Add several CinderXXXAvailabilityZone parameters, where XXX is
any of the cinder volume service's storage backends. The
parameters are optional, and when set they override the
"backend_availability_zone" for the corresponding backend.

Implements: blueprint split-controlplane-cinder-volume-az
Depends-On: Ic407b747474b567858ad36beabc8a7d8c5022343
Change-Id: Idb035bf112cbab41547bd89935df4c175bf665f4
2018-10-23 09:18:53 -04:00
Martin Schuppert
5dd4018141 Instance create fails due to wrong default secontext with NFS
With NovaNfsEnabled instance create fails due to wrong default
secontext. The default in THT is set to nova_var_lib_t in
Ie4fe217bd119b638f42c682d21572547f02f17b2 while
system_u:object_r:nfs_t:s0 should have access. The virt_use_nfs
boolean, which is turned on by openstack-selinux, should cover
this use case.

This changes the default to context=system_u:object_r:nfs_t:s0

Change-Id: I2a28462b6f6bc9f8a41a81ea8c65471f05df3b85
Closes-Bug: 1781894
2018-07-24 08:12:07 +02:00
Alan Bishop
d989c01ce4 Clean up env files for Cinder's Netapp backend
Remove the misleading comment from the cinder-netapp-config.yaml file in
the environments directory that claimed the file is deprecated in favor
of the 'sample-env-generator' file in the environments/storage directory.
The sample-env-generator has not been widely adopted, and the Netapp
file in the environments directory is still being maintained.

Update the sample-env-generator files so they specify the
OS::TripleO::Services::CinderBackendNetApp composable service.

Closes-Bug: 1779397
Change-Id: If0343e96700bf13c0d49c3ae3c311ada1c26a0c0
2018-06-29 13:58:57 -04:00
Pranali Deore
a089310bf3 Add support of shared staging location for glance-direct
In the case of an HA setup (ie with 3 controller nodes) if image
import method is 'glance-direct', it would require a shared
consistent staging location on the controller nodes otherwise it
might happen that the image is uploaded to one node and triggering
the import is executed on a different node - where the image does not
exist.

Hence when import method 'glance-direct' is needed, operators need to
enable it explicitly with enabling NFS across the controller nodes
to share the staging.

Change-Id: I9a5bff5e5d0c3e109b73d691cc9088904a0401ac
Closes-Bug: #1765439
2018-05-22 21:40:13 +05:30
Giulio Fidente
753a350418 Remove support for puppet-ceph
Deployment of a managed Ceph cluster using puppet-ceph
is not supported from the Pike release. From Queens it
is not supported use of puppet-ceph when using an
external Ceph cluster either.

This change removes the old templates necessary to
support deployment of Ceph via puppet-ceph.

Implements: blueprint remove-puppet-ceph
Change-Id: I17b94e8023873f3129a55e69efd751be0674dfcb
2018-05-18 14:00:30 +02:00
Rajesh Tailor
6b6ae966ba Allow configuration of NFS backend for Nova
Allow NFS configuration of storage backend for Nova.
This way the instances files will be stored on a shared
NFS storage.

Implements: bp tripleo-nova-nfs

Depends-On: Id15aec6324814a871e87f19f24999b0e3b8a8f05

Change-Id: Ie4fe217bd119b638f42c682d21572547f02f17b2
2018-05-16 11:18:48 +05:30
Pranali Deore
5f7fdb1656 Add support of netapp configuration for glance in THT
This change adds all the required netapp environment configuartion
for glance in heat template.

Change-Id: Ib44796f77cbc50c3b3c37dcdd37a59f48a7baa1a
Closes-Bug: #1740456
2018-03-01 17:07:34 +05:30
Pradeep Kilambi
b2b37b6bda Add S3 storage driver as an option and its config
S3 storage driver is a supported option for gnocchi upstream.

The puppet change has been added in this commit
I5a49fa34d344c4464ea883bf96350f00b0a17eac

Change-Id: Ib44278ef87c8c4d1c39a50d24c1f02fe1fe4042e
2018-02-08 10:50:12 -05:00
Martin André
64854af783 Fix glance config generation when using NFS
This commit sets the 'tripleo::glance::nfs_mount::edit_fstab' variable
to false to prevent the glance_api to write to /etc/fstab and fail in
the case we're using an NFS backend.

This also updates the NFS mount option and mount point to align with
the config from the puppet-tripleo module at:

https://github.com/openstack/puppet-tripleo/blob/master/manifests/glance/nfs_mount.pp

Change-Id: Ied27fac67d28b3104441ada469942a19e3c07a89
Closes-Bug: #1732928
2017-11-22 15:00:38 -05:00
Alan Bishop
e828e8c7bb Enable Cinder as a backend for Glance
Enable Cinder as a backend for Glance by adding 'cinder' to the list of
allowed choices for the GlanceBackend heat parameter.

Update the glance-api docker configuration to allow the feature to work.
This is necessary because the feature uses iSCSI, which requires additional
privileges.

Depends-On: I850047e32f3608b3ce490e52e2e540695cb1a4ff
Change-Id: I42241747de931103a04aa5ee2ed18fd46197d183
2017-10-12 08:11:52 -04:00
Jan Provaznik
ad8589212c Let mds create manila key and fs
ceph-ansible will take care of setting up client keys both
in ceph and on client side. It will also create filesystem
for manila. To assure that manila manifest can work in future
both with puppet and with ceph-ansible, creation of filesystem
is moved to ceph-mds manifest and creation of manila key on ceph
side is moved to ceph-base (so manila key is always created),
manila key is added to ceph-external for external ceph deployments.
Key creation is removed from manila.pp in patch
I2b5567a39ac8737e80758b705818cc1807dc8bf1

Change-Id: I6308a317ffe0af244396aba5197c85e273e69f68
Related-To: Ia3ef9e9a2b159dacea01e38762145ff2bcc7ba27
Depends-On: I3f18bbe476c4f43fa4e162cc66c5df443122cd0c
2017-08-18 16:22:10 +02:00
Ben Nemec
56bf9cfb78 Fix enable-ceph sample environment
The necessary resource registry entries were missing from this env
and the old environment was not deprecated.

Change-Id: I6a9b148514fc5da1f96b9fd7fe09f564c2f82419
2017-07-26 13:36:52 +00:00
Ben Nemec
edfcb50f17 Update cinder-netapp-config environment
It appears some new parameters were added to this template, so the
sample environment needs to be updated to include them.

Change-Id: Ifc68cbf7d18acf72d6d1ed2beefbc0cd18d72bf8
2017-06-20 15:53:23 -05:00
Ben Nemec
8d086b1710 Add storage sample environments
Starts converting storage-related sample environments to the tool,
and adds a few new ones for demonstration purposes.

This has required the addition of a new category of parameter
overrides in the tool.  There are some parameters that are part of
the public API of roles that should not normally be included in a
sample environment for that role.  Examples are EndpointMap and
ServiceNetMap.  Those are both passed into most (all?) roles, but
their template defaults are not useful (both default to {}).
Unless we are explicitly creating a sample environment that
overrides those defaults we don't want them included.

Parameters such as RoleName and RoleParameters are similar.  We
can't change them because they are part of the composable roles
interface and that would break any existing custom roles, but we
don't really want them included normally either.  It's possible
these could be made completely private, but there have been some
very preliminary discussions about generating role samples that
might actually want to set them.

In order to avoid issues with editing the unit test file in editors
that strip trailing whitespace, the minor formatting bug where
params like EndpointMap had a trailing space after the name has
also been fixed.

Change-Id: If11f30c734bfbc17d463a9890c736d7477186fb9
2017-06-12 15:02:50 -05:00