-
Notifications
You must be signed in to change notification settings - Fork 387
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
[Ceph] Add Integration Package with Pool Disk data stream #5070
[Ceph] Add Integration Package with Pool Disk data stream #5070
Conversation
/test |
🌐 Coverage report
|
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!
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.
{\"stored\":2142673,\"objects\":4,\"kb_used\":6336,\"bytes_used\":6488064,\"percent_used\":8.4362458437681198e-05,\"max_avail\":25633505280}},
can we have these fields recorded and mapped ?
description: Available bytes of the pool. | ||
metric_type: gauge | ||
unit: byte | ||
- name: id |
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.
please change to pool_id
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.
In this data stream, we have pool_disk
as parent field. So the current output will be like ceph.pool_disk.id
. Based on the suggested scenario, we will have ceph.pool_disk.pool_id
. So I think this change would make pool redundant. WDYT?
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.
Please keept is pool_id
- name: id | ||
type: long | ||
description: Id of the pool. | ||
- name: name |
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.
please change to pool_name
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.
In this data stream, we have pool_disk as parent field. So the current output will be like ceph.pool_disk.name
. Based on the suggested scenario, we will have ceph.pool_disk.pool_name
. So I think this change would make pool redundant. WDYT?
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.
Please keep it as pool_name
Based on the response we have ingested objects, bytes_used and max_avail fields. For the |
…into package-ceph_pool_disk
…into package-ceph_pool_disk
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
Package ceph - 0.5.0 containing this change is available at https://epr.elastic.co/search?package=ceph |
What does this PR do?
Checklist
How to test this PR locally
elastic-package test
Related issues