Skip to content

Update configuration.nix with promtail#840

Open
jeffreytjung wants to merge 3 commits intosocallinuxexpo:masterfrom
jeffreytjung:patch-9
Open

Update configuration.nix with promtail#840
jeffreytjung wants to merge 3 commits intosocallinuxexpo:masterfrom
jeffreytjung:patch-9

Conversation

@jeffreytjung
Copy link
Contributor

Promtail is a part of the monitoring stack because it is the processor for logs.

Promtail will send logs to a Loki service which is what injests the logs. Both services are needed, but promtail goes in front of Loki. It is the point where all of the rsyslog services that are running on the access points will send their logs. Basically, send logs to Promtail’s port, not directly to Loki. It is important to note that promtail only accepts newer RFC 5424 ("IETF") formatted syslog messages.

The Grafana dashboard will then be able to have Loki presets loaded into modules.

This push is just a build that puts the promtail servicee on the local NixOS host.

Description of PR

Previous Behavior

New Behavior

Tests

Promtail is a part of the monitoring stack because it is the processor for logs. 

Promtail will send logs to a Loki service which is what injests the logs. Both services are needed, but promtail goes in front of Loki.

The Grafana dashboard will then be able to have Loki presets loaded into modules.

This push is just a build that puts the promtail servicee on the local NixOS host.
Copy link
Collaborator

@owendelong owendelong left a comment

Choose a reason for hiding this comment

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

Seems reasonable, but someone else will need to resolve the merge conflicts.

Copy link
Member

@kylerisse kylerisse left a comment

Choose a reason for hiding this comment

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

this is stale and might not align with new monitoring setup, don't merge

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

Comments