Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Tape/NAS/File-Copy Support #72

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Tape/NAS/File-Copy Support #72

wants to merge 4 commits into from

Conversation

Jannos-443
Copy link
Contributor

@Jannos-443 Jannos-443 commented Jan 29, 2021 •
edited
Loading

Added:

  • VeeamV10 NAS Backup Monitoring
  • File Copy Monitoring
  • Tape Monitoring

Jannos-443 and others added 3 commits January 29, 2021 15:14
@Jannos-443
Update PRTG-VeeamBRStats.ps1
010d6fa 
VeeamV10 NAS Backup Support
@Jannos-443
Update PRTG-VeeamBRStats.ps1
bfa59cd
@Jannos-443
Tape/NAS/File-Copy Support
bfb54c5 
Added Tape/NAS/File-Copy Backup Monitoring.
@Jannos-443 Jannos-443 changed the title Update PRTG-VeeamBRStats.ps1 Tape/NAS/File-Copy Support Jan 31, 2021
@vMarkusK
Copy link
Owner

vMarkusK commented Feb 2, 2021

Wow, this looks pretty awesome. Great enhancement!

Unfortunately, I'm currently not able to verify it end-to-end. I'll try to get some folks to test it out.

@vMarkusK
Copy link
Owner

vMarkusK commented Feb 2, 2021

@Jannos-443 Do you think extending the debug section is required?

@vMarkusK vMarkusK linked an issue Feb 2, 2021 that may be closed by this pull request
PRTG-VeeamBRStats.ps1 - Tape Jobs? #2
Open
@Jannos-443
Copy link
Contributor Author

@Jannos-443 Do you think extending the debug section is required?

I think the SessionObject should be enough.

@Jannos-443
Changes
e7c09d4 
- BackupTransfer and BackupRead independend from Region "Backup Jobs" 
- Added "Unrecognized Tapes" to the result
- Added "Hanging Jobs" detection (Jobs running longer than 24h
- changed LimitMaxError to 0.1 to be in the green area while on zero.
- Minor Formatting changes
@Jannos-443
Copy link
Contributor Author

In my envirement the Script runs up to 9 Minutes, i had to change the Timeout to 900. (15 minutes)
The Tape Job "Get-Command" is as slow as Tape ;)

@vMarkusK
Copy link
Owner

Do you use SQL Express in your environment?

@Jannos-443
Copy link
Contributor Author

SQL Standard on the Veeam Server, but maybe it's due to some larger File to Tape Jobs we are running.

@vMarkusK
Copy link
Owner

Is there any CPU or disc contention during the report execution?

@vMarkusK
Copy link
Owner

vMarkusK commented Mar 3, 2021

@Jannos-443 have you already verified your changes against v11?

I have recently created a new branch for the required changes for v11.

@Jannos-443
Copy link
Contributor Author

Will check the CPU and Disk usage.

Upgrade to v11 is planned around mid march.

@vMarkusK
Copy link
Owner

vMarkusK commented Mar 4, 2021

I have merged #76 into master might you please verify your additions with the new master?
The Script should be backward compatible.

@Jannos-443
Copy link
Contributor Author

Sorry for the delay but our Cloud Connect Provider is upgrading on 16.4 so we have to wait for them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

PRTG-VeeamBRStats.ps1 - Tape Jobs?
2 participants
@Jannos-443 @vMarkusK