Merge pull request #42 from AsrOneSdk/sanjkuma-dev

Use WriteWarning instead of WriteDebug.
Azure · Feb 9, 2015 · 8bb31e3 · 8bb31e3
2 parents 702c98d + 498d7e7
commit 8bb31e3
Show file tree

Hide file tree

Showing 5 changed files with 21 additions and 16 deletions.
diff --git a/...t/RecoveryServices/Commands.RecoveryServices.Test/ScenarioTests/RecoveryServicesTests.ps1 b/...t/RecoveryServices/Commands.RecoveryServices.Test/ScenarioTests/RecoveryServicesTests.ps1
@@ -72,15 +72,15 @@ function Test-E2E_DeleteAndDissociate
                     {
                         if ($association.AssociationStatus -eq "Paired")
                         {
-                            // We have got the paired profile. Fire delete and dissociate
+                            # We have got the paired profile. Fire delete and dissociate
                             $pcPri = Get-AzureSiteRecoveryProtectionContainer -Id $association.PrimaryProtectionContainerId
                             $pcRec = Get-AzureSiteRecoveryProtectionContainer -Id $association.RecoveryProtectionContainerId
                             $job = Start-AzureSiteRecoveryProtectionProfileDissociationJob -PrimaryProtectionContainer $pcPri -RecoveryProtectionContainer $pcRec -ProtectionProfile $profile
 
 							# Validate_ProfileDissociation_JobSucceeded
 							if ($Validate_ProfileDissociation_JobSucceeded -eq $true)
 							{
-								WaitForJobCompletion -JobId $job.ID
+								WaitForJobCompletion -JobId $job.ID -NumOfSecondsToWait 600
                                 $job = Get-AzureSiteRecoveryJob -Id $job.ID
 								Assert-True { $job.State -eq "Succeeded" }
 							}
@@ -134,7 +134,7 @@ function Test-E2E_CreateAndAssociate
 			# Validate_ProfileAssociation_JobSucceeded
 			if ($Validate_ProfileAssociation_JobSucceeded -eq $true)
 			{
-				WaitForJobCompletion -JobId $job.ID
+				WaitForJobCompletion -JobId $job.ID -NumOfSecondsToWait 600
                 $job = Get-AzureSiteRecoveryJob -Id $job.ID
 				Assert-True { $job.State -eq "Succeeded" }
 			}

diff --git a/...erviceManagement/RecoveryServices/Commands.RecoveryServices/RecoveryServicesCmdletBase.cs b/...erviceManagement/RecoveryServices/Commands.RecoveryServices/RecoveryServicesCmdletBase.cs
@@ -180,9 +180,10 @@ protected void ValidateUsageById(string replicationProvider, string paramName)
             }
             else
             {
-                this.WriteDebugWithTimestamp(
+                this.WriteWarningWithTimestamp(
+                    string.Format(
                     Properties.Resources.IDBasedParamUsageNotSupportedFromNextRelease,
-                    paramName);
+                    paramName));
             }
         }
 

diff --git a/...ecoveryServices/Commands.RecoveryServices/Service/SetAzureSiteRecoveryProtectionEntity.cs b/...ecoveryServices/Commands.RecoveryServices/Service/SetAzureSiteRecoveryProtectionEntity.cs
@@ -187,9 +187,10 @@ public override void ExecuteCmdlet()
                                 }
                                 else
                                 {
-                                    this.WriteDebugWithTimestamp(
+                                    this.WriteWarningWithTimestamp(
+                                        string.Format(
                                         Properties.Resources.MandatoryParamFromNextRelease,
-                                        "ProtectionProfile");
+                                        "ProtectionProfile"));
                                     string pcId = this.ProtectionContainerId ?? this.ProtectionEntity.ProtectionContainerId;
                                     var pc = RecoveryServicesClient.GetAzureSiteRecoveryProtectionContainer(
                                         pcId);

diff --git a/...veryServices/Commands.RecoveryServices/Service/StartAzureSiteRecoveryCommitFailoverJob.cs b/...veryServices/Commands.RecoveryServices/Service/StartAzureSiteRecoveryCommitFailoverJob.cs
@@ -95,9 +95,10 @@ public override void ExecuteCmdlet()
             {
                 if (string.IsNullOrEmpty(this.Direction))
                 {
-                    this.WriteDebugWithTimestamp(
-                        Properties.Resources.MandatoryParamFromNextRelease,
-                        Constants.Direction);
+                    this.WriteWarningWithTimestamp(
+                        string.Format(
+                            Properties.Resources.MandatoryParamFromNextRelease,
+                            Constants.Direction));
                 }
 
                 switch (this.ParameterSetName)

diff --git a/...coveryServices/Commands.RecoveryServices/Service/StartAzureSiteRecoveryTestFailoverJob.cs b/...coveryServices/Commands.RecoveryServices/Service/StartAzureSiteRecoveryTestFailoverJob.cs
@@ -156,16 +156,18 @@ public override void ExecuteCmdlet()
             {
                 if (this.NetworkType == null)
                 {
-                    this.WriteDebugWithTimestamp(
-                        Properties.Resources.MandatoryParamFromNextRelease,
-                        Constants.NetworkType);
+                    this.WriteWarningWithTimestamp(
+                        string.Format(
+                            Properties.Resources.MandatoryParamFromNextRelease,
+                            Constants.NetworkType));
                 }
 
                 if (this.VmNetworkId != null)
                 {
-                    this.WriteDebugWithTimestamp(
-                        Properties.Resources.IDBasedParamUsageNotSupportedFromNextRelease,
-                        "VmNetworkId");
+                    this.WriteWarningWithTimestamp(
+                        string.Format(
+                            Properties.Resources.IDBasedParamUsageNotSupportedFromNextRelease,
+                            "VmNetworkId"));
                 }
 
                 if (this.NetworkType == Constants.Existing && (this.Network == null && this.VmNetworkId == null))