Skip to content

Commit a9c69cd

Browse files
Fixing pipelining issues
1 parent ffb6416 commit a9c69cd

11 files changed

+18
-18
lines changed

src/ResourceManager/SiteRecovery/Commands.SiteRecovery/Network/GetAzureRMSiteRecoveryNetwork.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,8 @@ public class GetAzureRMSiteRecoveryNetwork : SiteRecoveryCmdletBase
3232
/// Gets or sets Server object.
3333
/// </summary>
3434
[Parameter(ParameterSetName = ASRParameterSets.ByServerObject, Mandatory = true, ValueFromPipeline = true)]
35-
[Parameter(ParameterSetName = ASRParameterSets.ByName, Mandatory = true)]
36-
[Parameter(ParameterSetName = ASRParameterSets.ByFriendlyName, Mandatory = true)]
35+
[Parameter(ParameterSetName = ASRParameterSets.ByName, Mandatory = true, ValueFromPipeline = true)]
36+
[Parameter(ParameterSetName = ASRParameterSets.ByFriendlyName, Mandatory = true, ValueFromPipeline = true)]
3737
[ValidateNotNullOrEmpty]
3838
public ASRServer Server { get; set; }
3939

src/ResourceManager/SiteRecovery/Commands.SiteRecovery/Network/GetAzureRMSiteRecoveryNetworkMapping.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,8 +31,8 @@ public class GetAzureRMSiteRecoveryNetworkMapping : SiteRecoveryCmdletBase
3131
/// <summary>
3232
/// Gets or sets Primary Server object.
3333
/// </summary>
34-
[Parameter(ParameterSetName = ASRParameterSets.EnterpriseToEnterprise, Mandatory = true)]
35-
[Parameter(ParameterSetName = ASRParameterSets.EnterpriseToAzure, Mandatory = true)]
34+
[Parameter(ParameterSetName = ASRParameterSets.EnterpriseToEnterprise, Mandatory = true, ValueFromPipeline = true)]
35+
[Parameter(ParameterSetName = ASRParameterSets.EnterpriseToAzure, Mandatory = true, ValueFromPipeline = true)]
3636
[ValidateNotNullOrEmpty]
3737
public ASRServer PrimaryServer { get; set; }
3838

src/ResourceManager/SiteRecovery/Commands.SiteRecovery/Network/NewAzureRMSiteRecoveryNetworkMapping.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,8 @@ public class NewAzureRMSiteRecoveryNetworkMapping : SiteRecoveryCmdletBase
3232
/// <summary>
3333
/// Gets or sets Primary Network object.
3434
/// </summary>
35-
[Parameter(ParameterSetName = ASRParameterSets.EnterpriseToEnterprise, Mandatory = true)]
36-
[Parameter(ParameterSetName = ASRParameterSets.EnterpriseToAzure, Mandatory = true)]
35+
[Parameter(ParameterSetName = ASRParameterSets.EnterpriseToEnterprise, Mandatory = true, ValueFromPipeline = true)]
36+
[Parameter(ParameterSetName = ASRParameterSets.EnterpriseToAzure, Mandatory = true, ValueFromPipeline = true)]
3737
[ValidateNotNullOrEmpty]
3838
public ASRNetwork PrimaryNetwork { get; set; }
3939

src/ResourceManager/SiteRecovery/Commands.SiteRecovery/Policy/RemoveAzureSiteRecoveryPolicy.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ public class RemoveAzureSiteRecoveryPolicy : SiteRecoveryCmdletBase
3232
/// <summary>
3333
/// Gets or sets Name of the Policy.
3434
/// </summary>
35-
[Parameter(Mandatory = true)]
35+
[Parameter(Mandatory = true, ValueFromPipeline = true)]
3636
public ASRPolicy Policy { get; set; }
3737

3838
#endregion Parameters

src/ResourceManager/SiteRecovery/Commands.SiteRecovery/ProtectionEntity/UpdateAzureSiteRecoveryProtectionDirection.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,8 +62,8 @@ public class UpdateAzureSiteRecoveryProtection : SiteRecoveryCmdletBase
6262
/// <summary>
6363
/// Gets or sets Failover direction for the recovery plan.
6464
/// </summary>
65-
[Parameter(ParameterSetName = ASRParameterSets.ByRPObject, Mandatory = true, ValueFromPipeline = true)]
66-
[Parameter(ParameterSetName = ASRParameterSets.ByPEObject, Mandatory = true, ValueFromPipeline = false)]
65+
[Parameter(ParameterSetName = ASRParameterSets.ByRPObject, Mandatory = true)]
66+
[Parameter(ParameterSetName = ASRParameterSets.ByPEObject, Mandatory = true)]
6767
[ValidateSet(
6868
Constants.PrimaryToRecovery,
6969
Constants.RecoveryToPrimary)]

