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: Add Loki to collect container logs #266

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

Conversation

alvicsam
Copy link

@alvicsam alvicsam commented Aug 6, 2022

The PR adds Grafana Loki that helps in collecting logs and reading them from Grafana.

To make logs prettier I had to set the logging option for each container.
Example of logs in Grafana:

image

@keriati
Copy link
Contributor

keriati commented Oct 12, 2022

This would be a nice addition.

@rparadinhabv
Copy link

Nice feature to add.

- 3100
volumes:
- ./loki:/etc/loki
command: -config.file=/etc/loki/loki-config.yaml
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouln't the extension by yml

command: -config.file=/etc/promtail/promtail-config.yml
networks:
- monitor-net
logging: *default-logging
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probablly this needs the monitoring label so it shows up with the other monitoring services in grafana?

((im adding this to my setup, don't have any authority here but thanks 👍 ))

- monitor-net
logging: *default-logging

promtail:
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

missing restart: unless-stopped

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.

5 participants