Skip to content

Commit e3adf1b

Browse files
committed
vSphere upi: change to infra id for folders
This commit changes from using the cluster name for folder creation naming to infraid. We think that the reuse of build cluster namespaces might be contributing to ci failures
1 parent 7c9b71a commit e3adf1b

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)