Allow to configure subpaths within S3 buckets

[mapno]

Is your feature request related to a problem? Please describe.

Tempo's S3 config does not support using specific folder within an S3/GCS/Azure bucket to use as its root folder. This doesn't allow cleanly sharing buckets with other apps.

Describe the solution you'd like

Tempo should add support for subpaths within S3/GCS/Azure buckets.

Describe alternatives you've considered

Bucket names with / are not allowed per S3's bucket naming rules.

Additional context

Similar issues in Mimir grafana/mimir#1682 and Loki grafana/loki#5889

[shrey]

I'd like to take this issue up!

[WildCatFish]

Hi, @shrey is there any update on this issue?
We are having the same difficulty.

[kousikmitra]

Hi, I would like to pick this up since there no reply from assigned user. Let me know if I can work on this.

One question,

• Should this change only effect s3 and not gcs or azure?
• I did a little dig up, and I hope this is the package tempodb/backend/s3 that will need necessary changes. Please let me know if I am on right track!

[joe-elliott]

@kousikmitra We would love to have you give this a shot. Thanks for taking an interest!

Should this change only effect s3 and not gcs or azure?

It would be cool if we made it to all 3 backends at once. However, we appreciate any contribution and if s3 is all you can carve out some time to for, we understand.

I did a little dig up, and I hope this is the package tempodb/backend/s3 that will need necessary changes. Please let me know if I am on right track!

Yes, this change should be entirely doable in tempodb/backend/s3.

[kousikmitra]

@kousikmitra We would love to have you give this a shot. Thanks for taking an interest!

Should this change only effect s3 and not gcs or azure?

It would be cool if we made it to all 3 backends at once. However, we appreciate any contribution and if s3 is all you can carve out some time to for, we understand.

@mapno I would love to make the change for other backends. If thats ok I will create another ticket and work on it.

[mapno]

Please, go ahead. Thanks for contributing to Tempo!

[kousikmitra]

@mapno I have raised a PR with all changes. Can you please reopen the issue and update the description?

[mapno]

Done!