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

sp_Blitz check ID 93 on Linux thinks backups exist on the same drive #1995

Closed
ucjonathan opened this Issue Mar 9, 2019 · 3 comments

Comments

Projects
None yet
3 participants
@ucjonathan
Copy link

ucjonathan commented Mar 9, 2019

Version of the script
SET @Version = '7.3';
SET @VersionDate = '20190219';

What is the current behavior?
Check 93 thinks that backups are on the same drive as the database. The layout is

/sql/data
/sql/backup

These are separate EBS volumes on the EC2 instance. I know it's probably just doing string comparison, but I would recommend skipping this check on paths that start with /

If the current behavior is a bug, please provide the steps to reproduce.

Run sp_Blitz on a linux installation of SQL server.

What is the expected behavior?

Probably to skip the check entirely.

Which versions of SQL Server and which OS are affected by this issue? Did this work in previous versions of our procedures?

SQL Server 2017 cu13 on RHEL 7.4

@BrentOzar

This comment has been minimized.

Copy link
Member

BrentOzar commented Mar 9, 2019

@bsquidwrd

This comment has been minimized.

Copy link
Contributor

bsquidwrd commented Mar 14, 2019

I can implement this fix as it seems simple enough to get my hands wet on this project 👍

@BrentOzar

This comment has been minimized.

Copy link
Member

BrentOzar commented Mar 19, 2019

Nice job! Added to the March release, and will credit both of you in the post. Thanks!

@BrentOzar BrentOzar closed this Mar 19, 2019

@BrentOzar BrentOzar added this to the 2019-03 milestone Mar 19, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.