Add dcn_storage scenario for DCN adoption with Ceph HCI#3697
Add dcn_storage scenario for DCN adoption with Ceph HCI#3697fultonj wants to merge 1 commit intoopenstack-k8s-operators:mainfrom
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/9b385574906f4b3da67d3dd8b1162f3c ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 58m 58s |
fultonj
force-pushed
the
dcn_adoption_hci
branch
from
0f09936 to
1b02a0e
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/3e8db1cb29c840358716a3d81671f6e2 ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 09m 32s |
fultonj
force-pushed
the
dcn_adoption_hci
branch
2 times, most recently
from
a079ff6 to
0c8c064
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/6a3d5961addf4f318432d51aaac1373a ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 21m 57s |
| - name: Check glance config status on each controller | ||
| delegate_to: "{{ controller }}" | ||
| become: true | ||
| ansible.builtin.shell: | |
There was a problem hiding this comment.
Not sure there's a specific reason to use this shell task, but wouldn't be better to use ansible.builtin.stat and collect the nodes based on the stat.exists output (processed by L152)?
There was a problem hiding this comment.
Yes, I'll replace it with stat in my next patch set. This command used to grep the conf file to check if it was already updated with the desired glance config so NEEDS_UPDATE used to have more meaning than just file existence. Since that's gone this can now be simplified.
| ansible.builtin.shell: | ||
| cmd: | | ||
| set -o pipefail | ||
| grep -E '^enabled_backends\s*=' "{{ _glance_config_file }}" | head -1 | sed 's/enabled_backends\s*=\s*//' |
There was a problem hiding this comment.
not blocking, but in case you have time, consider consolidate grep + head + sed into a single command (e.g. based on awk).
This is really a nice to have that you can ignore.
There was a problem hiding this comment.
Good call. I put the following in my next patch set.
ansible.builtin.command:
cmd: awk -F'\s*=\s*' '/^enabled_backends\s*=/{print $2; exit}' "{{ _glance_config_file }}"
…Ceph HCI Add VM layout for DCN deployment with 3 Ceph clusters. Changes from dcn_nostorage: - Increase extra_disks_size from 15G to 30G for Ceph OSDs - All 9 compute nodes (central, dcn1, dcn2) configured for HCI - Stack name is passed to adoption ceph playbook - Fix os-net-config to use stack-specific ctlplane network (ctlplanedcn1/ctlplanedcn2) instead of hardcoded 'ctlplane' - Fix tripleo-ansible-inventory template computes group condition to match DCN group names like 'osp-dcn1-compute-az1s' which contain 'compute' but not 'computes' - Add adoption_dcn_export.yml playbook to export central stack for DCN site deployments - Add adoption_dcn_update_central.yml playbook to update central glance to use ceph at the DCN sites. This scenario pairs with data-plane-adoption's dcn_storage.yaml to deploy OSP 17.1 with Ceph at central and edge sites. Co-Authored-By: Claude <noreply@anthropic.com> Signed-off-by: John Fulton <fulton@redhat.com>
fultonj
force-pushed
the
dcn_adoption_hci
branch
from
0c8c064 to
bc3d58a
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/e6be698cf271445c919df9abf3b6a0ce ✔️ openstack-k8s-operators-content-provider SUCCESS in 27m 53s |
2 participants
Add VM layout for DCN deployment with 3 Ceph clusters.
Changes from dcn_nostorage:
This scenario pairs with data-plane-adoption's dcn_storage.yaml to deploy OSP 17.1 with Ceph at central and edge sites.
Related: openstack-k8s-operators/data-plane-adoption#1245