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

[release-v2.3] Fix compactor ignore configurable S3 headers #3181

Merged
merged 1 commit into from
Nov 27, 2023

Conversation

joe-elliott
Copy link
Member

Backport 57c3300 from #3154


What this PR does: Fix bug in compactor use of s3 headers

Which issue(s) this PR fixes:
Fixes #3149

Checklist

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

Hi :)
I am not a Go dev, but I need this fixed in order to use Tempo on my environment, It would be great if someone can help me complete the Checklist and merge this

* Fix compactor use of configurable S3 headers

* Update changelog

(cherry picked from commit 57c3300)
@joe-elliott joe-elliott merged commit c3bdd90 into release-v2.3 Nov 27, 2023
17 checks passed
@joe-elliott joe-elliott deleted the backport-3154-to-release-v2.3 branch November 27, 2023 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport type/bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants