This refactors resource dependencies to improve the following points.
- Avoid unnecessary dependencies across services. For example aodh
service does not require cinder db.
- Restart only api service when config files like paste.ini, which
are used only be api service is changed.
Change-Id: Iadb8552abf55228729bf5d31795b1e4bbb8b9929
This ensures the parameters used by if-else logic accept only boolean
values because non-boolean can result in unexpected behavior.
Change-Id: Ia67f4aba2c3064f9b0ba93430b9c0f7a14800ced
This change removes direct reference to some classes in
puppetlabs-apache. Details are explained below.
- The api class doesn't need access to anything defined in
apache::params
- The following classes are included by the openstacklib::wsgi::apache
resource type, and current inclusions are just redundant.
Change-Id: I88684f7f246b226844dc724a35d1cec95742704e
The previous approach could lead to restarting Apache after
Anchor['heat::service::end']. The new approach is already used in other
Puppet modules and guarantees restarting Apache between the service
anchors.
Change-Id: Ia4ccc8a6477f490f42b588b3e6a9d5c0f7ff6d88
Not only the heat-api and heat-api-cfn services are notified, but also
the apache service if the API is run using apache. This ensures apache
is restarted on config changes.
Change-Id: I68feb50a466b56b027573cda099191f6b26ddbfd
This includes a resource that will generically create the vhost for
the specified API.
Co-Authored-By: Thomas Herve <therve@redhat.com>
Depends-On: I9a9246522810de546a7c460ab1133d6bf9081a15
Change-Id: I253f46f5ad943971dd9ea6995591c72a36953bdb
Setting the certificate and key in heat's configuration has the
effect of enabling SSL. So, one can actually enable SSL using those
and ignore the use_ssl flag.
This commit enforces that flag, so if the flag is not set, then those
parameters shouldn't be persisted in Heat's configuration.
A unit test was removed for each of the correspondent API's because they no
longer apply. If ssl is not enabled, then heat_config resources containing
those configurations are not present in the resource catalog.
Change-Id: I87fb234943829d7f0d8786e98e90b81e9a71c8ef
Previously the anchors and dependencies that allow external hooks were
all in the main ::heat class. However, if you wanted to include just
::heat::db::mysql, then it would fail, since it assumed the main heat
class was included. This moves all of those resources and relationships
into a new class, ::heat::deps. All of the classes will now include
this class so that the anchors and deps are always evaluated even if
only a portion of the classes are used, and even if ::heat isn't pulled
in.
Change-Id: I4297df160a7afae2b66c1ac76e37de313fa4fb09
Closes-Bug: #1507934
This adds defined anchor points for external modules to hook into the
software install, config and service dependency chain. This allows
external modules to manage software installation (virtualenv,
containers, etc) and service management (pacemaker) without needing rely
on resources that may change or be renamed.
Change-Id: I032ee01505e0cbc125b0e219c436b77c93f57720
Currently we specify the ordering of config resources wherever it is
necessary based on the presence of the file it will write to, or the
presence of the package in charge of providing the file it will write
to.
Those kind of ordering can be specified directly at the resource level
using the autorequire mechanism. With this patch, any config resource
will make sure the package in charge of providing the file will be
installed first.
Change-Id: I6476060c97d350640b5a254738a60e319ad522e9
In order to be able to take an action after all the packages of the
module have been installed/updated or all the services have been
started/restarted, we set a 'heat-package' and 'heat-service' tag
for each package and service of this module.
At the moment, there is a generic openstack tag that is not specific
enough if one wants to take action upon a single module change.
Use case :
If an action needs to be taken after all the packages have been
installed or updated : Package <| tag == 'heat-package' |> -> X
Change-Id: I38a6b422054dbf0fceacf6b7e329dbb3cb0fa9cb
This adds a package_ensure parameter to all classes that did not have it
and updated all package resources to use the package_ensure parameter.
Change-Id: I49c3e50153bc5eba323c5a766ed1f30c23653985
Other components offer the option to decide whether or not to run the
db sync command. Heat was missing this feature. This commit add this
feature for Heat.
Change-Id: I06b669123fe08e02c66ee34dab78a943ff5de90c
Some users wish to override the default package provider by their own.
Tag all packages with the 'openstack' to allow mass resource attributes
override using resource collectors.
Closes-bug: #1391209
Change-Id: I09e54700438894e22d29605fec51bb056baf4050
Signed-off-by: Gael Chamoulaud <gchamoul@redhat.com>
This changes the puppet-lint requirement to 1.1.x, so that we can use
puppet-lint plugins. Most of these plugins are for 4.x compat, but some
just catch common errors.
Change-Id: If5f03538be85cee4a1d3b4c9a87eae1230432114
- This puppet-lint plugin checks if all parameters are documented
- Fix some unaligned arrows
- https://github.com/domcleal/puppet-lint-param-docs
Change-Id: I5e73747b726191bc4fc55e6e227892507e185871
Signed-off-by: Gael Chamoulaud <gchamoul@redhat.com>
This commit allow a deployer to manage the policies via this module
It relies on augeas to change only the policy needed. The init takes
a hash of policies and apply them.
Change-Id: If41f08571577bb799a373202dc58c2577bbe7f74
puppet-heat lacks of disabling service managing. This patch adds
$manage_service parameter to all relevant classes.
Closes-bug: #1359823
Change-Id: I54245b39f3484ccdb9910aa0fa4c8dc2bae3f0ce
This commit allows one to specify cert and key file
to run heat server in SSL mode.
Note: The flag use_ssl per se is not used in heat yet,
its purpose here it to verify collateral parameters.
Change-Id: I94d0b461adc752b028770aea71bf4e4612722539
Allow to configure the number of workers for Heat API services.
Change-Id: I94f938ede1f7f5e97adb406d592206e667d30f8d
Signed-off-by: Emilien Macchi <emilien.macchi@enovance.com>
This patch is aiming to correct a fault in puppet-heat that was not
executing the heat-manage db_sync that was making this module in the
end not fully working.
At this stage there are no tests to support this change but I am
willing to add those in a later commit.
This patch it's been rebased according to this change: https://review.openstack.org/46227
This new patchset include a patch from Sebastien Badia and should
make all the tests green with Change id: I2aa8fc070e5f5ee18040200e4a5b4f6161550280
Change-Id: Ib7731ab429ea92947f52bcf015facc8864466586
This patch is needed to completely remove any require heat::db that
point to heat::db that was removed in change:
I4584cdb12d1f9e624228e6ee34b4bcfbf649a12e
The patch is connected to the fact that in the cited change we
missed those references.
Change-Id: Idd9dcefd52c63e9c8cd74fc39bc4b7ba04f981bc
Upstream commits [1,2] modified heat to use a single config file.
This modifies the puppet config of each component to use different
sections of the same file.
Some notes about the change:
- each of heat-api, heat-api-cfn, heat-api-cloudwatch has indvidual
options for bind_port, bind_host
- heat-engine options live under DEFAULT (it does not bind to any
ports)
- new log_dir global
- slight fixes to ENCRYPTION_KEY generation for new heat.conf name
- include rspec tests for api-cfn & api-cloudwatch
Fixes bug 1214824
[1] https://review.openstack.org/#/c/39980/
[2] https://review.openstack.org/#/c/43697/
Change-Id: I7d8865a809da71696b944ab0654a1f728cd827df
Since we have now an only config file (heat.conf) with this patch:
https://review.openstack.org/#/c/36476/
This patch updates the module and delete old configuration files
support and add a test.
Also, I fixed some tab issues in manifests.
Fix bug #1207858
Change-Id: If3d044e2581156fa5ce4de19f5c740328efa0aa8
Takashi Kajinami