vMarkusK / Advanced-PRTG-Sensors

Collection of my Advanced PRTG Sensors
https://mycloudrevolution.com/
Apache License 2.0
69 stars 29 forks source link

Add ability to monitor Cloud Repositories #40

Closed homern closed 4 years ago

homern commented 5 years ago

with Get-VBRBackupRepository on a Cloud Repository the Sizes return with -1 so it Calculates to 100% Free Space

vMarkusK commented 5 years ago

Thanks for reporting this issue.

Might you please share the return of this command to verify a possible workaround:

Get-VBRBackupRepository | Select Type
homern commented 5 years ago

thanks for the fast reaction, the output is:

PS C:_XX> Get-VBRBackupRepository | Select Type Type CifsShare Cloud PS C:_XX>

homern commented 5 years ago

i also get an Exception at line 169 if I don't remove the .tolower() for this Repo, the CIFS is working fine

$HostName = $(Get-VBRServer | Where-Object {$_.Id -eq $r.HostId}).Name.ToLower()

C:_XX\Untitled1.ps1 : You cannot call a method on a null-valued expression.

  • CategoryInfo : NotSpecified: (:) [Write-Error], WriteErrorException
  • FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,Untitled1.ps1

C:_XX\Untitled1.ps1 : at Get-vPCRepoInfo, C:_XX\Untitled1.ps1: line 169 at , C:_XX\Untitled1.ps1: line 510

  • CategoryInfo : NotSpecified: (:) [Write-Error], WriteErrorException
  • FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,Untitled1.ps1
vMarkusK commented 5 years ago

Might you please verify the Version of the script in PR #41

homern commented 5 years ago

this works now without the Error Message for the Cloud Repo and also Removes the Cloud Repo (as said in the Commit)

vMarkusK commented 5 years ago

Great.

Are you fine with that solution?

homern commented 5 years ago

for the moment its ok, but for the long run it would be nice to monitor the Storage left in Cloud Backup Repos

vMarkusK commented 5 years ago

I fully agree with you.

I will try to build a test environment for that that feature requests. But that will take some time.

homern commented 5 years ago

ok, I will close the Issue now and wait for the feature

thanks

vMarkusK commented 4 years ago

42 is also about this request

vMarkusK commented 4 years ago

Might you please test the new version: https://github.com/mycloudrevolution/Advanced-PRTG-Sensors/blob/IncludeCloudRepos/Veeam/PRTG-VeeamBRStats.ps1