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

Allow sending app logs to logit using filebeat #183

Merged
merged 1 commit into from
Mar 25, 2024

Conversation

RMcVelia
Copy link
Collaborator

Context

Enable filebeat to send logs to logit.io
It will send logs to logit if the container has annotation logit.io/send: true

Changes proposed in this pull request

Terraform updates

Guidance to review

Deployed to cluster5, with app container sending logs to logit.io
make development terraform-plan ENVIRONMENT=cluster5

App deployed using terraform-modules branch 906-spike-application-log-shipping

Checklist

  • I have performed a self-review of my code, including formatting and typos
  • I have cleaned the commit history
  • I have added the Devops label
  • I have attached the pull request to the trello card

@RMcVelia
Copy link
Collaborator Author

Copy link
Member

Choose a reason for hiding this comment

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

Can this file be moved outside of the prometheus directory? Maybe config/filebeat/

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Moved to config/filebeat

@RMcVelia RMcVelia force-pushed the 906-spike-application-log-shipping branch 3 times, most recently from 50a8d24 to 165ad5f Compare March 22, 2024 13:31
@RMcVelia RMcVelia force-pushed the 906-spike-application-log-shipping branch from 165ad5f to 8ee303f Compare March 22, 2024 14:02
@RMcVelia RMcVelia merged commit fdce2ef into main Mar 25, 2024
7 checks passed
@RMcVelia RMcVelia deleted the 906-spike-application-log-shipping branch March 25, 2024 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants