Added return type boolean log for Remove Flow Log Command (#26958)

Co-authored-by: Riddhi Nilawar <[email protected]>
Azure · Jan 6, 2025 · e6c98e5 · e6c98e5
1 parent 705bbe3
commit e6c98e5
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/src/Network/Network/ChangeLog.md b/src/Network/Network/ChangeLog.md
@@ -19,6 +19,7 @@
 --->
 
 ## Upcoming Release
+* Updated `Remove-AzNetworkWatcherFlowLog` command to return boolean value
 * Updated vnv and ipam cmdlets
 * Allowed TA interval to be set as 0 incase TA is disabled
 * Onboarded Azure Virtual Network Manager Cmdlets for IpamPool

diff --git a/src/Network/Network/NetworkWatcher/FlowLog/RemoveAzNetworkWatcherFlowLogCommand.cs b/src/Network/Network/NetworkWatcher/FlowLog/RemoveAzNetworkWatcherFlowLogCommand.cs
@@ -146,12 +146,20 @@ public override void Execute()
             this.Name,
             () =>
             {
-                this.FlowLogs.Delete(this.ResourceGroupName, this.NetworkWatcherName, this.Name);
-
-                if (PassThru)
+                try
                 {
+                    this.FlowLogs.Delete(this.ResourceGroupName, this.NetworkWatcherName, this.Name);
                     WriteObject(true);
                 }
+                catch (Exception ex)
+                {
+                    WriteObject(false);
+                    throw ex;
+                }
+                if (PassThru)
+                {
+                    WriteObject("PassThru: " + true);
+                }
             });
         }
     }