-
Notifications
You must be signed in to change notification settings - Fork 11.8k
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
CloudWatch Logs: Add labels to alert and expression queries #77529
Conversation
|
||
// set the group key as the display name for sync queries | ||
for i := 1; i < len(newFrame.Fields); i++ { | ||
valueField := newFrame.Fields[i] | ||
for j := 1; j < len(newFrame.Fields); j++ { |
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.
changed this to a j
because I realized it's nested in a for loop using i
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.
nice, tested and labels are showing properly now 🎉
Hello @iwysiu!
Please, if the current pull request addresses a bug fix, label it with the |
(cherry picked from commit 64e28c0)
What is this feature?
Labels the grouped fields for sync CloudWatch Logs queries with the fields they were grouped on.
Why do we need this feature?
In order for CloudWatch Logs queries returning multiple time series to be useful in alerts, the different time series need to be labeled with their grouping information
Who is this feature for?
Users that want to use CloudWatch Logs queries with multiple time series in alerts.
Which issue(s) does this PR fix?:
Fixes #63388
Special notes for your reviewer:
Making this a bug because it should work, even if it never has.
Please check that: