Merge pull request #727 from Pierre-PvF/pvf-dev

Get-RubrikVolumeGroupMount fix - Added filter for MountPoints for New-RubrikVolumeGroupMount
rubrikinc · Mar 8, 2021 · 41e2263 · 41e2263
2 parents e294309 + f9bc2b0
commit 41e2263
Show file tree

Hide file tree

Showing 3 changed files with 27 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -29,6 +29,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 
 ### Added
 
+* Added 5.3 endpoint for `Get-RubrikVolumeGroupMount`, resolving issue [Issue 729](https://github.com/rubrikinc/rubrik-sdk-for-powershell/issues/729)
 * Added support to `Get-RubrikUser` to work around all of the API endpoint changes in CDM 5.3 as per [Issue 723](https://github.com/rubrikinc/rubrik-sdk-for-powershell/issues/723)
 * Added new parameter, `SLAPrimaryClusterId`, to `Protect-RubrikFileset` & `New-RubrikSnapshot`, fixing [Issue 720](https://github.com/rubrikinc/rubrik-sdk-for-powershell/issues/720)
 * Added additional parameters: `-DoNotProtect` & `-Inherit` to `Set-RubrikSQLInstance` as requested in [Issue 717](https://github.com/rubrikinc/rubrik-sdk-for-powershell/issues/717)

diff --git a/Rubrik/Private/Get-RubrikAPIData.ps1 b/Rubrik/Private/Get-RubrikAPIData.ps1
@@ -1752,6 +1752,21 @@ function Get-RubrikAPIData {
                 Filter      = ''
                 Success     = '200'
             }
+            '5.3' = @{
+                Description = 'Retrieve information for all Volume mounts'
+                URI         = '/api/v1/volume_group/snapshot/mount'
+                Method      = 'Get'
+                Body        = ''
+                Query           = @{
+                    id          = 'source_volume_group_id'
+                    source_host = 'source_host_name'
+                    offset      = 'offset'
+                    limit       = 'limit'
+                }
+                Result      = 'data'
+                Filter      = ''
+                Success     = '200'
+            }
         }
         'Get-RubrikVMSnapshot'                = @{
             '1.0' = @{

diff --git a/Rubrik/Public/New-RubrikVolumeGroupMount.ps1 b/Rubrik/Public/New-RubrikVolumeGroupMount.ps1
@@ -23,9 +23,16 @@ function New-RubrikVolumeGroupMount
 
       .EXAMPLE
       $snapshot = Get-RubrikVolumeGroup "MyVolumeGroup" | Get-RubrikSnapshot -Latest
+      
       New-RubrikVolumeGroupMount -TargetHost "MyTargetHostName" -VolumeGroupSnapshot $snapshot -ExcludeDrives @("D","E")
-
       This will create a new VolumeGroup Mount on MyTargetHostName with the latest snapshot retrieved in the first line, while exlcluding drives D & E
+
+      New-RubrikVolumeGroupMount -TargetHost "MyTargetHostName" -VolumeGroupSnapshot $snapshot -ExcludeMountPoint @("C:\MFAPDB05Log\")
+      This will create a new VolumeGroup Mount on MyTargetHostName with the latest snapshot retrieved in the first line, while exlcluding the volume mounted on C:\MFAPDB05Log\
+
+      New-RubrikVolumeGroupMount -TargetHost "MyTargetHostName" -VolumeGroupSnapshot $snapshot -ExcludeMountPoint @("Log")
+      To exclude all MountPoints with a certain string 
+
   #>
 
   [CmdletBinding(SupportsShouldProcess = $true,ConfirmImpact = 'High')]
@@ -39,6 +46,7 @@ function New-RubrikVolumeGroupMount
     # Rubrik server IP or FQDN
     [Parameter(ParameterSetName = 'Create')]
     [Array]$ExcludeDrives,
+    [Array]$ExcludeMountPoints,
     [String]$Server = $global:RubrikConnection.server,
     # API version
     [String]$api = $global:RubrikConnection.api
@@ -82,9 +90,9 @@ function New-RubrikVolumeGroupMount
 
     foreach ($disk in $VolumeGroupSnapshot.includedVolumes)
     {
-        if ($ExcludeDrives -contains $disk.mountPoints.Replace(":\",""))
+        if ($ExcludeDrives -contains $disk.mountPoints.Replace(":\","") -Or [bool]($disk.mountPoints -match $ExcludeMountPoints) )
         {
-            Write-Verbose -Message "Skipping Disk $disk.mountPoints" -Verbose
+            Write-Verbose -Message "Disk/MountPoint $disk.mountPoints is excluded" -Verbose
         } 
         else 
         {