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.

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

Inspector - Backup Space report including backups to other drives #168

Closed
PeterJenner1992 opened this issue Oct 18, 2019 · 2 comments
Closed
Assignees
Labels

Comments

@PeterJenner1992
Copy link

@PeterJenner1992 PeterJenner1992 commented Oct 18, 2019

i'm looking at the backup space section. And we've had a warning come through for lack of space on the backup drive.
When i've looked into it, it looks as though it's been triggered by a backup which is backing up to a different location.
So you have backup location A, which the report is saying doesn't have space
And Backup location B where the Database is going (which has plenty of space for the backup)
it's like it's ignoring where the backup is actually taking to

@Adedba Adedba changed the title Backup Space report including backups to other drives Inspector - Backup Space report including backups to other drives Oct 18, 2019
@Adedba

This comment has been minimized.

Copy link
Collaborator

@Adedba Adedba commented Oct 18, 2019

Thanks for reporting the issue @PeterJenner1992

Unfortunately you have hit a design restriction , we wrongly assumed that all servers being reportied on by your Central server would all use the same backup path when indeed this is not actually going to be the case! but I think we can make that better for V2.

so the way it is now is....
Inspector.Settings has a setting called BackupPath ...or something similar and this has a single value which is used for all the servers which are in Inspector.CurrentServers WHERE Isactive = 1
Problem with that is that you may have servers and/or databases being reported on which are taking backups to a a different location and this is where it falls over.

Leave this one with me and I will find a way to make this work better and get the change into V2.

@Adedba Adedba self-assigned this Oct 18, 2019
@Adedba Adedba added the Development label Oct 18, 2019
@Adedba Adedba added this to the InspectorV2 Release milestone Oct 18, 2019
@Adedba Adedba added this to To do in Inspector V2 Oct 18, 2019
@Adedba Adedba moved this from To do to Testing in Inspector V2 Oct 25, 2019
@Adedba Adedba moved this from Testing to Done in Inspector V2 Nov 1, 2019
@Adedba

This comment has been minimized.

Copy link
Collaborator

@Adedba Adedba commented Nov 4, 2019

The backupsPath setting in the Inspector. Settings table will support a comma delimited list of paths, each path will be checked individually and highlight individually.

@Adedba Adedba closed this in f8ad742 Nov 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Inspector V2
  
Done
2 participants
You can’t perform that action at this time.