Remove ApiVersion Parameter and change Get-AzResourceGroupOperation implementation to use SDK (#12642)

* Remove ApiVersion Parameter from deployments

* update changelog

* Delete ApiVersion param

* Switch to SDK use in ResourceGroup Deployments cmdlets

* Include review fixes

* Changed help file to reflect new output type

* Rollback breaking changes from PR #12141

* Change Get-AzResourceGroupDeploymentOperation implementation to use SDK

* Update help

* Rollback changes to help files of untouched cmdlets

* Suppress breaking change issues

* remove subscriptionId param

* Update breakingchanges csv

* Suppress breaking changes

Author: Xynoclafe

src/Resources/ResourceManager/Implementation/CmdletBase/ResourceManagerCmdletBase.cs

@@ -25,15 +25,13 @@ namespace Microsoft.Azure.Commands.ResourceManager.Cmdlets.Implementation
     using Microsoft.Azure.Commands.ResourceManager.Cmdlets.RestClients;
     using Microsoft.Azure.Commands.ResourceManager.Cmdlets.SdkClient;
     using Microsoft.Rest.Azure;
-    using Microsoft.WindowsAzure.Commands.Common.CustomAttributes;
     using Newtonsoft.Json.Linq;
     using System;
     using System.Collections.Generic;
     using System.Linq;
     using System.Management.Automation;
     using System.Runtime.ExceptionServices;
     using System.Threading;
-    using System.Threading.Tasks;
 
     /// <summary>
     /// The base class for resource manager cmdlets.
@@ -66,14 +64,6 @@ protected CancellationToken? CancellationToken
         /// </summary>
         private SubscriptionSdkClient subscriptionSdkClient;
 
-        /// <summary>
-        /// Gets or sets the API version.
-        /// </summary>
-        [CmdletParameterBreakingChange("ApiVersion", ChangeDescription = "Parameter is being deprecated without being replaced")]
-        [Parameter(Mandatory = false, HelpMessage = "When set, indicates the version of the resource provider API to use. If not specified, the API version is automatically determined as the latest available.")]
-        [ValidateNotNullOrEmpty]
-        public virtual string ApiVersion { get; set; }
-
         /// <summary>
         /// Gets or sets the switch that indicates if pre-release API version should be considered.
         /// </summary>
@@ -223,42 +213,6 @@ protected virtual void OnStopProcessing()
             // no-op
         }
 
-        /// <summary>
-        /// Determines the API version.
-        /// </summary>
-        /// <param name="resourceId">The resource Id.</param>
-        /// <param name="pre">When specified, indicates if pre-release API versions should be considered.</param>
-        protected Task<string> DetermineApiVersion(string resourceId, bool? pre = null)
-        {
-            return string.IsNullOrWhiteSpace(this.ApiVersion)
-                ? ApiVersionHelper.DetermineApiVersion(
-                    context: DefaultContext,
-                    resourceId: resourceId,
-                    cancellationToken: this.CancellationToken.Value,
-                    pre: pre ?? this.Pre,
-                    cmdletHeaderValues: this.GetCmdletHeaders())
-                : Task.FromResult(this.ApiVersion);
-        }
-
-        /// <summary>
-        /// Determines the API version.
-        /// </summary>
-        /// <param name="providerNamespace">The provider namespace.</param>
-        /// <param name="resourceType">The resource type.</param>
-        /// <param name="pre">When specified, indicates if pre-release API versions should be considered.</param>
-        protected Task<string> DetermineApiVersion(string providerNamespace, string resourceType, bool? pre = null)
-        {
-            return string.IsNullOrWhiteSpace(this.ApiVersion)
-                ? ApiVersionHelper.DetermineApiVersion(
-                    DefaultContext,
-                    providerNamespace: providerNamespace,
-                    resourceType: resourceType,
-                    cancellationToken: this.CancellationToken.Value,
-                    pre: pre ?? this.Pre,
-                    cmdletHeaderValues: this.GetCmdletHeaders())
-                : Task.FromResult(this.ApiVersion);
-        }
-
         /// <summary>
         /// Gets a new instance of the <see cref="ResourceManagerRestRestClient"/>.
         /// </summary>

