Skip to content

[bm_sno] Allow SNO deploy for LVMS (leave free space)#3971

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

[bm_sno] Allow SNO deploy for LVMS (leave free space)#3971
bogdando wants to merge 1 commit into
openstack-k8s-operators:mainfrom
bogdando:lvms_bm_sno_edpm

Conversation

@bogdando

@bogdando bogdando commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

Modify bm_sno role to let it provision thin volumes by LVM Storage operator for a proper bare metal performance in BM SNO CI jobs.

Related: OSPRH-26767

@centosinfra-prod-github-app

centosinfra-prod-github-app Bot commented Jun 1, 2026

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/25be0f851c1a4c7ebbaed857e6a96dc8

❌ openstack-k8s-operators-content-provider FAILURE in 6m 41s
⚠️ 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
✔️ cifmw-pod-zuul-files SUCCESS in 5m 14s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 20s
✔️ cifmw-pod-pre-commit SUCCESS in 9m 00s
✔️ cifmw-molecule-bm_sno SUCCESS in 4m 51s

@bogdando

bogdando commented Jun 1, 2026

Copy link
Copy Markdown
Contributor Author

recheck openstack-k8s-operators-content-provider

@centosinfra-prod-github-app

centosinfra-prod-github-app Bot commented Jun 1, 2026

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/c5d9575cd8a34755bba7180093225039

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 09m 14s
❌ podified-multinode-edpm-deployment-crc RETRY_LIMIT in 6m 58s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 40m 17s
✔️ cifmw-crc-podified-edpm-baremetal-minor-update SUCCESS in 1h 57m 04s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 43s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 45s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 31s
✔️ cifmw-molecule-bm_sno SUCCESS in 4m 52s

@bogdando

bogdando commented Jun 1, 2026

Copy link
Copy Markdown
Contributor Author

recheck podified-multinode-edpm-deployment-crc

@centosinfra-prod-github-app

centosinfra-prod-github-app Bot commented Jun 1, 2026

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/abf30aabfb9e463f82c8b3de68688d87

❌ openstack-k8s-operators-content-provider FAILURE in 6m 51s
⚠️ 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
✔️ cifmw-pod-zuul-files SUCCESS in 6m 15s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 10m 07s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 22s
✔️ cifmw-molecule-bm_sno SUCCESS in 4m 53s

@bogdando bogdando force-pushed the lvms_bm_sno_edpm branch from 72fea71 to 18f2039 Compare June 3, 2026 15:52
@centosinfra-prod-github-app

centosinfra-prod-github-app Bot commented Jun 3, 2026

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/0821e0010ca544909b8e13d83a8392f8

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 27m 43s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 24m 44s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 35m 41s
❌ cifmw-crc-podified-edpm-baremetal-minor-update FAILURE in 2h 13m 36s
✔️ cifmw-pod-zuul-files SUCCESS in 5m 20s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 39s
✔️ cifmw-pod-pre-commit SUCCESS in 9m 27s
✔️ cifmw-molecule-bm_sno SUCCESS in 5m 02s

@bogdando

bogdando commented Jun 5, 2026

Copy link
Copy Markdown
Contributor Author

recheck cifmw-crc-podified-edpm-baremetal-minor-update

@bogdando bogdando force-pushed the lvms_bm_sno_edpm branch from 18f2039 to 440e0bc Compare June 5, 2026 14:56
@fultonj

fultonj commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

FWIW I have no objection to this change.

@bogdando bogdando requested review from arxcruz and evallesp June 8, 2026 11:35
Valkyrie00
Valkyrie00 reviewed Jun 8, 2026
View reviewed changes
Comment thread roles/bm_sno/README.md
Valkyrie00
Valkyrie00 reviewed Jun 8, 2026
View reviewed changes
Comment thread roles/bm_sno/templates/lvms_partition_machineconfig.yaml.j2 Outdated
Valkyrie00
Valkyrie00 reviewed Jun 8, 2026
View reviewed changes
Comment thread roles/bm_sno/tasks/main.yml
@bogdando bogdando force-pushed the lvms_bm_sno_edpm branch from 440e0bc to 59abffa Compare June 8, 2026 12:50
@bogdando bogdando force-pushed the lvms_bm_sno_edpm branch from 59abffa to 0e8b8b1 Compare June 8, 2026 14:55
@bogdando
[bm_sno] Allow SNO deploy for LVMS (leave free space)
997d53f 
Modify bm_sno role to let it provision thin volumes by
LVM Storage operator for a proper bare metal performance in BM
SNO CI jobs.

Signed-off-by: Bohdan Dobrelia <bdobreli@redhat.com>
@bogdando bogdando force-pushed the lvms_bm_sno_edpm branch from 0e8b8b1 to 997d53f Compare June 8, 2026 14:56
Valkyrie00
Valkyrie00 approved these changes Jun 8, 2026
View reviewed changes

@Valkyrie00 Valkyrie00 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci openshift-ci Bot added the lgtm label Jun 8, 2026
evallesp
evallesp approved these changes Jun 8, 2026
View reviewed changes

@evallesp evallesp left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/approve

@openshift-ci

openshift-ci Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: evallesp

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

The pull request process is described 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

@openshift-ci openshift-ci Bot added the approved label Jun 8, 2026
@bogdando

bogdando commented Jun 8, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

testing this more (not related to this PR but disk paths values on my setup...)

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

Reviewers

@Valkyrie00 Valkyrie00 Valkyrie00 approved these changes

@evallesp evallesp evallesp approved these changes

@arxcruz arxcruz Awaiting requested review from arxcruz

Assignees

@Valkyrie00 Valkyrie00

Labels

approved do-not-merge/work-in-progress lgtm Ready For Review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@bogdando @fultonj @Valkyrie00 @evallesp