-
-
Notifications
You must be signed in to change notification settings - Fork 141
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
New Check - Agent Alerts are configured #236
Comments
This one would be difficult in my opinion, only because just having the alert configured does not mean anything if you don't have the notification for it setup as well. |
Just logged off for the night, but is there no way to check for the notification piece also? Also kind of envisioning the agent tests to build up a better picture, with database mail configured, operator set up, alerts configured etc. |
Well, an alert being configured is irrelevant on whether they are using Database mail or not...there are other options for notifications. It is a matter of saying what items should be in place (or validated) to say an alert is configured properly. |
|
Hi
|
I like this :-) Good work @PsPsam , Thank you Please can you run the Unit checks to ensure that the output is formatted correctly |
PR created I hope I did i right this time. If not let me know, git is new to me and PR is scary for me :D |
Agent alerts are setup.
Glenn Berry suggests 19-25 & 825
https://www.red-gate.com/simple-talk/sql/database-administration/provisioning-a-new-sql-server-instance-part-three/
Brent Ozar suggests 16-25 & 823-825
https://www.brentozar.com/blitz/configure-sql-server-alerts/
The text was updated successfully, but these errors were encountered: