Feature metrics update - Githubissues

blakelead / couchbase_exporter

Export metrics from Couchbase Server for Prometheus consumption

Other

35 stars 18 forks source link

Feature metrics update #48

Closed pandrieux closed 3 years ago

pandrieux commented 4 years ago

define missing metrics in json files and/or metrics list doc

coveralls commented 4 years ago

Pull Request Test Coverage Report for Build 186

0 of 0 changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at ?%

Totals
Change from base Build 184:	0.0%
Covered Lines:
Relevant Lines:	0

💛 - Coveralls

blakelead commented 4 years ago

Hi @pandrieux, I'll check this asap

pandrieux commented 4 years ago

Hello @blakelead. I simply harmonized files markdown vs json. I'm actually working with couchbase 6.0, and i'm wondering if this exporter has been tested on this version ? I have some metrics empty on my dashboard (I display all metrics), but I don't yet mastered all the subtleties of couchbase/prometheus... kind of a newbie on this.

blakelead commented 4 years ago

I'll have to verify what you added because some metrics listed in the readme do not correspond to actual Couchbase API metrics. For example, error_count cannot be listed in the JSON because I calculate it from a list of XDCR errors.

I did test it with Couchbase 6.0 but not extensively. If you have specific issues with the exporter I'd be happy to help! Don't hesitate to create an issue