Skip to content

Commit

Permalink
Merge pull request #358 from fultonj/OSPRH-9222
Browse files Browse the repository at this point in the history
Use pre/post Ceph servicesOverride in uni05epsilon HCI

This patch applies very similar changes as the commit below for the same reasons but to uni05epsilon.
commit 3ebad70
Jira: https://issues.redhat.com/browse/OSPRH-9222

Reviewed-by: Andrew Bays <[email protected]>
  • Loading branch information
softwarefactory-project-zuul[bot] authored Aug 13, 2024
2 parents 710ac6c + ae17388 commit de3eca1
Show file tree
Hide file tree
Showing 7 changed files with 71 additions and 1 deletion.
9 changes: 9 additions & 0 deletions examples/dt/uni05epsilon/dataplane-post-ceph.md
Original file line number Diff line number Diff line change
Expand Up @@ -114,3 +114,12 @@ Wait for post-Ceph dataplane deployment to finish.
```bash
oc wait osdpd edpm-deployment-post-ceph --for condition=Ready --timeout=40m
```

## Final OpenStackDataPlaneNodeSet services list

The `OpenStackDataPlaneNodeSet` must contain the full `services` list
so that during updates all required services are updated. Thus, the
pre-ceph and post-ceph deployments used a `servicesOverride` so that
only a subset of the services would be configured either before or
after Ceph was deployed. Any subsequent deployments should not pass a
`servicesOverride` unless necessary.
13 changes: 13 additions & 0 deletions examples/dt/uni05epsilon/deployment/kustomization.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,3 +7,16 @@ components:

resources:
- values.yaml

replacements:
- source:
kind: ConfigMap
name: edpm-deployment-values-post-ceph
fieldPath: data.servicesOverride
targets:
- select:
kind: OpenStackDataPlaneDeployment
fieldPaths:
- spec.servicesOverride
options:
create: true
7 changes: 7 additions & 0 deletions examples/dt/uni05epsilon/deployment/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,3 +9,10 @@ metadata:
data:
deployment:
name: edpm-deployment-post-ceph
servicesOverride:
- install-certs
- ceph-client
- ovn
- neutron-metadata
- libvirt
- nova-custom
Original file line number Diff line number Diff line change
Expand Up @@ -7,3 +7,16 @@ components:

resources:
- values.yaml

replacements:
- source:
kind: ConfigMap
name: edpm-deployment-values
fieldPath: data.servicesOverride
targets:
- select:
kind: OpenStackDataPlaneDeployment
fieldPaths:
- spec.servicesOverride
options:
create: true
11 changes: 11 additions & 0 deletions examples/dt/uni05epsilon/edpm-pre-ceph/deployment/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,3 +9,14 @@ metadata:
data:
deployment:
name: edpm-deployment-pre-ceph
servicesOverride:
- bootstrap
- download-cache
- configure-network
- validate-network
- install-os
- ceph-hci-pre
- configure-os
- ssh-known-hosts
- run-os
- reboot-os
9 changes: 8 additions & 1 deletion examples/dt/uni05epsilon/edpm-pre-ceph/nodeset/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -151,7 +151,9 @@ data:
subnetName: subnet1
- name: tenant
subnetName: subnet1

# The nova-custom service is omitted since it is not yet
# defined. It will be defined and set after Ceph is deployed.
# See deployment servicesOverride for effective services list.
services:
- bootstrap
- download-cache
Expand All @@ -163,3 +165,8 @@ data:
- ssh-known-hosts
- run-os
- reboot-os
- install-certs
- ceph-client
- ovn
- neutron-metadata
- libvirt
10 changes: 10 additions & 0 deletions examples/dt/uni05epsilon/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,16 @@ data:

nodeset:
services:
- bootstrap
- download-cache
- configure-network
- validate-network
- install-os
- ceph-hci-pre
- configure-os
- ssh-known-hosts
- run-os
- reboot-os
- install-certs
- ceph-client
- ovn
Expand Down

0 comments on commit de3eca1

Please sign in to comment.