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

🔥 Prometheus Endpunkt hinzugefügt #687

Merged
merged 7 commits into from
May 21, 2024
Merged

Conversation

gnmyt
Copy link
Owner

@gnmyt gnmyt commented May 21, 2024

🔥 Prometheus Endpunkt hinzugefügt

MySpeed erhält nun wie gewünscht einen Prometheus-Endpunkt. Dieser zeigt dann immer die Daten des letzten Tests an. Der Endpunkt zeigt dabei folgende Werte an: Download, Upload, Ping, Time, Current Server. Dies schließt #515

Hier eine Beispiel-Konfiguration für MySpeed in der prometheus.yml:

  - job_name: 'myspeed'
    scrape_interval: 1m # Je nach Testzeitraum anpassen
    scrape_timeout: 8s
    metrics_path: /api/prometheus/metrics
    basic_auth:
      username: prometheus
      password: dein-passwort # Nur notwendig, wenn ein Passwort in der Weboberfläche gesetzt wurde
    static_configs:
      - targets: ['localhost:5216']

Wichtig ist es, das scrape_interval anzupassen. Weiß man bereits vorher, dass man nur automatisierte Tests laufen lässt und diese auf jede Stunde gestellt hat, reicht es hier aus, 1h festzulegen. So spart man sich die Bandbreite und unnötige Duplikate. Macht man öfters manuelle Tests und möchte sicher gehen, dass diese angezeigt werden, kann man hier variieren.

Screenshots

image

Änderungen vorgenommen an ...

  • Server
  • Client
  • Dokumentation
  • Sonstiges: ___

@gnmyt gnmyt added the enhancement New feature or request label May 21, 2024
@gnmyt gnmyt self-assigned this May 21, 2024
@gnmyt gnmyt linked an issue May 21, 2024 that may be closed by this pull request
2 tasks
@gnmyt gnmyt merged commit 15475d6 into development May 21, 2024
@gnmyt gnmyt deleted the features/prometheus branch May 21, 2024 17:16
@CrazyWolf13
Copy link

Hey @gnmyt

Vielen Dank für den Prometheus Endpunkt, habe den soeben eingerichtet und er funktioniert super!

Hast Du bereits ein Grafana Dashboard dazu erstellt, oder eines gefunden?

@fynnsh Hast Du ein Grafana Board auf basis vom Prometheus Endpunkt?


For all the internation people, and love to OSS, I just asked in german, if there is yet a public myspeed grafana dashboard avaiable, so not everyone has to spend hours on creating their own.

@fynnsh
Copy link

fynnsh commented Aug 6, 2024

Hi @CrazyWolf13 @gnmyt, freut mich das wir einen Prometheus Endpunkt haben. Kann später ein Dashboard zusammenbauen

@fynnsh
Copy link

fynnsh commented Aug 6, 2024

MySpeed-1722980881766.json

Werde das noch verbessern und auf Grafana publishen wenn ich Zeit habe

@CrazyWolf13
Copy link

@fynnsh Bei mir hat der direkte import leider nicht funktioniert, nutzt du eine InfluxDB dazwischen oder direkt von Prometheus ?

Weil auswählen kann ich meine Prometheus instance, jedoch zeigt es mir dann nur "No Data"

@CrazyWolf13
Copy link

CrazyWolf13 commented Aug 7, 2024

Ändern der Variable Instance zu Text und dann IP+Port hineinfügen hat funktioniert, oben zeit es mir eben garkeine auswahl an:
grafik

EDIT:
grafik
So kann es gefixt werden.

@serega404
Copy link

Hello, I use victoria metrics, it is compatible with prometheus, I added a data collection config, but I got this error

   warn        VictoriaMetrics/app/vmselect/main.go:297        error in "/api/v1/series?end=1726765514&match%5B%5D=label_values%28myspeed_ping%2C%29&start=1726743914": cannot parse matches[]=label_values(myspeed_ping,): expecting metricSelector; got "label_values(myspeed_ping)"
   warn        VictoriaMetrics/app/vmselect/main.go:297        error in "/api/v1/series?end=1726767499&match%5B%5D=label_values%28myspeed_ping%2C%29&start=1726745899": cannot parse matches[]=label_values(myspeed_ping,): expecting metricSelector; got "label_values(myspeed_ping)"

/api/prometheus/metrics data:
msedge_LsBWSnhKrq

I do not know which side is the problem, but I have an assumption that the problem is in the empty lines

@CrazyWolf13
Copy link

Hi
Don't know victoria, but it may be a good idea to test on your end with prometheus, just to be sure where the problem actually relies

@serega404
Copy link

I launched prometheus and made sure that the collection of metrics works, I also updated victoria metrics and their metric collector and everything worked, apparently the problem was in the old version of their metric collector

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

Successfully merging this pull request may close these issues.

Add NodeExporter Function to Scrape Metrics [Feature]
4 participants