fix(blooms): max block size #14615

salvacorts · 2024-10-25T09:23:47Z

What this PR does / why we need it:

Max block size is not correctly enforces since we did not look at the unflushed index and blooms pages size

Special notes for your reviewer:

Checklist

Reviewed the CONTRIBUTING.md guide (required)
Documentation added
Tests updated
Title matches the required conventional commits format, see here
- Note that Promtail is considered to be feature complete, and future development for logs collection will be in Grafana Alloy. As such, feat PRs are unlikely to be accepted unless a case can be made for the feature actually being a bug fix to existing behavior.
Changes that require user attention or interaction to upgrade are documented in docs/sources/setup/upgrade/_index.md
If the change is deprecating or removing a configuration option, update the deprecated-config.yaml and deleted-config.yaml files respectively in the tools/deprecated-config-checker directory. Example PR

salvacorts · 2024-10-25T10:48:44Z

closing in favour of #14577

pull-request-size bot added the size/L label Oct 25, 2024

salvacorts force-pushed the salvacorts/fix-x-block-size branch from 9fe0903 to e4832e6 Compare October 25, 2024 10:15

salvacorts changed the title ~~Fix max block size~~ fix(blooms): max block size Oct 25, 2024

salvacorts marked this pull request as ready for review October 25, 2024 10:33

salvacorts requested a review from a team as a code owner October 25, 2024 10:33

Fix max block size

ee946ee

salvacorts force-pushed the salvacorts/fix-x-block-size branch from e4832e6 to ee946ee Compare October 25, 2024 10:43

salvacorts closed this Oct 25, 2024

Provide feedback