Skip to content
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.

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

NAS-132946 / 25.04 / Complete VM support in virt plugin #15151

Merged
merged 1 commit into from
Dec 18, 2024

Conversation

Qubad786
Copy link
Contributor

@Qubad786 Qubad786 commented Dec 6, 2024

Context

VM support has been added to virt plugin which allows specifying VM based images to be used for VM creation or not using a source image at VM creation and later using an ISO to bootstrap the VM.
For an ISO to be used with a VM, the iso needs to be imported as an incus volume and then that incus volume attached to the virt instance. A CRUD for virt volumes has been added and the ability to import iso as a virt volume which then can later be attached to virt VM instance. Virt volumes are currently only supported for VMs.

While we are here, validation has been improved and some bug fixes have been done wrt USB/GPU devices.

@bugclerk
Copy link
Contributor

bugclerk commented Dec 6, 2024

@bugclerk bugclerk changed the title Complete VM support in virt plugin NAS-132946 / 25.04 / Complete VM support in virt plugin Dec 6, 2024
@bugclerk
Copy link
Contributor

bugclerk commented Dec 6, 2024

@Qubad786 Qubad786 force-pushed the mrehan/virt-vm-support branch 7 times, most recently from 7743d57 to 7f48baa Compare December 12, 2024 17:26
@Qubad786 Qubad786 force-pushed the mrehan/virt-vm-support branch from 7f48baa to e16d7d1 Compare December 15, 2024 12:45
@Qubad786 Qubad786 removed the WIP label Dec 15, 2024
@Qubad786 Qubad786 requested review from a team and william-gr December 15, 2024 13:03
@Qubad786 Qubad786 force-pushed the mrehan/virt-vm-support branch from e16d7d1 to d8d0bc6 Compare December 16, 2024 11:00
Copy link
Member

@william-gr william-gr left a comment

Choose a reason for hiding this comment

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

Looking pretty good! Just some minor concerns

@Qubad786 Qubad786 force-pushed the mrehan/virt-vm-support branch from d8d0bc6 to dd96008 Compare December 17, 2024 20:22
@Qubad786 Qubad786 added the WIP label Dec 17, 2024
@Qubad786 Qubad786 force-pushed the mrehan/virt-vm-support branch 2 times, most recently from d41e676 to 7cdfa15 Compare December 18, 2024 16:38
@Qubad786 Qubad786 removed the WIP label Dec 18, 2024
@Qubad786 Qubad786 requested a review from william-gr December 18, 2024 16:38
Copy link
Member

@william-gr william-gr left a comment

Choose a reason for hiding this comment

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

Awesome, thank you!

@Qubad786 Qubad786 force-pushed the mrehan/virt-vm-support branch from 7cdfa15 to 698ca9b Compare December 18, 2024 17:09
@Qubad786 Qubad786 force-pushed the mrehan/virt-vm-support branch from 698ca9b to d2355e4 Compare December 18, 2024 18:35
@Qubad786 Qubad786 merged commit 48ed775 into master Dec 18, 2024
2 checks passed
@Qubad786 Qubad786 deleted the mrehan/virt-vm-support branch December 18, 2024 21:17
@bugclerk
Copy link
Contributor

This PR has been merged and conversations have been locked.
If you would like to discuss more about this issue please use our forums or raise a Jira ticket.

@truenas truenas locked as resolved and limited conversation to collaborators Dec 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants