-
Notifications
You must be signed in to change notification settings - Fork 5k
[8.x](backport #40714) [vSphere] add triggered alarm to existing metricsets. #40812
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
Conversation
* add alert names * update host metricset * add alert metrics to all the metricsets * add changelog entry * update data_test * add triggerd alarms * added entity name * mage check --------- Co-authored-by: Niraj Rathod <[email protected]> Co-authored-by: Kush Rana <[email protected]> Co-authored-by: harnish-elastic <[email protected]> (cherry picked from commit 885a2db) # Conflicts: # metricbeat/module/vsphere/datastore/datastore.go # metricbeat/module/vsphere/datastorecluster/datastorecluster.go # metricbeat/module/vsphere/fields.go # metricbeat/module/vsphere/host/host.go # metricbeat/module/vsphere/virtualmachine/virtualmachine.go
|
Cherry-pick of 885a2db has failed: To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally |
|
This pull request doesn't have a |
|
This pull request has not been merged yet. Could you please review and merge it @niraj-elastic? 🙏 |
2 similar comments
|
This pull request has not been merged yet. Could you please review and merge it @niraj-elastic? 🙏 |
|
This pull request has not been merged yet. Could you please review and merge it @niraj-elastic? 🙏 |
|
This pull request is now in conflicts. Could you fix it? 🙏 |
niraj-elastic
left a comment
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
Description
Added
alert.namesfield to fetch information related to alert in all the metricsets.Checklist
CHANGELOG.next.asciidocorCHANGELOG-developer.next.asciidoc.Related issues
This is an automatic backport of pull request #40714 done by [Mergify](https://mergify.com).