This enables the ironic::nova class, to manage [nova] options in
ironic.conf, to disable/enable power notifications from ironic to
nova.
Partial-Bug: #2000308
Change-Id: I273524330c11910e15f8eccdb9025db6e7c881e1
This change enables the separate class to manage the [disk_utils]
paramers so that users can use puppet parameters to manage options
in this section.
Depends-on: https://review.opendev.org/823903
Depends-on: https://review.opendev.org/850298
Change-Id: I6d0fd53d0785a5d7428de6aee74a5db9d7d239b7
The hiera function is deprecated and does not work with the latest
hieradata version 5. It should be replaced by the new lookup
function[1].
[1] https://puppet.com/docs/puppet/7/hiera_automatic.html
With the lookup function, we can define value type and merge behavior,
but these are kept default at this moment to limit scope of this change
to just simple replacement. Adding value type might be useful to make
sure the value is in expected type (especially when a boolean value is
expected), but we will revisit that later.
example:
lookup(<NAME>, [<VALUE TYPE>], [<MERGE BEHAVIOR>], [<DEFAULT VALUE>])
Change-Id: Ie280f3dcab6c887c3e5a6662c8d05c8fe98a8cf9
Configure json_rpc section for both ironic-api and ironic-conductor.
JSON RPC service uses keystonemiddleware to authenticate with
keystone. Configure the keystone_authtoken section for
ironic-conductor as well.
Change-Id: I9fc0f27a781e6374ad52575533c099fbc23382fd
Downcase in puppet 6.14 throws an error if the input to it is Undef. We
can avoid this by checking for a value before trying to downcase.
See context https://review.rdoproject.org/r/#/c/26297/
Change-Id: Ib2e97060523a4198a14949a15c9171b56928699c
When Ironic Conductor class is called, it expects the
PXE directories exist. That is only the case for the step 4.
While there is also a case when the conductor class invoked
for the step 3 & db sync case. For that case also inlcude
the missing ironic::pxe class to ensure the PXE directories
created.
Closes-Bug: #1845222
Change-Id: I394f56ba9b213c75378bdf21999d23509632523c
Signed-off-by: Bogdan Dobrelya <bdobreli@redhat.com>
* add ironic::drivers::staging to enable use of staging drivers
(disabled by default)
* remove mentions of SSH drivers (removed upstream, similar
functionality is provided by staging drivers)
Implements: blueprint ovirt-driver
Change-Id: I89da8187c2c549ad2f6d569890080ce144be46c0
Depends-On: Ibef8f33f306b9d3b598698345ce3b0978f62a2ad
The step is typically set with the hieradata setting an integer value:
{"step": 1}
However it would be useful for the value to be a string so that
substitutions are possible, for example:
{"step": "%{::step}"}
This change ensures the step parameter defaults to an integer by
calling Integer(hiera('step'))
This change was made by manually removing the undef defaults from
fluentd.pp, uchiwa.pp, and sensu.pp then bulk updating with:
find ./ -type f -print0 |xargs -0 sed -i "s/= hiera('step')/= Integer(hiera('step'))/"
Change-Id: I8a47ca53a7dea8391103abcb8960a97036a6f5b3
The SSH drivers are deprecated, pxe_ipmitool + virtualbmc should be used instead.
This is a follow-up to blueprint switch-to-virtualbmc.
Change-Id: I4fd567dffa3992042eebcf495334b8130e1bdc9f
This enables configuring new-style drivers (aha hardware types).
Part of blueprint ironic-driver-composition
Change-Id: I72eb8b06cca14073d1d1c82462fb702630e02de3
Using keystone_authtoken credentials for this purpose is deprecated, and also
prevents ironic-conductor from being used as a separate role.
As a side effect, this change makes it possible to potentially enable
ironic-inspector support in the future (it's not enabled yet).
Change-Id: I21180678bec911f1be36e3b174bae81af042938c
Partial-Bug: #1661250
Configure drac, ilo and ssh drivers out of box.
Remove deprecated ironic::drivers::deploy.
This change brings the default driver list closer to one of undercloud.
Change-Id: I8b9a136a0ff22916d7c468bbb0df7248bc35a5c2
Partially-Implements: blueprint ironic-integration
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