-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Alerting] Update README #100478
[Alerting] Update README #100478
Conversation
|tags|The tags associated with this rule. This will eventually be removed in favor of `rule.tags`.| | ||
|createdBy|The user ID of the user that created this rule. This will eventually be removed in favor of `rule.createdBy`.| | ||
|updatedBy|The user ID of the user that last updated this rule. This will eventually be removed in favor of `rule.updatedBy`.| | ||
|rule.name|The name of this rule.| |
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.
These rule
inputs were added in this PR
- `get` | ||
- `getAlertState` | ||
- `getRuleState` | ||
- `getAlertSummary` |
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.
These privilege strings were updated in this PR
Pinging @elastic/kibana-alerting-services (Team:Alerting Services) |
## Terminology | ||
|
||
**Alert Type**: A function that takes parameters and executes actions to alert instances. | ||
> Disclaimer: We are actively working to update the terminology of the Alerting Framework. While all user-facing terminology has been updated, much of the codebase is still a work in progress. |
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.
Terminology disclaimer added here.
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
💚 Build Succeeded
Metrics [docs]Unknown metric groupsReferences to deprecated APIs
History
To update your PR or re-run it, just comment with: cc @ymao1 |
* Updating readme * Updating readme * Fix plugin list docs
💚 Backport successful
This backport PR will be merged automatically after passing CI. |
* Updating readme * Updating readme * Fix plugin list docs Co-authored-by: ymao1 <[email protected]>
Summary
Updated Alerting README to reflect updated terminology. Also updated some sections that should have been updated with recent PR changes.