- removed Status parameter, we always need extension status retrieved back to read AutoBackup and AutoPatching settings

Author: sethusrinivasan

src/ResourceManager/Compute/Commands.Compute/Extension/SqlServer/GetAzureVMSqlServerExtensionCommand.cs

@@ -57,13 +57,6 @@ public class GetAzureVMSqlServerExtensionCommand : VirtualMachineExtensionBaseCm
         [ValidateNotNullOrEmpty]
         public string Name { get; set; }
 
-        [Parameter(
-            Position = 3,
-            ValueFromPipelineByPropertyName = true,
-            HelpMessage = "To show the status.")]
-        [ValidateNotNullOrEmpty]
-        public SwitchParameter Status { get; set; }
-
         public override void ExecuteCmdlet()
         {
             base.ExecuteCmdlet();
@@ -73,45 +66,21 @@ public override void ExecuteCmdlet()
                 Name = VirtualMachineSqlServerExtensionContext.ExtensionPublishedNamespace + "." + VirtualMachineSqlServerExtensionContext.ExtensionPublishedName;
             }
 
-            if (Status)
-            {
-                var result = VirtualMachineExtensionClient.GetWithInstanceView(ResourceGroupName, VMName, Name);
-                var extension = result.ToPSVirtualMachineExtension(ResourceGroupName);
+            var result = VirtualMachineExtensionClient.GetWithInstanceView(ResourceGroupName, VMName, Name);
+            var extension = result.ToPSVirtualMachineExtension(ResourceGroupName);
 
-                if (
-                    extension.Publisher.Equals(VirtualMachineSqlServerExtensionContext.ExtensionPublishedNamespace,
-                        StringComparison.InvariantCultureIgnoreCase) &&
-                    extension.ExtensionType.Equals(VirtualMachineSqlServerExtensionContext.ExtensionPublishedName,
-                        StringComparison.InvariantCultureIgnoreCase))
-                {
-                    WriteObject(GetSqlServerExtensionContext(extension));
-                }
-                else
-                {
-                    WriteObject(null);
-                }
+            if (
+                extension.Publisher.Equals(VirtualMachineSqlServerExtensionContext.ExtensionPublishedNamespace,
+                    StringComparison.InvariantCultureIgnoreCase) &&
+                extension.ExtensionType.Equals(VirtualMachineSqlServerExtensionContext.ExtensionPublishedName,
+                    StringComparison.InvariantCultureIgnoreCase))
+            {
+                WriteObject(GetSqlServerExtensionContext(extension));
             }
             else
             {
-                var result = VirtualMachineExtensionClient.Get(ResourceGroupName, VMName, Name);
-                var extension = result.ToPSVirtualMachineExtension(ResourceGroupName);
-
-                if (
-                    extension.Publisher.Equals(
-                        VirtualMachineSqlServerExtensionContext.ExtensionPublishedNamespace,
-                        StringComparison.InvariantCultureIgnoreCase) &&
-                    extension.ExtensionType.Equals(
-                        VirtualMachineSqlServerExtensionContext.ExtensionPublishedName,
-                        StringComparison.InvariantCultureIgnoreCase))
-                {
-                    WriteObject(GetSqlServerExtensionContext(extension));
-                }
-                else
-                {
-                    WriteObject(null);
-                }
+                WriteObject(null);
             }
-
         }
 
         private VirtualMachineSqlServerExtensionContext GetSqlServerExtensionContext(PSVirtualMachineExtension extension)
@@ -138,6 +107,8 @@ private VirtualMachineSqlServerExtensionContext GetSqlServerExtensionContext(PSV
                     PublicSettings = JsonConvert.SerializeObject(extensionPublicSettings),
                     ProtectedSettings = extension.ProtectedSettings,
                     ProvisioningState = extension.ProvisioningState,
+                    AutoBackupSettings = extensionPublicSettings.AutoBackupSettings,
+                    AutoPatchingSettings = extensionPublicSettings.AutoPatchingSettings,
                     Statuses = extension.Statuses
                 };