dcu / mongodb_exporter

MongoDB exporter for prometheus.io
MIT License
356 stars 216 forks source link

Fix issue #127 #128

Open vkuznet opened 4 years ago

hloeung commented 2 years ago

Confirmed this fixes it for me, with MongoDB 4.4.11.

pauldintel commented 7 months ago

@vkuznet @hloeung what is the fix and how this working for you ? I am using the latest 1.0 binary published in website and getting the issue. I want to use Grafana dashboard and not planned to switch to percona . so any help would be appreciated . image

vkuznet commented 7 months ago

I didn't follow the main branch and don't know if it was resolved. I only provided a fix for whatever mongodb we used at that time. According to my first post there are still open issues: https://github.com/dcu/mongodb_exporter/issues/127, https://github.com/dcu/mongodb_exporter/issues/122, and https://github.com/dcu/mongodb_exporter/issues/132

pauldintel commented 7 months ago

@vkuznet -well I see you have said it is fixed though here ... not sure what I am missing ! Are you saying panic issue is not resolved it ? https://github.com/dcu/mongodb_exporter/issues/127

I am getting Panic issue only when that use/pass exist in database except I am getting authentication issue. Hence it is weird deadlock condition now !

pauldintel commented 7 months ago

Confirmed this fixes it for me, with MongoDB 4.4.11.

Can you please share the details . How this works for you .. I am on Mongo 7 and still getting server Panic problem : image

vkuznet commented 7 months ago

@vkuznet -well I see you have said it is fixed though here ... not sure what I am missing ! Are you saying panic issue is not resolved it ? #127

I am getting Panic issue only when that use/pass exist in database except I am getting authentication issue. Hence it is weird deadlock condition now !

Well, it is not clear from your posting what do you use, the stock MongoDB executable or you patched its code with this PR and recompiled the MongoDB. If former, as I wrote it seems that this PR is not merged into mainstream and therefore mainstream code will still fail. If latter, then you claim you used MongoDB 7 while this PR was developed for earlier version of Mongo, therefore it may or may not work. In any case, my advise to get source code, apply the patch, resolve conflicts (if any) and recompile it. Then you can use it to see if it will still work for you.