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

Metric encoding is broken when using --enable-gzip-encoding flag. #1371

Closed
AlexShemeshWix opened this issue Jan 31, 2021 · 6 comments · Fixed by #1372
Closed

Metric encoding is broken when using --enable-gzip-encoding flag. #1371

AlexShemeshWix opened this issue Jan 31, 2021 · 6 comments · Fixed by #1372
Assignees
Labels
helm kind/bug Categorizes issue or PR as related to a bug.

Comments

@AlexShemeshWix
Copy link

What happened: Ive enabled gzip compression for kube-state-metrics using extraArgs param in helm chatr (--enable-gzip-encoding)

What you expected to happen: i expect metrics to be compressed but accessible

How to reproduce it (as minimally and precisely as possible): Deploy latest kube-state-metrics chart 2.11.0 with extraAgrs that contains --enable-gzip-encoding param. Wait till kube-state-metrics pod starts. Try to access metrics using any browser. It will fail with "Failed to load resource: net::ERR_CONTENT_DECODING_FAILED" message in console.

Anything else we need to know?:

Environment:

  • kube-state-metrics version: v1.9.7
  • Kubernetes version (use kubectl version): v1.17.13
  • Cloud provider or hardware configuration: AWS
  • Other info:
@AlexShemeshWix AlexShemeshWix added the kind/bug Categorizes issue or PR as related to a bug. label Jan 31, 2021
@lou-lan
Copy link
Contributor

lou-lan commented Feb 1, 2021

It looks like the curl command can be accessed successfully, but the browser or Prometheus access fails.

@lou-lan
Copy link
Contributor

lou-lan commented Feb 1, 2021

I will try to find the cause of the problem.

@lou-lan
Copy link
Contributor

lou-lan commented Feb 1, 2021

/assign

@AlexShemeshWix
Copy link
Author

Any updates?

@lou-lan
Copy link
Contributor

lou-lan commented Mar 1, 2021

Any updates?

Already merged into rc2 branch.

@lilic
Copy link
Member

lilic commented Mar 19, 2021

This is fixed in master as well, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
helm kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants