Skip to content

Commit 0791e81

Browse files
Merge pull request #10106 from jcpowermac/upi-folder-naming
OCPBUGS-65748: vSphere upi - change to infra id for folders
2 parents 4b8f8de + e3adf1b commit 0791e81

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

upi/vsphere/powercli/upi-functions.ps1

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -315,7 +315,7 @@ function New-OpenshiftVMs {
315315
$tag = Get-Tag -Server $node.server -Category $tagCategory -Name "$($metadata.infraID)" -ErrorAction continue 2>$null
316316

317317
# Get correct template / folder
318-
$folder = Get-Folder -Server $node.server -Name $clustername -Location $node.datacenter
318+
$folder = Get-Folder -Server $node.server -Name $metadata.infraID -Location $node.datacenter
319319
$template = Get-VM -Server $node.server -Name $vm_template -Location $($node.datacenter)
320320

321321
# Clone the virtual machine from the imported template
@@ -369,4 +369,4 @@ function Set-SecureBoot {
369369
$spec.BootOptions = $boot
370370

371371
$VM.ExtensionData.ReconfigVM($spec)
372-
}
372+
}

upi/vsphere/powercli/upi.ps1

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -177,13 +177,13 @@ foreach ($fd in $fds)
177177

178178
# If the folder already exists
179179
Write-Output "Checking for folder in failure domain $($fd.datacenter)/$($fd.cluster)"
180-
$folder = Get-Folder -Server $viserver -Name $clustername -Location $fd.datacenter -ErrorAction continue 2>$null
180+
$folder = Get-Folder -Server $viserver -Name $metadata.infraID -Location $fd.datacenter -ErrorAction continue 2>$null
181181

182182
# Otherwise create the folder within the datacenter as defined in the upi-variables
183183
if (-Not $?) {
184184
Write-Output "Creating folder $($clustername) in datacenter $($fd.datacenter) of vCenter $($fd.server)"
185-
(get-view (Get-Datacenter -Server $viserver -Name $fd.datacenter).ExtensionData.vmfolder).CreateFolder($clustername)
186-
$folder = Get-Folder -Server $viserver -Name $clustername -Location $fd.datacenter
185+
(get-view (Get-Datacenter -Server $viserver -Name $fd.datacenter).ExtensionData.vmfolder).CreateFolder($metadata.infraID)
186+
$folder = Get-Folder -Server $viserver -Name $metadata.infraID -Location $fd.datacenter
187187
New-TagAssignment -Server $viserver -Entity $folder -Tag $tag > $null
188188
}
189189

@@ -257,7 +257,7 @@ $tagCategory = Get-TagCategory -Server $fds[0].server -Name "openshift-$($metada
257257
$tag = Get-Tag -Server $fds[0].server -Category $tagCategory -Name "$($metadata.infraID)" -ErrorAction continue 2>$null
258258
$rp = Get-ResourcePool -Name $($metadata.infraID) -Location $(Get-Cluster -Server $fds[0].server -Name $($fds[0].cluster))
259259
$datastoreInfo = Get-Datastore -Name $fds[0].datastore -Server $fds[0].server -Location $fds[0].datacenter
260-
$folder = Get-Folder -Server $fds[0].server -Name $clustername -Location $fds[0].datacenter
260+
$folder = Get-Folder -Server $fds[0].server -Name $metadata.infraID -Location $fds[0].datacenter
261261
$template = Get-VM -Server $fds[0].server -Name $vm_template -Location $fds[0].datacenter
262262

263263
# Create LB for Cluster

0 commit comments

Comments
 (0)