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

Add query structure example #6994

Open
wants to merge 61 commits into
base: main
Choose a base branch
from
Open

Add query structure example #6994

wants to merge 61 commits into from

Conversation

vagimeli
Copy link
Collaborator

Description

Adds alerting query structure example

Issues Resolved

Closed #6628

Checklist

  • By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
    For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Melissa Vagi <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
@vagimeli
Copy link
Collaborator Author

@AWSHurneyt Are you the appropriate dev to help me fill in the content gaps for alerting dashboards and visualizations?

Signed-off-by: Melissa Vagi <[email protected]>

Signed-off-by: Melissa Vagi <[email protected]>
@AWSHurneyt
Copy link
Contributor

@AWSHurneyt Are you the appropriate dev to help me fill in the content gaps for alerting dashboards and visualizations?

Integration of alerting and visualizations was part of the feature anywhere project I believe. @lezzago could you assist with the visualization questions?

Signed-off-by: Melissa Vagi <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
@vagimeli vagimeli added Needs SME Waiting on input from subject matter expert 3 - Tech review PR: Tech review in progress labels Jun 24, 2024
_observing-your-data/alerting/dashboards-alerting.md Outdated Show resolved Hide resolved
_observing-your-data/alerting/dashboards-alerting.md Outdated Show resolved Hide resolved

<img src="{{site.url}}{{site.baseurl}}/images/dashboards/create-monitor-menu.png" alt="Create monitor interface" width="400" height="400">
<img src="{{site.url}}{{site.baseurl}}/images/dashboards/add-monitor.png" alt="Adding an alerting monitor from within a visualization" width="700"/>
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed images per writing style guidelines.

Signed-off-by: Melissa Vagi <[email protected]>
@vagimeli
Copy link
Collaborator Author

vagimeli commented Aug 20, 2024

@AWSHurneyt Please re-review at your availability. Will you also provide code examples that work when I test using Dev Tools? Thank you.

@vagimeli vagimeli added backport 2.17 Backport for version 2.17 and removed backport 2.14 PR: Backport label for 2.14 labels Sep 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 - Tech review PR: Tech review in progress 4 - Doc review PR: Doc review in progress backport 2.17 Backport for version 2.17 Needs SME Waiting on input from subject matter expert
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[DOC] Add alert query structure
6 participants