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

Allow configuring S3 StorageClass #1684

Closed
amitsetty opened this issue Aug 23, 2022 · 3 comments · Fixed by #1714
Closed

Allow configuring S3 StorageClass #1684

amitsetty opened this issue Aug 23, 2022 · 3 comments · Fixed by #1714
Labels
good first issue Good for newcomers

Comments

@amitsetty
Copy link
Contributor

Currently I can’t configure grafana tempo to use the SATA S3 StorageClass

Would want to add this as a configurable feature

@joe-elliott
Copy link
Member

Can you give more information about how you would expect this feature to be enabled in Tempo? Is this something that can be configured on the bucket?

@ArpStorm1
Copy link

Hey @joe-elliott ,
I think what @amitsetty is meaning is to change the default storage class.
The default storage class is STANDARD, and it is the most expensive class in Amazon.
It would be great to have a configurable option to change that to STANDARD_IA or even GLACIER.

@joe-elliott
Copy link
Member

Ok, did some research. I figured this was something that was configurable on the bucket, but it seems like this is set at upload time?

I am open to a PR to set this. I believe it would need to be set in two places:

putObjectOptions := minio.PutObjectOptions{

options := minio.PutObjectOptions{

Also, i'm thinking we need a method to generate PutObjectOptions given the current config.

I would however warn against using storage classes such as Glacier as Tempo objects are accessed constantly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants