Skip to content

Commit 9007a56

Browse files
author
Hovsep
committed
Merge pull request Azure#2241 from krkhan/dev
Update VMEncryption extension namespace, allow EncryptionDisable for Linux VMs
2 parents 046d63b + 8a7de83 commit 9007a56

File tree

2 files changed

+16
-13
lines changed

2 files changed

+16
-13
lines changed

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ namespace Microsoft.Azure.Commands.Compute.Extension.AzureDiskEncryption
2323
/// </summary>
2424
public class AzureDiskEncryptionExtensionContext : PSVirtualMachineExtension
2525
{
26-
public const string LinuxExtensionDefaultPublisher = "Microsoft.OSTCExtensions";
26+
public const string LinuxExtensionDefaultPublisher = "Microsoft.Azure.Security";
2727
public const string LinuxExtensionDefaultName = "AzureDiskEncryptionForLinux";
2828
public const string LinuxExtensionDefaultVersion = "0.1";
2929

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

Lines changed: 15 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -140,6 +140,21 @@ private VirtualMachineExtension GetVmExtensionParameters(VirtualMachine vmParame
140140
AutoUpgradeMinorVersion = !DisableAutoUpgradeMinorVersion.IsPresent
141141
};
142142
}
143+
if (OperatingSystemTypes.Linux.Equals(currentOSType))
144+
{
145+
this.Name = this.Name ?? AzureDiskEncryptionExtensionContext.LinuxExtensionDefaultName;
146+
147+
vmExtensionParameters = new VirtualMachineExtension
148+
{
149+
Location = vmParameters.Location,
150+
Publisher = AzureDiskEncryptionExtensionContext.LinuxExtensionDefaultPublisher,
151+
VirtualMachineExtensionType = AzureDiskEncryptionExtensionContext.LinuxExtensionDefaultName,
152+
TypeHandlerVersion = (this.TypeHandlerVersion) ?? AzureDiskEncryptionExtensionContext.LinuxExtensionDefaultVersion,
153+
Settings = SettingString,
154+
ProtectedSettings = ProtectedSettingString,
155+
AutoUpgradeMinorVersion = !DisableAutoUpgradeMinorVersion.IsPresent
156+
};
157+
}
143158

144159
return vmExtensionParameters;
145160
}
@@ -214,18 +229,6 @@ public override void ExecuteCmdlet()
214229
}
215230

216231
currentOSType = virtualMachineResponse.StorageProfile.OsDisk.OsType;
217-
if (OperatingSystemTypes.Linux.Equals(currentOSType))
218-
{
219-
ThrowTerminatingError(
220-
new ErrorRecord(
221-
new ArgumentException(
222-
string.Format(
223-
CultureInfo.CurrentUICulture,
224-
"Disable-AzureDiskEncryption cmdlet is supported only for Windows virtual machines")),
225-
"InvalidType",
226-
ErrorCategory.NotImplemented,
227-
null));
228-
}
229232

230233
if (this.Force.IsPresent ||
231234
this.ShouldContinue(Properties.Resources.DisableAzureDiskEncryptionConfirmation, Properties.Resources.DisableAzureDiskEncryptionCaption))

0 commit comments

Comments
 (0)