Skip to content

Conversation

@dhtclk
Copy link
Collaborator

@dhtclk dhtclk commented Oct 28, 2025

Summary

Quick start guide for clickstack using redis

Checklist

@vercel
Copy link

vercel bot commented Oct 28, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
clickhouse-docs Ready Ready Preview Comment Nov 5, 2025 9:42pm
3 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
clickhouse-docs-jp Ignored Ignored Nov 5, 2025 9:42pm
clickhouse-docs-ru Ignored Ignored Preview Nov 5, 2025 9:42pm
clickhouse-docs-zh Ignored Ignored Preview Nov 5, 2025 9:42pm

@dhtclk dhtclk marked this pull request as ready for review October 30, 2025 14:24
@dhtclk dhtclk requested a review from a team as a code owner October 30, 2025 14:24
@dhtclk dhtclk requested a review from gingerwizard October 30, 2025 20:57
Copy link
Member

@Blargian Blargian left a comment

Choose a reason for hiding this comment

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

LGTM, left a few minor suggestions

@gingerwizard
Copy link
Collaborator

@dhtclk Let's also fix the left menu. The actual case for Nginx is not correct.

This configuration:
- Reads Redis logs from their standard location
- Parses Redis's log format using regex to extract structured fields (`pid`, `role`, `timestamp`, `log_level`, `message`)
Copy link
Collaborator

Choose a reason for hiding this comment

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

I would expect the log level to be set to in the open telemetry schema to the SeverityText field. Likewise, the ServiceName field should be set.

@dhtclk dhtclk merged commit cd3d101 into main Nov 5, 2025
15 checks passed
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.

4 participants