Skip to content

[libvirt_manager] Fix vm UUID #3567

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
evallesp merged 1 commit into from
Dec 19, 2025
Merged

[libvirt_manager] Fix vm UUID #3567

evallesp merged 1 commit into from
Dec 19, 2025
+1 −0

Conversation

@cjeanner
Copy link
Contributor

@cjeanner cjeanner commented Dec 17, 2025 •
edited
Loading

Fixing the UUID allows to get predictible value for that field. This is useful when working with GitOps principles and baremetal on virutal manchines, since the UUID is used by sushy-emulator to access the right VM.

Signed-off-by: Cédric Jeanneret cjeanner@redhat.com

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 17, 2025

[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 danpawlik 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

@cjeanner cjeanner force-pushed the libvirt-manager/fix-uuid branch from cd1f73e to fc38956 Compare December 17, 2025 12:28
@cjeanner cjeanner changed the title [Libvirt-manager] Fix vm UUID [libvirt-manager] Fix vm UUID Dec 17, 2025
@cjeanner
[libvirt_manager] Fix vm UUID
fb0d65d 
Fixing the UUID allows to get predictible value for that field. This is
useful when working with GitOps principles and baremetal on virutal
manchines, since the UUID is used by sushy-emulator to access the right
VM.

Signed-off-by: Cédric Jeanneret <cjeanner@redhat.com>
@cjeanner cjeanner force-pushed the libvirt-manager/fix-uuid branch from fc38956 to fb0d65d Compare December 17, 2025 12:29
@cjeanner cjeanner changed the title [libvirt-manager] Fix vm UUID [libvirt_manager] Fix vm UUID Dec 17, 2025
@softwarefactory-project-zuul
Copy link

softwarefactory-project-zuul bot commented Dec 17, 2025

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://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/a0c0a4c34f274f83b5d3a701d19f2b19

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 16m 08s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 29m 09s
❌ cifmw-crc-podified-edpm-baremetal FAILURE in 29m 43s
✔️ cifmw-crc-podified-edpm-baremetal-minor-update SUCCESS in 1h 56m 59s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 46s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 13s
✔️ cifmw-pod-pre-commit SUCCESS in 7m 57s
❌ build-push-container-cifmw-client FAILURE in 24m 11s
❌ cifmw-molecule-libvirt_manager FAILURE in 4m 28s
✔️ cifmw-molecule-reproducer SUCCESS in 14m 22s

@hjensas
Copy link
Contributor

hjensas commented Dec 18, 2025

recheck

@evallesp evallesp merged commit f95c9fa into openstack-k8s-operators:main Dec 19, 2025
9 checks passed
@cjeanner cjeanner deleted the libvirt-manager/fix-uuid branch January 5, 2026 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hjensas hjensas hjensas approved these changes

@evallesp evallesp evallesp approved these changes

Assignees

No one assigned

Labels

Ready For Review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@cjeanner @hjensas @evallesp