Skip to content
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

Go memstats no longer reported in 1.0 #7272

Closed
francisdb opened this issue Sep 9, 2016 · 3 comments
Closed

Go memstats no longer reported in 1.0 #7272

francisdb opened this issue Sep 9, 2016 · 3 comments
Assignees
Milestone

Comments

@francisdb
Copy link

Since this is not mentioned in the changelog I guess this is a bug?

Influxdb 0.13

> curl -s http://127.0.0.1:8086/debug/vars | grep memstats
"memstats": {"Alloc":240914768,"TotalAlloc":5988867901232,"Sys":8533 ...
>

Influxdb 1.0

> curl -s http://127.0.0.1:8086/debug/vars | grep memstats
>
@jsternberg
Copy link
Contributor

It looks like it is not under "runtime". I'm investigating to see if that's where it's supposed to be or if it got put into the wrong section when we refactored expvars.

@jsternberg jsternberg self-assigned this Sep 9, 2016
@jwilder
Copy link
Contributor

jwilder commented Sep 9, 2016

I see them under runtime in 1.0.

$ curl -s localhost:8086/debug/vars | grep runtime
"runtime": {"name":"runtime","tags":{},"values":{"Alloc":6187664,"Frees":72753,"HeapAlloc":6187664,"HeapIdle":5644288,"HeapInUse":8249344,"HeapObjects":19503,"HeapReleased":0,"HeapSys":13893632,"Lookups":41,"Mallocs":92256,"NumGC":5,"NumGoroutine":28,"PauseTotalNs":812000,"Sys":18757880,"TotalAlloc":20621504}},

memstats and cmdline are builtin expvar keys which were probably implicitly added in 0.13.

@jsternberg
Copy link
Contributor

Fixed via #7277.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants