Skip to content

Commit

Permalink
vol-mig: validate hotplug volumes
Browse files Browse the repository at this point in the history
Make the hotplug volumes entry positive since now they are a valid
option for volume migration.

Signed-off-by: Alice Frosi <[email protected]>
alicefr committed Nov 14, 2024
1 parent da1d00d commit b5d68e0
Showing 1 changed file with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
@@ -81,11 +81,11 @@ var _ = Describe("Volume Migration", func() {
), libvmi.NewVirtualMachine(libvmi.New(
libvmi.WithPersistentVolumeClaim("disk0", "vol2"), withFilesystemVolume("disk1", "vol4"),
)), fmt.Errorf("invalid volumes to update with migration: filesystems: [disk1]")),
Entry("with an invalid hotplugged volume", libvmi.New(
libvmi.WithPersistentVolumeClaim("disk0", "vol0"), withFilesystemVolume("disk1", "vol1"),
Entry("with valid hotplugged volume", libvmi.New(
libvmi.WithPersistentVolumeClaim("disk0", "vol0"), withHotpluggedVolume("disk1", "vol1"),
), libvmi.NewVirtualMachine(libvmi.New(
libvmi.WithPersistentVolumeClaim("disk0", "vol2"), withHotpluggedVolume("disk1", "vol4"),
)), fmt.Errorf("invalid volumes to update with migration: hotplugged: [disk1]")),
)), nil),
)
})

0 comments on commit b5d68e0

Please sign in to comment.