Skip to content

Conversation

yeya24
Copy link
Contributor

@yeya24 yeya24 commented Jun 16, 2025

What this PR does:

This PR tries to add more Parquet metrics.

Cleaner:

  • Add cortex_bucket_parquet_unconverted_blocks_count metric for total number of TSDB blocks that should be converted to Parquet but not available yet. This can be useful to see the backlog.

Converter:

  • Add cortex_parquet_converter_convert_block_delay_minutes histogram metric. This is to track the delay from the time when TSDB block is uploaded to the time the block is converted to Parquet format.
  • Add cortex_parquet_converter_block_convert_failures_total metric for number of failed block conversions. This tracks all conversion failures except context cancellation error.

Which issue(s) this PR fixes:
Fixes #

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

@dosubot dosubot bot added component/compactor type/observability To help know what is going on inside Cortex labels Jun 16, 2025
@yeya24 yeya24 requested review from alanprot and danielblando June 17, 2025 02:00
@yeya24 yeya24 force-pushed the converter-metrics branch from 3bbdf9b to cfd5a08 Compare June 17, 2025 07:43
@SungJin1212
Copy link
Member

LGTM!

Copy link
Member

@alanprot alanprot left a comment

Choose a reason for hiding this comment

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

LGTM!!!

Thanks

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jun 17, 2025
@yeya24 yeya24 enabled auto-merge (squash) June 17, 2025 16:39
Signed-off-by: yeya24 <[email protected]>
@yeya24 yeya24 merged commit 8e3674d into cortexproject:master Jun 17, 2025
31 of 32 checks passed
@yeya24 yeya24 deleted the converter-metrics branch June 18, 2025 00:09
bogdan-st pushed a commit to bogdan-st/cortex that referenced this pull request Jun 30, 2025
…etrics (cortexproject#6821)

* add parquet converter failures, block delay and total unconverted block
metrics

Signed-off-by: yeya24 <[email protected]>

* changelog

Signed-off-by: yeya24 <[email protected]>

* address comment

Signed-off-by: yeya24 <[email protected]>

---------

Signed-off-by: yeya24 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/compactor lgtm This PR has been approved by a maintainer size/L type/observability To help know what is going on inside Cortex
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants