-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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(jitsi-meet): added grafana loki and otel integration for log analysis #1844
Conversation
Please do not commit log files, please add .ds_store in .gitignore https://gist.github.com/lohenyumnam/2b127b9c3d1435dc12a33613c44e6308 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are a number of stray files here, like logs and .DS_Store files, please delete those.
log-analyser/grafana/grafana.db
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this should not be checked in?
log-analyser/.gitignore
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should I move this gitignore content to gitignore file which is in root directory or is it okay like this? @aaronkvanmeerten @ltorje-8x8
grafana.yml
Outdated
logging: | ||
driver: syslog | ||
options: | ||
syslog-address: "udp://127.0.0.1:5123" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a syslog listener on this port?
log-analyser.yml
Outdated
|
||
loki: | ||
container_name: loki | ||
user: "0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does this need to be user 0?
loki: | ||
endpoint: "http://loki:3100/loki/api/v1/push" | ||
otlp: | ||
endpoint: tempo:4317 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we have a tempo endpoint here?
period: 24h | ||
|
||
ruler: | ||
alertmanager_url: http://localhost:9093 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is probably not intended/needed?
log-analyser/README.md
Outdated
|
||
```bash | ||
git clone https://github.com/jitsi/docker-jitsi-meet.git | ||
cd grafana-loki-otel |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cd grafana-loki-otel |
Looks good to me, asking for a second pass from @saghul |
* 29a4523 feat(jvb) add JVB_CC_TRUST_BWE environment variable * 0f7be54 feat(prometheus): Prometheus container and basic scraping configuration (#1838) * 9c87bba feat(jitsi-meet): added grafana loki and otel integration for log analysis (#1844) * fcb90ba fix(web): whiteboard config.js syntax error (#1851) * 811518b misc: working on unstable
Added grafana loki and otel integration for log analysis.
This PR provides a simple and effective setup for log management and analysis using Docker, Grafana Loki and OpenTelemetry for Jitsi Meet.