-
Notifications
You must be signed in to change notification settings - Fork 6k
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
[feature] collects windows metrics from wmi_exporter #6001
Comments
hi @irvintim, will you able to test this collector? |
Yes we'd be very interested in testing this. I am out of the office for
this week but we could plan to do some testing starting next week.
…On Mon, May 20, 2019, 7:03 AM Ilya Mashchenko ***@***.***> wrote:
hi @irvintim <https://github.com/irvintim>, will you able to test this
collector?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#6001?email_source=notifications&email_token=ABLU4VVXXDAHEIK3Q74OXQ3PWKVRVA5CNFSM4HMO25XKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVY5SKY#issuecomment-494000427>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABLU4VTUZHJXQLQNFSGZT7TPWKVRVANCNFSM4HMO25XA>
.
|
Nice, ping me when you are ready then 👍 |
@ilyam8 I am caught up following my outage, and would like to start testing this collector. Looking over what I think you are doing, I need to install the wmi_exporter on my Windows test server, and then on a linux box running netdata use your collector go plugin to receive the data from the windows box. Does the collector assign this data to a specific host representing the windows box, or is it shown as data for the netdata server itself? If you have any specific instructions for me that would be great. Thanks for your efforts!! Tim |
Yes @irvintim Here you go windows server:
linux server:
After it you will see your windows servers on the netdata dashboard as |
I followed your instructions but am not seeing the The error_log has the following error: Here is the config that I am using:
When I go to the URL listed above I get the following -- this is just a snippet of the frst few lines, lots more data is returned:
Version: |
#[ GLOBAL ]
update_every: 1
autodetection_retry: 300
priority: 70000
#[ JOBS ]
jobs:
- name: sutter
url: http://10.10.0.11:9182/metrics |
That did the trick.
I'll have to dig into that to see what's up there.
I'll dig into the CPU issue, and also get this test running on a couple more servers and let them fly for a few days to see how things pan out. Thanks! Tim |
I tried wmi_exporter 32 and 64 bit (it's a 64-bit machine), but got the same error. I have opened an issue on the wmi_exporter github project. |
We could add windows icon https://fontawesome.com/v4.7.0/icon/windows (instead of puzzle-piece) , it would look better imo |
@ilyam8 some follow up from our testing:
I do understand your point on the presentation of the data in a section of the main server dashboard, but ultimately the solution we are looking for would treat each host independently and not intermix data from one server into the dashboard of another server. But what you have done here is a great start.
Option 3 is probably the easiest to implement but unnecessarily uses resources and that's not a good thing for a monitoring service. |
👍 have you tried to increase http timeout and lower collection frequency? #[ JOBS ]
jobs:
- name : sutter
url : http://10.10.0.11:9182/metrics
update_every : 5
timeout : 5
i understand your point, it was a quick implementation, just to have something.
No way they will do it, and i agree with it, they are doing prometheus exporter and it is not responsible for pushing metrics in netdata format. |
I guess this could be done on the cloud side @cakrit |
I had already set the timeout to 5, as wmi_exporter was responding in > 1 second each time and so I was getting no data initially. I just set the update_every value to 5, I'll let you know if that changes anything. Someone responded to my issue on the wmi_exporter site and suggested we try: https://github.com/leoluk/perflib_exporter . From the README on perflib_exporter:
|
The
|
it is ok, not an error actually (plugin logs it if collection takes more than 1 second) |
Feature idea summary
#92 is very popular and important but it won't be implemented in the near feature, a lot of work.
What we can do is to parse prometheus wmi_exporter (give it a ⭐ btw) metrics and convert it to netdata format (charts 😄 ). This can be done relatively fast.
The result will be something like
cgroups
plugin.Expected behavior
new
go.d.plugin
module that collects windows metrics fromwmi_exporter
The text was updated successfully, but these errors were encountered: