Skip to content

(Request) [AWS S3 Exporter] Custom File Paths #37915

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

Closed
KevinThompsonYCRX opened this issue Feb 14, 2025 · 2 comments · Fixed by #37954
Closed

(Request) [AWS S3 Exporter] Custom File Paths #37915

KevinThompsonYCRX opened this issue Feb 14, 2025 · 2 comments · Fixed by #37954
Labels
enhancement New feature or request exporter/awss3

Comments

@KevinThompsonYCRX
Copy link

Component(s)

No response

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

The existing file path structure is excessively long & does not necessarily line up with existing partitions, it would be nice to make this slightly configurable.

The current filepath is defined here: https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/exporter/awss3exporter/internal/upload/partition.go#L52

Describe the solution you'd like

Use a formatting library like this: https://github.com/itchyny/timefmt-go

then you just expect this as part of the s3_prefix input & remove the s3_partition input

Describe alternatives you've considered

No response

Additional context

No response

@KevinThompsonYCRX KevinThompsonYCRX added enhancement New feature or request needs triage New item requiring triage labels Feb 14, 2025
@atoulme
Copy link
Contributor

atoulme commented Feb 14, 2025

I am open to contributions to make this better.

@atoulme atoulme added exporter/awss3 and removed needs triage New item requiring triage labels Feb 14, 2025
Copy link
Contributor

Pinging code owners for exporter/awss3: @atoulme @pdelewski. See Adding Labels via Comments if you do not have permissions to add labels yourself. For example, comment '/label priority:p2 -needs-triaged' to set the priority and remove the needs-triaged label.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request exporter/awss3
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants