Set Cinder customServiceConfig globally in DCN DT #459
Conversation
fultonj
force-pushed
the
OSPRH-11915
branch
2 times, most recently
from
923d166 to
572aacf
Compare
fultonj
requested review from
krcmarik and
sbekkerm
and removed request for
cjeanner
The customServiceConfig for cinder in the DCN DT is used to set the storage_availability_zone. We wish to do this for all Cinder services in the default site (`az0`), not just for cinderAPI or cinderBackup as the AZ was missing from the cinder-scheduler. Thus, we will move it to the global section under the cinder template and out of the cinderAPI and cinderBackup since they will inherit it. We still need to set the backend_availability_zone in all cinderVolumes except az0 since it will also inherit the new default value. However, the additional line is kept in az0's customServiceConfig to make the pattern easier to see for az0, az1, az2, etc. Jira: https://issues.redhat.com/browse/OSPRH-11915 Signed-off-by: John Fulton <[email protected]>
|
This has passed a CI test and is ready to merge. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: fultonj, krcmarik The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
Build succeeded (gate pipeline). ✔️ noop SUCCESS in 0s |
softwarefactory-project-zuul
bot
merged commit 5576566
into
openstack-k8s-operators:main
|
/cherry-pick 18.0-fr1 |
|
@fultonj: new pull request created: #463 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
…59-to-18.0-fr1 [18.0-fr1] Set Cinder customServiceConfig globally in DCN DT This is an automated cherry-pick of #459 /assign fultonj Reviewed-by: Andrew Bays <[email protected]>
The customServiceConfig for cinder in the DCN DT is used to set the storage_availability_zone. We wish to do this for all Cinder services in the default site (
az0), not just for cinderAPI or cinderBackup as the AZ was missing from the cinder-scheduler. Thus, we will move it to the global section under the cinder template and out of the cinderAPI and cinderBackup since they will inherit it. We still need to set the backend_availability_zone in all cinderVolumes except az0 since it will also inherit the new default value. However, the additional line is kept in az0's customServiceConfig to make the pattern easier to see for az0, az1, az2, etc.Jira: https://issues.redhat.com/browse/OSPRH-11915