Skip to content

Commit 771e652

Browse files
authored
Fixing file URLs for WinRMExtension (#5485)
Fixing file URLs for WinRMExtension
1 parent cb4249e commit 771e652

File tree

9 files changed

+16
-25
lines changed

9 files changed

+16
-25
lines changed

Tasks/AzureFileCopy/Tests/L0AddAzureVMCustomScriptExtension.ps1

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ Assert-Throws {
2828
# Test 4 "Should fail to provision winrm custom script extension and remove the failed extension"
2929
$extensionName="WinRMCustomScriptExtension"
3030
Register-Mock Set-AzureMachineCustomScriptExtension {
31-
return Set-AzureMachineCustomScriptExtension -resourceGroupName $resourceGroupName -vmName $vmName -name $extensionName -fileUri $configWinRMScriptFile, $makeCertFile, $winrmConfFile -run $invalidCustomScriptName -argument $dnsName -location $location
31+
return Set-AzureMachineCustomScriptExtension -resourceGroupName $resourceGroupName -vmName $vmName -name $extensionName -fileUri $configWinRMScriptFile, $makeCertFile -run $invalidCustomScriptName -argument $dnsName -location $location
3232
} -ParametersEvaluator { $run -eq $scriptToRun }
3333

3434
Assert-Throws {

Tasks/AzureFileCopy/Tests/MockHelper.ps1

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -566,11 +566,6 @@ function Set-AzureMachineCustomScriptExtension
566566
throw "Cannot validate argument on parameter 'Location'. The argument is null or empty."
567567
}
568568

569-
if($fileUri.Count -eq 2)
570-
{
571-
$extensions[0]["SubStatuses"][1]["Message"]="'.\winrmconf.cmd' is not recognized as an internal or external command,\noperable program or batch file."
572-
$response["Status"]="Succeeded"
573-
}
574569
elseif($run -eq $invalidCustomScriptName)
575570
{
576571
$extensions[0]["SubStatuses"][1]["Message"]="The argument '$invalidCustomScriptName' to the -File parameter does not exist. Provide the path to an existing '.ps1' file as an argument to the -File parameter."

Tasks/AzureFileCopy/Utility.ps1

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1290,9 +1290,8 @@ function Add-AzureVMCustomScriptExtension
12901290
[string]$location,
12911291
[string]$connectedServiceName)
12921292

1293-
$configWinRMScriptFile="https://raw.githubusercontent.com/Azure/azure-quickstart-templates/master/201-vm-winrm-windows/ConfigureWinRM.ps1"
1294-
$makeCertFile="https://raw.githubusercontent.com/Azure/azure-quickstart-templates/master/201-vm-winrm-windows/makecert.exe"
1295-
$winrmConfFile="https://raw.githubusercontent.com/Azure/azure-quickstart-templates/master/201-vm-winrm-windows/winrmconf.cmd"
1293+
$configWinRMScriptFile="https://raw.githubusercontent.com/Azure/azure-quickstart-templates/501dc7d24537e820df7c80bce51aba9674233b2b/201-vm-winrm-windows/ConfigureWinRM.ps1"
1294+
$makeCertFile="https://raw.githubusercontent.com/Azure/azure-quickstart-templates/501dc7d24537e820df7c80bce51aba9674233b2b/201-vm-winrm-windows/makecert.exe"
12961295
$scriptToRun="ConfigureWinRM.ps1"
12971296
$extensionName="WinRMCustomScriptExtension"
12981297
$ruleName = "VSO-Custom-WinRM-Https-Port"
@@ -1317,7 +1316,7 @@ function Add-AzureVMCustomScriptExtension
13171316
return
13181317
}
13191318

1320-
$result = Set-AzureMachineCustomScriptExtension -resourceGroupName $resourceGroupName -vmName $vmName -name $extensionName -fileUri $configWinRMScriptFile, $makeCertFile, $winrmConfFile -run $scriptToRun -argument $dnsName -location $location
1319+
$result = Set-AzureMachineCustomScriptExtension -resourceGroupName $resourceGroupName -vmName $vmName -name $extensionName -fileUri $configWinRMScriptFile, $makeCertFile -run $scriptToRun -argument $dnsName -location $location
13211320
$resultDetails = $result | ConvertTo-Json
13221321
Write-Verbose "Set-AzureMachineCustomScriptExtension completed with response : $resultDetails"
13231322

Tasks/AzureFileCopy/task.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
"version": {
1414
"Major": 1,
1515
"Minor": 0,
16-
"Patch": 102
16+
"Patch": 103
1717
},
1818
"demands": [
1919
"azureps"
@@ -326,4 +326,4 @@
326326
"AFC_BlobStorageNotFound": "Storage account: {0} not found. Please specify existing storage account",
327327
"AFC_RootContainerAndDirectory": "'/S' option is not valid for $root containers."
328328
}
329-
}
329+
}

Tasks/AzureFileCopy/task.loc.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
"version": {
1414
"Major": 1,
1515
"Minor": 0,
16-
"Patch": 102
16+
"Patch": 103
1717
},
1818
"demands": [
1919
"azureps"

Tasks/AzureResourceGroupDeployment/Tests/mock_node_modules/azure-arm-compute/lib/operations/virtualMachineExtensions.js

Lines changed: 4 additions & 6 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Tasks/AzureResourceGroupDeployment/operations/WinRMExtensionHelper.ts

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -289,10 +289,9 @@ export class WinRMExtensionHelper {
289289

290290
private async AddWinRMExtension(vmId: string, vmName: string, dnsName: string, location: string) {
291291
var extensionName: string = "WinRMCustomScriptExtension";
292-
var configWinRMScriptFile: string = "https://raw.githubusercontent.com/Azure/azure-quickstart-templates/master/201-vm-winrm-windows/ConfigureWinRM.ps1";
293-
var makeCertFile: string = "https://raw.githubusercontent.com/Azure/azure-quickstart-templates/master/201-vm-winrm-windows/makecert.exe";
294-
var winrmConfFile: string = "https://raw.githubusercontent.com/Azure/azure-quickstart-templates/master/201-vm-winrm-windows/winrmconf.cmd";
295-
var fileUris = [configWinRMScriptFile, makeCertFile, winrmConfFile];
292+
var configWinRMScriptFile: string = "https://raw.githubusercontent.com/Azure/azure-quickstart-templates/501dc7d24537e820df7c80bce51aba9674233b2b/201-vm-winrm-windows/ConfigureWinRM.ps1";
293+
var makeCertFile: string = "https://raw.githubusercontent.com/Azure/azure-quickstart-templates/501dc7d24537e820df7c80bce51aba9674233b2b/201-vm-winrm-windows/makecert.exe";
294+
var fileUris = [configWinRMScriptFile, makeCertFile];
296295

297296
tl.debug("Adding custom script extension for virtual machine " + vmName);
298297
tl.debug("VM Location: " + location);

Tasks/AzureResourceGroupDeployment/task.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
"version": {
1515
"Major": 2,
1616
"Minor": 2,
17-
"Patch": 7
17+
"Patch": 8
1818
},
1919
"demands": [],
2020
"minimumAgentVersion": "2.119.1",

Tasks/AzureResourceGroupDeployment/task.loc.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
"version": {
1515
"Major": 2,
1616
"Minor": 2,
17-
"Patch": 7
17+
"Patch": 8
1818
},
1919
"demands": [],
2020
"minimumAgentVersion": "2.119.1",

0 commit comments

Comments
 (0)