src/Resources/ResourceManager/Implementation/CmdletBase/ResourceManagerCmdletBaseWithAPiVersion.cs

@@ -13,6 +13,8 @@
 // ----------------------------------------------------------------------------------
 
 using System.Management.Automation;
+using System.Threading.Tasks;
+using System.Collections.Generic;
 
 namespace Microsoft.Azure.Commands.ResourceManager.Cmdlets.Implementation
 {
@@ -23,6 +25,51 @@ public abstract class ResourceManagerCmdletBaseWithAPiVersion : ResourceManagerC
         /// </summary>
         [Parameter(Mandatory = false, HelpMessage = "When set, indicates the version of the resource provider API to use. If not specified, the API version is automatically determined as the latest available.")]
         [ValidateNotNullOrEmpty]
-        public override string ApiVersion { get; set; }
+        public string ApiVersion { get; set; }
+
+        private Dictionary<string, string> GetCmdletHeaders()
+        {
+            return new Dictionary<string, string>
+            {
+                {"ParameterSetName", this.ParameterSetName },
+                {"CommandName", this.CommandRuntime.ToString() }
+            };
+        }
+
+        /// <summary>
+        /// Determines the API version.
+        /// </summary>
+        /// <param name="resourceId">The resource Id.</param>
+        /// <param name="pre">When specified, indicates if pre-release API versions should be considered.</param>
+        protected Task<string> DetermineApiVersion(string resourceId, bool? pre = null)
+        {
+            return string.IsNullOrWhiteSpace(this.ApiVersion)
+                ? Components.ApiVersionHelper.DetermineApiVersion(
+                    context: DefaultContext,
+                    resourceId: resourceId,
+                    cancellationToken: this.CancellationToken.Value,
+                    pre: pre ?? this.Pre,
+                    cmdletHeaderValues: this.GetCmdletHeaders())
+                : Task.FromResult(this.ApiVersion);
+        }
+
+        /// <summary>
+        /// Determines the API version.
+        /// </summary>
+        /// <param name="providerNamespace">The provider namespace.</param>
+        /// <param name="resourceType">The resource type.</param>
+        /// <param name="pre">When specified, indicates if pre-release API versions should be considered.</param>
+        protected Task<string> DetermineApiVersion(string providerNamespace, string resourceType, bool? pre = null)
+        {
+            return string.IsNullOrWhiteSpace(this.ApiVersion)
+                ? Components.ApiVersionHelper.DetermineApiVersion(
+                    DefaultContext,
+                    providerNamespace: providerNamespace,
+                    resourceType: resourceType,
+                    cancellationToken: this.CancellationToken.Value,
+                    pre: pre ?? this.Pre,
+                    cmdletHeaderValues: this.GetCmdletHeaders())
+                : Task.FromResult(this.ApiVersion);
+        }
     }
 }

src/Resources/ResourceManager/Implementation/ResourceGroupDeployments/GetAzureResourceGroupDeploymentOperationCmdlet.cs

