Skip to content

Commit e8e30bc

Browse files
committed
capv - missing cores/socket,disk type
When implementing capv changes for the installer we missed two features. Enabling the quantity of cores per socket and the disk type of the template (and that of the cloned virtual machines). This PR resolves those two issues.
1 parent 8310b10 commit e8e30bc

File tree

2 files changed

+23
-9
lines changed

2 files changed

+23
-9
lines changed

pkg/asset/machines/vsphere/capimachines.go

Lines changed: 10 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -125,15 +125,16 @@ func GenerateMachines(ctx context.Context, clusterID string, config *types.Insta
125125
Network: capv.NetworkSpec{
126126
Devices: capvNetworkDevices,
127127
},
128-
Folder: providerSpec.Workspace.Folder,
129-
Template: providerSpec.Template,
130-
Datacenter: providerSpec.Workspace.Datacenter,
131-
Server: providerSpec.Workspace.Server,
132-
NumCPUs: providerSpec.NumCPUs,
133-
MemoryMiB: providerSpec.MemoryMiB,
134-
DiskGiB: providerSpec.DiskGiB,
135-
Datastore: providerSpec.Workspace.Datastore,
136-
ResourcePool: resourcePool,
128+
Folder: providerSpec.Workspace.Folder,
129+
Template: providerSpec.Template,
130+
Datacenter: providerSpec.Workspace.Datacenter,
131+
Server: providerSpec.Workspace.Server,
132+
NumCPUs: providerSpec.NumCPUs,
133+
NumCoresPerSocket: providerSpec.NumCoresPerSocket,
134+
MemoryMiB: providerSpec.MemoryMiB,
135+
DiskGiB: providerSpec.DiskGiB,
136+
Datastore: providerSpec.Workspace.Datastore,
137+
ResourcePool: resourcePool,
137138
},
138139
},
139140
}

pkg/infrastructure/vsphere/clusterapi/import.go

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -98,6 +98,19 @@ func importRhcosOva(ctx context.Context, session *session.Session, folder *objec
9898
NetworkMapping: networkMappings,
9999
}
100100

101+
switch diskProvisioningType {
102+
case "":
103+
// Disk provisioning type will be set according to the default storage policy of vsphere.
104+
case "thin":
105+
cisp.DiskProvisioning = string(types.OvfCreateImportSpecParamsDiskProvisioningTypeThin)
106+
case "thick":
107+
cisp.DiskProvisioning = string(types.OvfCreateImportSpecParamsDiskProvisioningTypeThick)
108+
case "eagerZeroedThick":
109+
cisp.DiskProvisioning = string(types.OvfCreateImportSpecParamsDiskProvisioningTypeEagerZeroedThick)
110+
default:
111+
return errors.Errorf("disk provisioning type %q is not supported", diskProvisioningType)
112+
}
113+
101114
m := ovf.NewManager(session.Client.Client)
102115
spec, err := m.CreateImportSpec(ctx,
103116
string(ovfDescriptor),

0 commit comments

Comments
 (0)