Change api-version for policy and locks

Author: vivsriaus

src/ResourceManager/Resources/Commands.ResourceManager/Cmdlets/Components/Constants.cs

@@ -57,7 +57,12 @@ public static class Constants
         /// <summary>
         /// The default policy API version.
         /// </summary>
-        public static readonly string PolicyApiVersion = "2015-10-01-preview";
+        public static readonly string PolicyApiVersion = "2016-04-01";
+
+        /// <summary>
+        /// The default Lock API version.
+        /// </summary>
+        public static readonly string LockApiVersion = "2015-01-01";
 
         /// <summary>
         /// The move action.

src/ResourceManager/Resources/Commands.ResourceManager/Cmdlets/Implementation/Lock/GetAzureResourceLockCmdlet.cs

@@ -80,15 +80,11 @@ private async Task<JObject> GetResource()
         {
             var resourceId = this.GetResourceId(this.LockName);
 
-            var apiVersion = await this
-                .DetermineApiVersion(resourceId: resourceId)
-                .ConfigureAwait(continueOnCapturedContext: false);
-
             return await this
                 .GetResourcesClient()
                 .GetResource<JObject>(
                     resourceId: resourceId,
-                    apiVersion: apiVersion,
+                    apiVersion: this.LockApiVersion,
                     cancellationToken: this.CancellationToken.Value)
                 .ConfigureAwait(continueOnCapturedContext: false);
         }
@@ -100,10 +96,6 @@ private async Task<ResponseWithContinuation<JObject[]>> ListResourcesTypeCollect
         {
             var resourceCollectionId = this.GetResourceId(this.LockName);
 
-            var apiVersion = await this
-                .DetermineApiVersion(resourceId: resourceCollectionId)
-                .ConfigureAwait(continueOnCapturedContext: false);
-
             var filter = this.AtScope
                 ? "$filter=atScope()"
                 : null;
@@ -112,7 +104,7 @@ private async Task<ResponseWithContinuation<JObject[]>> ListResourcesTypeCollect
                 .GetResourcesClient()
                 .ListObjectColleciton<JObject>(
                     resourceCollectionId: resourceCollectionId,
-                    apiVersion: apiVersion,
+                    apiVersion: this.LockApiVersion,
                     cancellationToken: this.CancellationToken.Value,
                     odataQuery: filter)
                 .ConfigureAwait(continueOnCapturedContext: false);

src/ResourceManager/Resources/Commands.ResourceManager/Cmdlets/Implementation/Lock/NewAzureResourceLockCmdlet.cs

@@ -74,21 +74,20 @@ protected override void OnProcessRecord()
                 resourceId,
                 () =>
                 {
-                    var apiVersion = this.DetermineApiVersion(resourceId: resourceId).Result;
                     var resourceBody = this.GetResourceBody();
 
                     var operationResult = this.GetResourcesClient()
                         .PutResource(
                             resourceId: resourceId,
-                            apiVersion: apiVersion,
+                            apiVersion: this.LockApiVersion,
                             resource: resourceBody,
                             cancellationToken: this.CancellationToken.Value)
                         .Result;
 
                     var managementUri = this.GetResourcesClient()
                       .GetResourceManagementRequestUri(
                           resourceId: resourceId,
-                          apiVersion: apiVersion);
+                          apiVersion: this.LockApiVersion);
 
                     var activity = string.Format("PUT {0}", managementUri.PathAndQuery);
                     var result = this.GetLongRunningOperationTracker(activityName: activity, isResourceCreateOrUpdate: true)

src/ResourceManager/Resources/Commands.ResourceManager/Cmdlets/Implementation/Lock/RemoveAzureResourceLockCmdlet.cs

@@ -56,12 +56,10 @@ protected override void OnProcessRecord()
                 resourceId,
                 () =>
                 {
-                    var apiVersion = this.DetermineApiVersion(resourceId: resourceId).Result;
-
                     var operationResult = this.GetResourcesClient()
                         .DeleteResource(
                             resourceId: resourceId,
-                            apiVersion: apiVersion,
+                            apiVersion: this.LockApiVersion,
                             cancellationToken: this.CancellationToken.Value)
                         .Result;
 
@@ -73,7 +71,7 @@ protected override void OnProcessRecord()
                     var managementUri = this.GetResourcesClient()
                         .GetResourceManagementRequestUri(
                             resourceId: resourceId,
-                            apiVersion: apiVersion);
+                            apiVersion: this.LockApiVersion);
 
                     var activity = string.Format("DELETE {0}", managementUri.PathAndQuery);
 

src/ResourceManager/Resources/Commands.ResourceManager/Cmdlets/Implementation/Lock/ResourceLockManagementCmdletBase.cs

@@ -65,7 +65,6 @@ public abstract class ResourceLockManagementCmdletBase : ResourceManagerCmdletBa
         /// <summary>
         /// Gets or sets the scope.
         /// </summary>
-        [Alias("Id", "ResourceId")]
         [Parameter(ParameterSetName = ResourceLockManagementCmdletBase.ScopeLevelLock, Mandatory = true, ValueFromPipelineByPropertyName = true, HelpMessage = "The scope. e.g. to specify a database '/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Sql/servers/{serverName}/databases/{databaserName}', to specify a resoruce group: '/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}'")]
         [ValidateNotNullOrEmpty]
         public string Scope { get; set; }
@@ -105,10 +104,21 @@ public abstract class ResourceLockManagementCmdletBase : ResourceManagerCmdletBa
         /// <summary>
         /// The Id of the lock.
         /// </summary>
+        [Alias("Id", "ResourceId")]
         [Parameter(ParameterSetName = ResourceLockManagementCmdletBase.LockIdParameterSet, Mandatory = true, ValueFromPipelineByPropertyName = true, HelpMessage = "The Id of the lock.")]
         [ValidateNotNullOrEmpty]
         public string LockId { get; set; }
 
+        /// <summary>
+        /// Determines the api version to be used
+        /// </summary>
+        public string LockApiVersion { get; set; }
+
+        public ResourceLockManagementCmdletBase()
+        {
+            this.LockApiVersion = string.IsNullOrWhiteSpace(this.ApiVersion) ? Constants.LockApiVersion : this.ApiVersion;
+        }
+
         /// <summary> 
         /// Gets the resource Id from the supplied PowerShell parameters. 
         /// </summary>