@@ -14,20 +14,14 @@
 
 namespace Microsoft.Azure.Commands.ResourceManager.Cmdlets.Implementation
 {
-    using Commands.Common.Authentication.Abstractions;
     using Common.ArgumentCompleters;
-    using Microsoft.Azure.Commands.ResourceManager.Cmdlets.Components;
-    using Microsoft.Azure.Commands.ResourceManager.Cmdlets.Extensions;
-    using Microsoft.Azure.Commands.ResourceManager.Common;
-    using Newtonsoft.Json.Linq;
-    using System;
     using System.Management.Automation;
-    using System.Threading.Tasks;
+    using Microsoft.Azure.Commands.ResourceManager.Cmdlets.SdkModels;
 
     /// <summary>
     /// Gets the deployment operation.
     /// </summary>
-    [Cmdlet("Get", ResourceManager.Common.AzureRMConstants.AzureRMPrefix + "ResourceGroupDeploymentOperation"), OutputType(typeof(PSObject))]
+    [Cmdlet("Get", ResourceManager.Common.AzureRMConstants.AzureRMPrefix + "ResourceGroupDeploymentOperation"), OutputType(typeof(PSDeploymentOperation))]
     public class GetAzureResourceGroupDeploymentOperationCmdlet : ResourceManagerCmdletBase
     {
         /// <summary>
@@ -38,13 +32,6 @@ public class GetAzureResourceGroupDeploymentOperationCmdlet : ResourceManagerCmd
         [ValidateNotNullOrEmpty]
         public string DeploymentName { get; set; }
 
-        /// <summary>
-        /// Gets or sets the subscription id parameter.
-        /// </summary>
-        [Parameter(Mandatory = false, ValueFromPipeline = true, ValueFromPipelineByPropertyName = true, HelpMessage = "The subscription to use.")]
-        [ValidateNotNullOrEmpty]
-        public Guid? SubscriptionId { get; set; }
-
         /// <summary>
         /// Gets or sets the resource group name parameter.
         /// </summary>
@@ -58,68 +45,12 @@ public class GetAzureResourceGroupDeploymentOperationCmdlet : ResourceManagerCmd
         /// </summary>
         protected override void OnProcessRecord()
         {
+            var deploymentOperations = ResourceManagerSdkClient.ListDeploymentOperationsAtResourceGroup(
+                ResourceGroupName, DeploymentName);
             base.OnProcessRecord();
 
-            if (this.SubscriptionId == null)
-            {
-                this.SubscriptionId = DefaultContext.Subscription.GetId();
-            }
-
-            this.RunCmdlet();
-        }
-
-        /// <summary>
-        /// Contains the cmdlet's execution logic.
-        /// </summary>
-        private void RunCmdlet()
-        {
-            PaginatedResponseHelper.ForEach(
-                getFirstPage: () => this.GetResources(),
-                getNextPage: nextLink => this.GetNextLink<JObject>(nextLink),
-                cancellationToken: this.CancellationToken,
-                action: resources => this.WriteObject(sendToPipeline: resources.CoalesceEnumerable().SelectArray(resource =>
-                    resource.ToPsObject("System.Management.Automation.PSCustomObject#DeploymentOperation")), enumerateCollection: true));
-        }
-
-        /// <summary>
-        /// Queries the ARM cache and returns the cached resource that match the query specified.
-        /// </summary>
-        private async Task<ResponseWithContinuation<JObject[]>> GetResources()
-        {
-            var resourceId = this.GetResourceId();
-
-            var apiVersion = string.IsNullOrWhiteSpace(this.ApiVersion) ? Constants.DeploymentOperationApiVersion : this.ApiVersion;
-
-            return await this
-                .GetResourcesClient()
-                .ListObjectColleciton<JObject>(
-                    resourceCollectionId: resourceId,
-                    apiVersion: apiVersion,
-                    cancellationToken: this.CancellationToken.Value)
-                .ConfigureAwait(continueOnCapturedContext: false);
+            WriteObject(deploymentOperations, true);
         }
 
-        /// <summary>
-        /// Gets the next set of resources using the <paramref name="nextLink"/>
-        /// </summary>
-        /// <param name="nextLink">The next link.</param>
-        private Task<ResponseWithContinuation<TType[]>> GetNextLink<TType>(string nextLink)
-        {
-            return this
-                .GetResourcesClient()
-                .ListNextBatch<TType>(nextLink: nextLink, cancellationToken: this.CancellationToken.Value);
-        }
-
-        /// <summary>
-        /// Gets the resource Id from the supplied PowerShell parameters.
-        /// </summary>
-        protected string GetResourceId()
-        {
-            return ResourceIdUtility.GetResourceId(
-                subscriptionId: this.SubscriptionId,
-                resourceGroupName: this.ResourceGroupName,
-                resourceType: Constants.MicrosoftResourcesDeploymentOperationsType,
-                resourceName: this.DeploymentName);
-        }
     }
-}
+}