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(operator): Enable time-based stream-sharding #16390

Merged
merged 1 commit into from
Feb 20, 2025

Conversation

xperimental
Copy link
Collaborator

@xperimental xperimental commented Feb 20, 2025

What this PR does / why we need it:

Loki 3.4.0 introduced a new feature that enables Loki to store log entries which are older than the time-window for a current log stream would allow: time-based stream sharding.

This PR enables this feature in the configuration generated by the operator.

Which issue(s) this PR fixes:

LOG-6757

Special notes for your reviewer:

  • Still need to test with custom per-tenant configuration. Global time-shard configuration is inherited by per-tenant configuration, so I don't think we need to add it to the runtime-config file.

Checklist

  • Reviewed the CONTRIBUTING.md guide (required)
  • Tests updated
  • Title matches the required conventional commits format, see here

@JoaoBraveCoding JoaoBraveCoding merged commit 1b4f1f5 into grafana:main Feb 20, 2025
67 checks passed
@xperimental xperimental deleted the enable-time-sharding branch February 20, 2025 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants