Bugfixes:

1. Bug 1526979: Get-AzureRmRecoveryServicesBackupItem should do case insensitive comparison for container name filter
2. AzureVmItem should have property displaying ServiceClientModel.AzureIaaSVMProtectedItem.LastBackupTime
3. Some foundation for - Bug 1524000: [PowerShell][New-Ask] For V2 Vm restores, block storage accounts with Kind "Blob Storage" at PowerShell

Removing traces of changes made for Storage.Kind

Author: Samuel Anudeep

src/ResourceManager/RecoveryServices.Backup/Commands.RecoveryServices.Backup.Models/AzureVmModels/AzureVmItem.cs

@@ -39,6 +39,11 @@ public class AzureVmItem : ItemBase
         /// </summary>
         public string LastBackupStatus { get; set; }
 
+        /// <summary>
+        /// Last Backup Time for the item
+        /// </summary>
+        public DateTime? LastBackupTime { get; set; }
+
         /// <summary>
         /// Protection Policy Name for the Item
         /// </summary>
@@ -63,6 +68,7 @@ public AzureVmItem(ProtectedItemResource protectedItemResource,
         {
             AzureIaaSVMProtectedItem protectedItem = (AzureIaaSVMProtectedItem)protectedItemResource.Properties;
             LastBackupStatus = protectedItem.LastBackupStatus;
+            LastBackupTime = protectedItem.LastBackupTime;
             ProtectionPolicyName = policyName;
             ProtectionState =
                 EnumUtils.GetEnum<ItemProtectionState>(protectedItem.ProtectionState.ToString());

src/ResourceManager/RecoveryServices.Backup/Commands.RecoveryServices.Backup.Providers/Providers/IaasVmPsBackupProvider.cs

@@ -677,7 +677,7 @@ public List<ItemBase> ListProtectedItems()
 
                     var delimIndex = containerUri.IndexOf(';');
                     string containerName = containerUri.Substring(delimIndex + 1);
-                    return containerName.Equals(container.Name);
+                    return containerName.ToLower().Equals(container.Name.ToLower());
                 }).ToList();
             }
 

src/ResourceManager/RecoveryServices.Backup/Commands.RecoveryServices.Backup.Test/ScenarioTests/IaasVm/ItemTests.ps1

@@ -47,6 +47,7 @@ function Test-AzureVMGetItems
 			-Name $vm.Name;
 		Assert-True { $items.Count -eq 1 }
 		Assert-True { $items.VirtualMachineId -contains $vm.Id }
+		Assert-NotNull $items[0].LastBackupTime
 
 		# VARIATION-3: Get items for container with ProtectionStatus filter
 		$items = Get-AzureRmRecoveryServicesBackupItem `

src/ResourceManager/RecoveryServices.Backup/Commands.RecoveryServices.Backup/Cmdlets/Restore/RestoreAzureRMRecoveryServicesBackupItem.cs

@@ -60,55 +60,17 @@ public override void ExecuteCmdlet()
             ExecutionBlock(() =>
             {
                 base.ExecuteCmdlet();
-                StorageAccountName = StorageAccountName.ToLower();
-                ResourceIdentity identity = new ResourceIdentity();
-                identity.ResourceName = StorageAccountName;
-                identity.ResourceProviderNamespace = "Microsoft.ClassicStorage/storageAccounts";
-                identity.ResourceProviderApiVersion = "2015-12-01";
-                identity.ResourceType = string.Empty;
-                identity.ParentResourcePath = string.Empty;
 
-                GenericResource resource = null;
-                try
-                {
-                    WriteDebug(string.Format("Query Microsoft.ClassicStorage with name = {0}",
-                        StorageAccountName));
-                    resource = RmClient.Resources.GetAsync(
-                        StorageAccountResourceGroupName,
-                        identity.ResourceProviderNamespace,
-                        identity.ParentResourcePath,
-                        identity.ResourceType,
-                        identity.ResourceName,
-                        identity.ResourceProviderApiVersion,
-                        CancellationToken.None).Result;
-                }
-                catch (Exception)
-                {
-                    identity.ResourceProviderNamespace = "Microsoft.Storage/storageAccounts";
-                    identity.ResourceProviderApiVersion = "2016-01-01";
-                    resource = RmClient.Resources.GetAsync(
-                        StorageAccountResourceGroupName,
-                        identity.ResourceProviderNamespace,
-                        identity.ParentResourcePath,
-                        identity.ResourceType,
-                        identity.ResourceName,
-                        identity.ResourceProviderApiVersion,
-                        CancellationToken.None).Result;
-                }
-
-                string storageAccountId = resource.Id;
-                string storageAccountlocation = resource.Location;
-                string storageAccountType = resource.Type;
-
-                WriteDebug(string.Format("StorageId = {0}", storageAccountId));
+                GenericResource storageAccountResource = GetStorageAccountResource();
+                WriteDebug(string.Format("StorageId = {0}", storageAccountResource.Id));
 
                 PsBackupProviderManager providerManager = new PsBackupProviderManager(
                     new Dictionary<Enum, object>()
                 {
                     {RestoreBackupItemParams.RecoveryPoint, RecoveryPoint},
-                    {RestoreBackupItemParams.StorageAccountId, storageAccountId},
-                    {RestoreBackupItemParams.StorageAccountLocation, storageAccountlocation},
-                    {RestoreBackupItemParams.StorageAccountType, storageAccountType}
+                    {RestoreBackupItemParams.StorageAccountId, storageAccountResource.Id},
+                    {RestoreBackupItemParams.StorageAccountLocation, storageAccountResource.Location},
+                    {RestoreBackupItemParams.StorageAccountType, storageAccountResource.Type}
                 }, ServiceClientAdapter);
 
                 IPsBackupProvider psBackupProvider = providerManager.GetProviderInstance(
@@ -119,5 +81,46 @@ public override void ExecuteCmdlet()
                 HandleCreatedJob(jobResponse, Resources.RestoreOperation);
             }, ShouldProcess(RecoveryPoint.ItemName, VerbsData.Restore));
         }
+
+        private GenericResource GetStorageAccountResource()
+        {
+            StorageAccountName = StorageAccountName.ToLower();
+            ResourceIdentity identity = new ResourceIdentity();
+            identity.ResourceName = StorageAccountName;
+            identity.ResourceProviderNamespace = "Microsoft.ClassicStorage/storageAccounts";
+            identity.ResourceProviderApiVersion = "2015-12-01";
+            identity.ResourceType = string.Empty;
+            identity.ParentResourcePath = string.Empty;
+
+            GenericResource resource = null;
+            try
+            {
+                WriteDebug(string.Format("Query Microsoft.ClassicStorage with name = {0}",
+                    StorageAccountName));
+                resource = RmClient.Resources.GetAsync(
+                    StorageAccountResourceGroupName,
+                    identity.ResourceProviderNamespace,
+                    identity.ParentResourcePath,
+                    identity.ResourceType,
+                    identity.ResourceName,
+                    identity.ResourceProviderApiVersion,
+                    CancellationToken.None).Result;
+            }
+            catch (Exception)
+            {
+                identity.ResourceProviderNamespace = "Microsoft.Storage/storageAccounts";
+                identity.ResourceProviderApiVersion = "2016-01-01";
+                resource = RmClient.Resources.GetAsync(
+                    StorageAccountResourceGroupName,
+                    identity.ResourceProviderNamespace,
+                    identity.ParentResourcePath,
+                    identity.ResourceType,
+                    identity.ResourceName,
+                    identity.ResourceProviderApiVersion,
+                    CancellationToken.None).Result;
+            }
+
+            return resource;
+        }
     }
 }

src/ResourceManager/RecoveryServices.Backup/Commands.RecoveryServices.Backup/RecoveryServicesBackupCmdletBase.cs

@@ -16,7 +16,7 @@
 using System.Collections.Generic;
 using System.Management.Automation;
 using Microsoft.Azure.Commands.Common.Authentication;
-using Microsoft.Azure.Commands.Common.Authentication.Models;
+using Microsoft.Azure.Commands.Common.Authentication.Abstractions;
 using Microsoft.Azure.Commands.RecoveryServices.Backup.Cmdlets.ServiceClientAdapterNS;
 using Microsoft.Azure.Commands.RecoveryServices.Backup.Helpers;
 using Microsoft.Azure.Commands.RecoveryServices.Backup.Properties;
@@ -26,7 +26,6 @@
 using CmdletModel = Microsoft.Azure.Commands.RecoveryServices.Backup.Cmdlets.Models;
 using ResourcesNS = Microsoft.Azure.Management.Internal.Resources;
 using SystemNet = System.Net;
-using Microsoft.Azure.Commands.Common.Authentication.Abstractions;
 
 namespace Microsoft.Azure.Commands.RecoveryServices.Backup.Cmdlets
 {
@@ -52,7 +51,7 @@ protected void InitializeAzureBackupCmdlet()
         {
             ServiceClientAdapter = new ServiceClientAdapter(DefaultContext);
 
-            WriteDebug("InsideRestore. going to create ResourceManager Client");
+            WriteDebug("Inside Restore. Going to create ResourceClient.");
             RmClient = AzureSession.Instance.ClientFactory.CreateArmClient<ResourcesNS.ResourceManagementClient>(DefaultContext, AzureEnvironment.Endpoint.ResourceManager);
 
             WriteDebug("Client Created successfully");