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

Show example of using loki for logging #880

Merged
merged 4 commits into from
Feb 26, 2021

Conversation

larsbijl
Copy link
Contributor

Store logs for the RQD, Cuebot and DB pod's in loki and make
the datasource available in grafana.

dd1

Copy link
Collaborator

@DiegoTavares DiegoTavares left a comment

Choose a reason for hiding this comment

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

I'll get someone from my team to integrate this in our environment. But I agree this should live on the sandbox to avoid too much complication for new users.

@larsbijl
Copy link
Contributor Author

larsbijl commented Feb 4, 2021

As we split this into it's own docker-compose file I've decided to take it a step further.

  • Add Prometheus
  • Hookup the metrics exporter to Prometheus
  • Add Postgres Prometheus exporter
  • Hookup Postgres
  • Add basic dashboards for both

opencue_cuebot
opencue_postgresql

sandbox/docker-compose.yml Outdated Show resolved Hide resolved
sandbox/README.md Outdated Show resolved Hide resolved
sandbox/README.md Outdated Show resolved Hide resolved
@larsbijl
Copy link
Contributor Author

larsbijl commented Feb 7, 2021

Thanks @IdrisMiles, changes pushed.

Copy link
Contributor

@IdrisMiles IdrisMiles left a comment

Choose a reason for hiding this comment

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

LGTM, this works great!
Looks like the branch may need rebasing

Lars van der Bijl added 4 commits February 26, 2021 13:50
Store logs for the RQD, Cuebot and DB pod's in loki and make
the datasource available in grafana.
Hookup the prometheus to the cuebot and postgres metrics
with a default dashboard to give some insight.
@larsbijl larsbijl merged commit d84a10f into AcademySoftwareFoundation:master Feb 26, 2021
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