src/ResourceManager/SiteRecovery/Commands.SiteRecovery/RecoveryPlan/EditAzureSiteRecoveryRecoveryPlan.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ public class EditAzureSiteRecoveryRecoveryPlan : SiteRecoveryCmdletBase
3434
/// <summary>
3535
/// Gets or sets Name of the Recovery Plan.
3636
/// </summary>
37-
[Parameter(Mandatory = true)]
37+
[Parameter(Mandatory = true, ValueFromPipeline = true)]
3838
[ValidateNotNullOrEmpty]
3939
public ASRRecoveryPlan RecoveryPlan { get; set; }
4040

src/ResourceManager/SiteRecovery/Commands.SiteRecovery/RecoveryPlan/NewAzureSiteRecoveryRecoveryPlan.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -100,9 +100,9 @@ public class NewAzureSiteRecoveryRecoveryPlan : SiteRecoveryCmdletBase
100100
/// <summary>
101101
/// Gets or sets Replication Frequency of the Policy in seconds.
102102
/// </summary>
103-
[Parameter(ParameterSetName = ASRParameterSets.EnterpriseToEnterprise, Mandatory = true)]
104-
[Parameter(ParameterSetName = ASRParameterSets.EnterpriseToAzure, Mandatory = true)]
105-
[Parameter(ParameterSetName = ASRParameterSets.HyperVSiteToAzure, Mandatory = true)]
103+
[Parameter(ParameterSetName = ASRParameterSets.EnterpriseToEnterprise, Mandatory = true, ValueFromPipeline = true)]
104+
[Parameter(ParameterSetName = ASRParameterSets.EnterpriseToAzure, Mandatory = true, ValueFromPipeline = true)]
105+
[Parameter(ParameterSetName = ASRParameterSets.HyperVSiteToAzure, Mandatory = true, ValueFromPipeline = true)]
106106
[ValidateNotNullOrEmpty]
107107
public ASRProtectionEntity[] ProtectionEntityList { get; set; }
108108

src/ResourceManager/SiteRecovery/Commands.SiteRecovery/RecoveryPlan/RemoveAzureSiteRecoveryRecoveryPlan.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ public class RemoveAzureSiteRecoveryRecoveryPlan : SiteRecoveryCmdletBase
3838
/// <summary>
3939
/// Gets or sets Name of the Recovery Plan.
4040
/// </summary>
41-
[Parameter(Mandatory = true, ParameterSetName = ASRParameterSets.ByObject)]
41+
[Parameter(Mandatory = true, ParameterSetName = ASRParameterSets.ByObject, ValueFromPipeline = true)]
4242
public ASRRecoveryPlan RecoveryPlan { get; set; }
4343

4444
#endregion Parameters

src/ResourceManager/SiteRecovery/Commands.SiteRecovery/RecoveryPlan/UpdateAzureSiteRecoveryRecoveryPlan.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ public class UpdateAzureSiteRecoveryRecoveryPlan : SiteRecoveryCmdletBase
3636
/// <summary>
3737
/// Gets or sets Name of the Recovery Plan.
3838
/// </summary>
39-
[Parameter(ParameterSetName = ASRParameterSets.ByRPObject, Mandatory = true)]
39+
[Parameter(ParameterSetName = ASRParameterSets.ByRPObject, Mandatory = true, ValueFromPipeline = true)]
4040
public ASRRecoveryPlan RecoveryPlan { get; set; }
4141

4242
/// <summary>

src/ResourceManager/SiteRecovery/Commands.SiteRecovery/Storage/Classification/GetAzureSiteRecoveryStorageClassificationMapping.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ public class GetAzureSiteRecoveryStorageClassificationMapping : SiteRecoveryCmdl
1919
/// <summary>
2020
/// Gets or sets name of classification.
2121
/// </summary>
22-
[Parameter(ParameterSetName = ASRParameterSets.ByName, Mandatory = true, ValueFromPipeline = true)]
22+
[Parameter(ParameterSetName = ASRParameterSets.ByName, Mandatory = true)]
2323
[ValidateNotNullOrEmpty]
2424
public string Name { get; set; }
2525
#endregion

src/ResourceManager/SiteRecovery/Commands.SiteRecovery/VM/SetAzureSiteRecoveryVM.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ public class SetAzureSiteRecoveryVM : SiteRecoveryCmdletBase
3333
/// <summary>
3434
/// Gets or sets ID of the Virtual Machine.
3535
/// </summary>
36-
[Parameter(Mandatory = true)]
36+
[Parameter(Mandatory = true, ValueFromPipeline = true)]
3737
[ValidateNotNullOrEmpty]
3838
public ASRVirtualMachine VirtualMachine { get; set; }
3939

@@ -195,7 +195,7 @@ public override void ExecuteSiteRecoveryCmdlet()
195195
LongRunningOperationResponse response = RecoveryServicesClient.UpdateVmProperties(
196196
Utilities.GetValueFromArmId(this.VirtualMachine.ID, ARMResourceTypeConstants.ReplicationFabrics),
197197
Utilities.GetValueFromArmId(this.VirtualMachine.ID, ARMResourceTypeConstants.ReplicationProtectionContainers),
198-
this.VirtualMachine.Name,
198+
replicationProtectedItemResponse.ReplicationProtectedItem.Name,
199199
input);
200200

201201
JobResponse jobResponse =

0 commit comments

Comments
 (0)