Skip to content

Conversation

@yeya24
Copy link
Contributor

@yeya24 yeya24 commented Nov 19, 2024

What this PR does:

Upgrade Thanos version to thanos-io/thanos#7915 as it is an important bug fix for store gateway.

Relevant changes to us:

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]

fetcher,
deduplicateBlocksFilter,
ignoreDeletionMarkFilter,
c.compactorCfg.CompactionInterval,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

They introduced a timeout for syncing metadatas. I just put compaction interval for now

@yeya24 yeya24 marked this pull request as draft November 19, 2024 00:42
@yeya24
Copy link
Contributor Author

yeya24 commented Nov 19, 2024

Convert to draft to figure out the change in thanos-io/thanos#7886 first.

Updated: Spoke with the Thanos PR author and we will just set to true

@yeya24
Copy link
Contributor Author

yeya24 commented Nov 19, 2024

Would be nice to reproduce this test failure.

https://github.com/cortexproject/cortex/actions/runs/11917739431/job/33213764870?pr=6346

@yeya24 yeya24 marked this pull request as ready for review November 19, 2024 17:35
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.

Thanks

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Nov 20, 2024
@yeya24 yeya24 merged commit 78a9e35 into cortexproject:master Nov 20, 2024
16 checks passed
@yeya24 yeya24 deleted the upgrade-thanos-fix-sg-bug branch November 20, 2024 22:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/store-gateway lgtm This PR has been approved by a maintainer size/S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants