Support for storage space data disks #998
Conversation
internal/hcsoci/hcsdoc_wcow.go
Outdated
| } | ||
| mdv2.HostPath = src | ||
| } else if mount.Type == "virtual-disk" || mount.Type == "physical-disk" || mount.Type == "ExtensibleVirtualDisk" { | ||
|
|
There was a problem hiding this comment.
Is it ok to eliminate the existing behavior of always checking a vSMB mount first regardless of mount.Type?
There was a problem hiding this comment.
I think so. We currently have a bug that doesn't allow using the same host path as both scsi and vsmb mount because of this.
|
I think it would be more useful long-term if we can just make this feature be generic EVD support. e.g. instead of |
|
@kevpar I like the idea of not having to make any other changes to support other types of EVDs. I just didn't do this because I didn't know if we wanted to keep the EVD part as an implementation detail. |
ambarve
force-pushed
the
storage_spaces
branch
2 times, most recently
from
5702964 to
3b963d1
Compare
|
This needs a rebase to grab the schema location changes (sorry 😞) |
| // TODO: Mapped pipes to add in v2 schema. | ||
| var config mountsConfig | ||
| for _, mount := range coi.Spec.Mounts { | ||
| if mount.Type != "" { |
There was a problem hiding this comment.
Do we know why this check was here in the first place?
When mounting a VHD at host path `C:\data\test.vhdx` into the container over SCSI and also sharing the same VHD inside the container over VSMB the current code just shares the VHD inside the container for both mounts instead of actually SCSI mounting the VHD for one of the mounts. This change fixes that. Signed-off-by: Amit Barve <[email protected]>
5 participants
This commit adds support in hcsshim to mount a virtual disk backed by storage spaces as a
data disk into a container. In container config the
host_pathin the mount entryshould use the format
space://{storage_space_pool_guid}{storage_space_disk_guid}tospecify a storage space virtual disk.
Signed-off-by: Amit Barve [email protected]