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

Create new grafana dashboard based on 11925 #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

titansmc
Copy link

@towolf
Copy link

towolf commented Apr 5, 2020

Hi, I've made this. If you want any adjustments, I can touch it up more.

@titansmc
Copy link
Author

titansmc commented Apr 7, 2020

@towolf That is fine to me, I was just wondering if it makes sense to upload the dashboard you have in this repo.

Cheers.

@towolf
Copy link

towolf commented Apr 7, 2020

Dunno, some people prefer simpler dashboards. I like to have more detail. Because, you never know.

@MiningPickaxe
Copy link

I had issues with that grafana dashboard, most querys don't match (at least with the docker image of the jitsi-prom-exporter).
For example: largest_conference{instance = $instance} had to be largest_conference{app="jitsi",jvb_instance= $instance} in my case.
The extraction of the variable didn't work either, I havent fixed that yet

@towolf
Copy link

towolf commented Apr 10, 2020

Hi @MiningPickaxe
You're right that largest_conference has a jvb_instance in its labels:

# curl -s 10.6.28.11:8999/metrics | grep largest 
# HELP largest_conference The current number of participants in the largest conference
# TYPE largest_conference gauge
largest_conference{app="jitsi",jvb_instance="jvb"} 0

But then, once its ingested in Prometheus there's also an instance label:
image

And that one refers to the host from where the metrics were scraped from.

And on the dashboard the $instance is grabbed like this. So I don't know why that doesn't work for you:
image

@MiningPickaxe
Copy link

Hi @towolf
could you maybe share your prometheus config related to this jitsi-prom-exporter?

@towolf
Copy link

towolf commented Apr 12, 2020

@MiningPickaxe since we are running this standalone I just configured a static config for Prometheus:

# cat /etc/prometheus/jobs/static/jitsi.yml
- targets:
    - '10.6.28.11:8999'
  labels:
    job: "jitsi"
    instance: "h7020.example.com"

EDIT:
Ah, now that I'm actually looking, we are adding the instance label ourselves ... 🤦

@MiningPickaxe
Copy link

Ah ok :) Makes more sense now.
Once the jitsi-prom-exporter uses MUC Nicknames instead of Usernames (which would allow for monitoring multiple bridges) and passes that over to prom, it would be nice to be able to filter through those. But that will probably be implemented SOON™

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

Successfully merging this pull request may close these issues.

3 participants