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

feat(prometheus): Docker Daemon scraping for monitoring #1865

Merged
merged 8 commits into from
Aug 6, 2024

Conversation

24kushang
Copy link
Contributor

Docker Daemon scraping for monitoring

Summary:

This PR works on adding the functionality of monitoring Docker daemon using Open Telemetry Collector service. This would allow the monitoring of Docker Daemon and hence giving system statistics.

Changes:

  1. New Service in otel-collector config
    • Introduced a new service named metrics which generates the docker daemon.
    • This connects to /var/run/docker.sock file and generates the docker service metrics.
  2. Port Exposed
    • Exposed 9464 port, so that Prometheus can scrape the metrics and can be further used in creating dashborads.

Contributor:
@24kushang


I need some guidance in connecting using the internal docker network.

@ltorje-8x8 ltorje-8x8 self-requested a review July 21, 2024 18:51
Removing testing code from the commit
@24kushang
Copy link
Contributor Author

image
Screenshot of the docker_statistics dashboard

@ltorje-8x8
Copy link

@24kushang
Copy link
Contributor Author

Would add the README.md in the next commit in the same PR.

prometheus/README.md Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
prometheus/README.md Outdated Show resolved Hide resolved
@bgrozev bgrozev merged commit 27150a2 into jitsi:master Aug 6, 2024
1 check passed
saghul added a commit that referenced this pull request Sep 26, 2024
* 9e17f95 feat(jigasi): allow unique account UID for each entry even if the same auth is used (#1917)
* ad2a577 fix(jigasi): transcriber VOSK URL variable into compose file (#1916)
* 6c08391 feat(jigasi): transcriber configuration supports vosk URL (#1915)
* 1db83ed feat(ldap): Start and configure only if enabled.
* ceb4a51 feat(prosody): Skip some modules for prosody-jvb.
* e7786d5 feat(jigasi): transcriber configuration for OCI and whisper and to skip saving (#1909)
* c5afcde feat(jigasi): support graceful shutdown via sidecar (#1908)
* 5d40297 feat(jigasi): allow optional label for xmpp servers (#1907)
* e7cb0f4 feat(jvb): autoscaler sidecar shutdown hook on finish (#1906)
* b0675c0 prosody: recording metadata module (#1734)
* 36b2e16 feat(web) - Added NGINX_KEEPALIVE_TIMEOUT environment variable
* 24bf3e4 fix(web) add HTTPS port to URL in env.example
* 2280f6b feat(web) add ability to run the web container targetting JaaS
* ff49cd9 Revert "feat(jigasi): install openjdk nonheadless, currently required for dep…" (#1899)
* 66c6ad8 feat(jigasi): install openjdk nonheadless, currently required for dependencies (#1895)
* 3f67408 feat(jigasi,transcriber): transcriber component and jigasi configuration updates (#1881)
* 3e93212 fix(jicofo): pass compose variable for jicofo transcription support (#1893)
* 084266c feat(transcriber): transcriber password in env.example and gen-passwords.sh (#1894)
* e1b14f2 feat(prosody): enable brewery muc modules in internal muc component (#1892)
* 1889c5d fix(web): enable hidden domain when transcriptions or recordings are enabled (#1891)
* 46377d4 feat(prosody): transcriber account support and flag configuration fix (#1890)
* 9f2489d feat(log-analyser): updated grafana dashboards (#1885)
* 1e589f3 feat(log-analyser): transfered docker logs from docker containers and updated readme (#1884)
* 1f056b4 web: remove unnecessary option
* f893e49 base: update tpl
* 7617503 fix(log-analyser): added Jitsi network to loki service
* 27150a2 feat(prometheus): Docker Daemon scraping for monitoring (#1865)
* d6b64a2 misc: working on unstable
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.

4 participants