Skip to content

Commit 3deed05

Browse files
Merge pull request #13210 from rawagner/vsphere_config_fix
OCPBUGS-25927: Fix config ini format
2 parents 0ed60b5 + 8281c37 commit 3deed05

3 files changed

Lines changed: 13 additions & 2 deletions

File tree

frontend/packages/vsphere-plugin/__tests__/__snapshots__/utils.spec.ts.snap

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ server=https://1.2.3.4/something
1212
datacenter=my-datacenter
1313
default-datastore=my-default-ds
1414
folder=/my/folder
15+
resourcepool-path=/my-datacenter/host/foo-cluster/Resources
1516
1617
[VirtualCenter "https://1.2.3.4/something"]
1718
datacenters=my-datacenter
@@ -28,6 +29,7 @@ server=https://1.2.3.4/something
2829
datacenter=my-datacenter
2930
default-datastore=my-default-ds
3031
folder=/my/folder
32+
resourcepool-path=/my-datacenter/host/foo-cluster/Resources
3133
3234
[VirtualCenter \\"https://1.2.3.4/something\\"]
3335
datacenters=my-datacenter
@@ -45,6 +47,7 @@ server=https://1.2.3.4/something
4547
datacenter=my-datacenter
4648
default-datastore=my-default-ds
4749
folder=/my/folder
50+
resourcepool-path=/my-datacenter/host/foo-cluster/Resources
4851
4952
[VirtualCenter \\"https://1.2.3.4/something\\"]
5053
datacenters=my-datacenter

frontend/packages/vsphere-plugin/src/components/persist.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -130,7 +130,7 @@ const persistProviderConfigMap = async (
130130
config: ConnectionFormContextValues,
131131
cloudProviderConfig?: ConfigMap,
132132
): Promise<string | undefined> => {
133-
const { vcenter, datacenter, defaultDatastore, folder } = config;
133+
const { vcenter, datacenter, defaultDatastore, folder, vCenterCluster } = config;
134134

135135
if (cloudProviderConfig) {
136136
const configIniString = mergeCloudProviderConfig(
@@ -172,6 +172,7 @@ server = "${vcenter}"
172172
datacenter = "${datacenter}"
173173
default-datastore = "${defaultDatastore}"
174174
folder = "${folder}"
175+
resourcepool-path = "/${datacenter}/host/${vCenterCluster}/Resources"
175176
176177
[VirtualCenter "${vcenter}"]
177178
datacenters = "${datacenter}"

frontend/packages/vsphere-plugin/src/components/utils.ts

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,13 @@ export const decodeBase64 = (data: string) => Buffer.from(data, 'base64').toStri
2727

2828
export const mergeCloudProviderConfig = (
2929
existingIni: string,
30-
{ vcenter, datacenter, defaultDatastore: defaultdatastore, folder }: ConnectionFormContextValues,
30+
{
31+
vcenter,
32+
datacenter,
33+
defaultDatastore: defaultdatastore,
34+
folder,
35+
vCenterCluster,
36+
}: ConnectionFormContextValues,
3137
): string => {
3238
const configIni = decode(existingIni);
3339

@@ -42,6 +48,7 @@ export const mergeCloudProviderConfig = (
4248
configIni.Workspace.datacenter = datacenter;
4349
configIni.Workspace['default-datastore'] = defaultdatastore;
4450
configIni.Workspace.folder = folder;
51+
configIni.Workspace['resourcepool-path'] = `/${datacenter}/host/${vCenterCluster}/Resources`;
4552

4653
Object.keys(configIni).forEach((k: string) => {
4754
if (k.startsWith('VirtualCenter')) {

0 commit comments

Comments
 (0)