Skip to content

[ceph] Add thin-provisioned LV support for Ceph OSDs#3972

Open
bogdando wants to merge 1 commit into
openstack-k8s-operators:mainfrom
bogdando:bm_sno_ceph
Open

[ceph] Add thin-provisioned LV support for Ceph OSDs#3972
bogdando wants to merge 1 commit into
openstack-k8s-operators:mainfrom
bogdando:bm_sno_ceph

Conversation

@bogdando
Copy link
Copy Markdown
Contributor

@bogdando bogdando commented Jun 1, 2026

When cifmw_ceph_thin_pool is defined, create thin LVs from an existing VG thin pool instead of loop-backed files. This supports baremetal hosts where the disk is fully consumed by a thin pool with no raw devices available for cephadm.
The new play runs before the existing loop device play and sets cifmw_ceph_spec_data_devices with explicit LV paths, which also causes the loop device play to be skipped.

Related OSPRH-27841

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Jun 1, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign dasm for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@bogdando
Copy link
Copy Markdown
Contributor Author

bogdando commented Jun 1, 2026

tested this downstream for BM SNO DCN job. I will test this also with an hci uni job for regressions - together with #3773

@centosinfra-prod-github-app
Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/5b3f91a9e3834f2090d41e7be01d89b1

openstack-k8s-operators-content-provider FAILURE in 6m 33s
⚠️ podified-multinode-edpm-deployment-crc SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
⚠️ cifmw-crc-podified-edpm-baremetal SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
⚠️ cifmw-crc-podified-edpm-baremetal-minor-update SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
⚠️ podified-multinode-hci-deployment-crc SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 54s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 25s

@bogdando
Copy link
Copy Markdown
Contributor Author

bogdando commented Jun 1, 2026

Comment thread hooks/playbooks/ceph.yml
Comment thread hooks/playbooks/ceph.yml Outdated
Comment thread hooks/playbooks/ceph.yml Outdated
Comment thread hooks/playbooks/ceph.yml
When cifmw_ceph_thin_pool is defined, create thin LVs from an existing
VG thin pool instead of loop-backed files. This supports baremetal hosts
where the disk is fully consumed by a thin pool with no raw devices
available for cephadm.
The new play runs before the existing loop device play and sets
cifmw_ceph_spec_data_devices with explicit LV paths, which also
causes the loop device play to be skipped.

Signed-off-by: Bohdan Dobrelia <bdobreli@redhat.com>
@centosinfra-prod-github-app
Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/9e01ec97df7c4a2e80a2ca62b2009d9c

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 29m 41s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 27m 18s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 33m 25s
✔️ cifmw-crc-podified-edpm-baremetal-minor-update SUCCESS in 2h 13m 53s
podified-multinode-hci-deployment-crc FAILURE in 55m 39s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 10m 01s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 56s

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants