Skip to content

Commit 8259119

Browse files
author
Kamran Khan
committed
Do not update VM encryption settings if extension install fails
1 parent 2e9f208 commit 8259119

File tree

1 file changed

+9
-1
lines changed

1 file changed

+9
-1
lines changed

src/ResourceManager/Compute/Commands.Compute/Extension/AzureDiskEncryption/SetAzureDiskEncryptionExtension.cs

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -444,12 +444,20 @@ public override void ExecuteCmdlet()
444444

445445
VirtualMachineExtension parameters = GetVmExtensionParameters(virtualMachineResponse);
446446

447-
this.VirtualMachineExtensionClient.CreateOrUpdateWithHttpMessagesAsync(
447+
AzureOperationResponse<VirtualMachineExtension> extensionPushResult = this.VirtualMachineExtensionClient.CreateOrUpdateWithHttpMessagesAsync(
448448
this.ResourceGroupName,
449449
this.VMName,
450450
this.Name,
451451
parameters).GetAwaiter().GetResult();
452452

453+
if (!extensionPushResult.Response.IsSuccessStatusCode)
454+
{
455+
ThrowTerminatingError(new ErrorRecord(new ApplicationException(string.Format(CultureInfo.CurrentUICulture, "Installation failed for extension {0}", parameters.VirtualMachineExtensionType)),
456+
"InvalidResult",
457+
ErrorCategory.InvalidResult,
458+
null));
459+
}
460+
453461
var op = UpdateVmEncryptionSettings();
454462
var result = Mapper.Map<PSAzureOperationResponse>(op);
455463
WriteObject(result);

0 commit comments

Comments
 (0)