- Notifications
You must be signed in to change notification settings - Fork914
docs: add oom/ood to notifications#16582
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
|
@defelmnq@stirby after talking with@bpmct about where this fits in, I put the tf block in a newresource monitoring page I'd also like to add aResource monitoring section that introduces it inExtending templates - I don't think it's a blocker for this PR, but I want people to be able to find it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
🔥 LGTM
Just a question - should we precise that the alerts are sent through SMTP ? (Means the deployment needs to have SMTP configured.)
## Prerequisites | ||
Notifications are sent through SMTP. | ||
Configure Coder to [use an SMTP server](../../monitoring/notifications.md#smtp-email). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
@defelmnq definitely yes to letting them know it's through SMTP - how does this seem?
9251e0d
intomainUh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
previews: