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
Add Vm.update_spdk_version(...) #1478
Conversation
8b7a669
to
ccf3b82
Compare
ee2c35b
to
ef5ee48
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
alright, though regenerating the unit file is a step into a different world just a little bit, where can't get some implied backwards compatibility anymore. We'll need to keep the re-generation in order with all extant VMs.
For example, if we move to using hotplug to set up cloud-hypervisor rather than command line options, there will be new considerations.
Not a blocker, but it's a transition.
6ba406d
to
4be85f3
Compare
When changing the SPDK version of a VM, we need to reinstall these to update these, as VM services have dependencies to SPDK.
ef5ee48
to
3943feb
Compare
edd5fb3
to
b4edf29
Compare
Previously to change SPDK version of a VM we needed to do multiple weird manual steps. This patch automates some of them and what we need to do is: 1. Install the new SPDK version on the host: ``` > Prog::Storage::SetupSpdk.assemble( vm_host.id, "some-spdk-version", start_service: true, allocation_weight: 100) ``` 2. Update all VMs we want to migrate to this SPDK version. Note that this will just update related configs and won't take effect until host reboot: ``` > vm.update_spdk_version("some-spdk-version") ``` 3. Reboot host: ``` > vm_host.incr_reboot ``` 4. Optionally, remove the old SPDK installation: ``` > Prog::Storage::RemoveSpdk.assemble(old_spdk_installation.id) ```
b4edf29
to
22d7814
Compare
Previously to change SPDK version of a VM we needed to do multiple weird manual steps. This patch automates some of them and what we need to do is: