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

TSDB retention period estimation #270

Merged
merged 1 commit into from
Jan 18, 2025
Merged

Conversation

mahendrapaipuri
Copy link
Owner

  • When TSDB is configured with only retention period, we can consider it as "actual" retention period. When it is configured with either retention size or retention period and size, we need to verify that actual retention period by making an API request to up query.

  • Minor refactoring on how we estimate the retention period.

  • Refactor the frontend loadBalancer initialization and move all middleware related logic into middleware file.

  • Add unit tests to test all different cases

* When TSDB is configured with only retention period, we can consider it as "actual" retention period. When it is configured with either retention size or retention period and size, we need to verify that actual retention period by making an API request to `up` query.

* Minor refactoring on how we estimate the retention period.

* Refactor the frontend loadBalancer initialization and move all middleware related logic into middleware file.

* Add unit tests to test all different cases

Signed-off-by: Mahendra Paipuri <[email protected]>
@mahendrapaipuri mahendrapaipuri added bug Something isn't working maintenance General maintenance labels Jan 18, 2025
@mahendrapaipuri mahendrapaipuri merged commit ac0286a into main Jan 18, 2025
15 checks passed
@mahendrapaipuri mahendrapaipuri deleted the fix_tsdb_retention_estm branch January 18, 2025 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working maintenance General maintenance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant