-
Notifications
You must be signed in to change notification settings - Fork 2
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
Content-type error since using Prometheus 3 #5
Comments
I ran into this as well. Could you explain your work around? |
@TheAceMan |
Ahh, thanks. Didn't realize the grep was ran after the change to show the modifications. Got it working. But this work around will be cleared on pod restart in the kubernetes cluster. Will have to look into a init container or some persistence if this is not going to be fixed. |
You just need to create your own docker image by overloading the initial one.
|
A bit hackish but the init container is working. Creating a image would require a container registry and pushing new images as zwave-js-ui base updates. Got it all working and hopefully it's a short term. |
Fixed by #6 |
Ran into this as well. The migration guide for Prometheus actually calls this failure out: https://prometheus.io/docs/prometheus/latest/migration/#scrape-protocols. If you cannot update to a version of E.g. If you're running this in k8s (which, surprisingly someone else in this thread seems to be doing!) and using the Prometheus Operator (or at least their CRDs), you can set apiVersion: monitoring.coreos.com/v1
kind: ServiceMonitor
metadata:
name: z-wave-js
namespace: z-wave-js
spec:
endpoints:
- path: /metrics
port: ui
+ fallbackScrapeProtocol: PrometheusText1.0.0
selector:
matchLabels:
app.kubernetes.io/component: z-wave-js
app.kubernetes.io/name: z-wave-js Or whatever. You should be able to do similar with the And for the sake of SEO for the next person that runs into this, the error I was receiving looked something like this:
|
Hello,
Upgrading to Prometheus 3, I encountered a content-type issue.
Prometheus expects 'text/plain' while the plugin returns 'text/html'
I fixed it locally but I think I would not be the only one.
That's why I give the correction:
The text was updated successfully, but these errors were encountered: