Skip to content

Commit 5222693

Browse files
committed
Comments as placeholder
To be added - check if servers are registered to the vault / site
1 parent 4b13641 commit 5222693

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

src/ServiceManagement/RecoveryServices/Commands.RecoveryServices/Service/RemoveAzureSiteRecoverySite.cs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,8 +58,9 @@ public override void ExecuteCmdlet()
5858
{
5959
try
6060
{
61-
//// Check if site has registered things in it
62-
61+
// Check if site has registered servers and prevent the operation
62+
// But the rest api is exposed and can be called directly
63+
// - best to add this validation in service then before deleting a site
6364
this.ConfirmAction(
6465
this.Force.IsPresent,
6566
string.Format(Properties.Resources.RemoveSiteWarning, this.Name),

src/ServiceManagement/RecoveryServices/Commands.RecoveryServices/Service/RemoveAzureSiteRecoveryVault.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,7 @@ public override void ExecuteCmdlet()
5757
{
5858
try
5959
{
60+
// Check if vault has servers registered to it - prevent the operation.
6061
this.ConfirmAction(
6162
this.Force.IsPresent,
6263
string.Format(Properties.Resources.RemoveVaultWarning, this.Vault.Name),

0 commit comments

Comments
 (0)