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

docs: Add workload alerting #9938

Merged
merged 5 commits into from
Sep 19, 2024
Merged

docs: Add workload alerting #9938

merged 5 commits into from
Sep 19, 2024

Conversation

tara-hpe
Copy link
Contributor

@tara-hpe tara-hpe commented Sep 16, 2024

Document the experimental feature: Webhooks improvement: see also #9944

Include the use cases: https://hpe-aiatscale.atlassian.net/browse/MD-458

Support workload alerting, different webhook modes, e.g., #9865

Improve the "integrations" section.

Docs Preview
https://determined-ai-docs.s3.us-west-2.amazonaws.com/previews/360dc8f29a2910aba1e59c63cbe54db3/integrations/notification/workload-alerting.html

Ticket

Description

Test Plan

Checklist

  • Changes have been manually QA'd
  • New features have been approved by the corresponding PM
  • User-facing API changes have the "User-facing API Change" label
  • Release notes have been added as a separate file under docs/release-notes/
    See Release Note for details.
  • Licenses have been included for new code which was copied and/or modified from any external code

@cla-bot cla-bot bot added the cla-signed label Sep 16, 2024
@determined-ci determined-ci requested a review from a team September 16, 2024 19:28
@determined-ci determined-ci added the documentation Improvements or additions to documentation label Sep 16, 2024
Copy link

netlify bot commented Sep 16, 2024

Deploy Preview for determined-ui canceled.

Name Link
🔨 Latest commit 43f59aa
🔍 Latest deploy log https://app.netlify.com/sites/determined-ui/deploys/66ec850265575f00083712dd

@tara-hpe tara-hpe requested review from gt2345 and removed request for a team September 16, 2024 19:28
@tara-hpe tara-hpe marked this pull request as draft September 16, 2024 19:28
@gt2345
Copy link
Contributor

gt2345 commented Sep 16, 2024

Workload alerting is based on our existing webhook feature, should we update/merge the docs

@tara-hpe tara-hpe requested a review from mackrorysd September 16, 2024 19:33
@tara-hpe tara-hpe force-pushed the docs/add-workload-alerting branch from b9fa80b to 2f09ce5 Compare September 17, 2024 17:40
@determined-ci determined-ci requested a review from a team September 17, 2024 17:40
@tara-hpe tara-hpe force-pushed the docs/add-workload-alerting branch from 0c02b11 to ded44ec Compare September 18, 2024 17:47
@tara-hpe tara-hpe marked this pull request as ready for review September 18, 2024 17:47
@tara-hpe
Copy link
Contributor Author

#9944

yes we'll have to merge #9944 first and then tweak #9938 and remove any duplicates

Support workload alerting, different webhook modes, e.g., #9865
@tara-hpe tara-hpe force-pushed the docs/add-workload-alerting branch from 053f034 to f5d63bf Compare September 19, 2024 18:52
@tara-hpe tara-hpe merged commit eb1b0de into main Sep 19, 2024
79 of 92 checks passed
@tara-hpe tara-hpe deleted the docs/add-workload-alerting branch September 19, 2024 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants