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

unexpected end of data error with metrics in v2.6.0 #5065

Closed
PsychoSid opened this Issue Jan 4, 2019 · 1 comment

Comments

Projects
None yet
1 participant
@PsychoSid
Copy link

PsychoSid commented Jan 4, 2019

Got an issue with our custom exporter in Prometheus 2.6 (worked in 2.5 and below).

We are now getting-
msg="append failed" err="unexpected end of data, got "EOF""

Errors - it's not a timeout as it's well below the duration/timeout values.

Curl'ing the data looks good and piping through "promtool check metrics" complains about the lack of help text but that is all.

The only thing I can possible see it that some parts have spaces in (some don't) for example:-

msg_queues_is_durable{server="myserver",vpn="#config-sync",queue="#CFGSYNC/OWNER/ONE/VPN/TWO VPNNAME/CFG"} 1.0

But this has never been an issue before, this from the openmetricsparse_test.go:-

func TestOpenMetricsParseErrors(t *testing.T) {

  | cases := []struct {
  | input string
  | err string
  | }{
  | {
  | input: "",
  | err: "unexpected end of data, got "EOF"",
  | },

@PsychoSid

This comment has been minimized.

Copy link
Author

PsychoSid commented Jan 7, 2019

It was the encoding. It's not gzip and the type wasn't coming in as text/plain but text/html.

I think this commit was the one that changed the behaviour.

https://github.com/prometheus/prometheus/pull/4912/files

The error message was a little odd. But makes sense ultimately.

Thanks again to Brian for pointing me in the right direction on this. Using https and the endpoint on k8s and going through a few layers made it fun :)

@PsychoSid PsychoSid closed this Jan 7, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.