AWS S3 exporter s3_partition_format uses local timezone instead of UTC

[mymmrac]

Component(s)

exporter/awss3

What happened?

Description

AWS S3 exporter s3_partition_format uses local timezone instead of UTC and there is no way to change timezone used for partitioning which might be unexpected since there might be multiple collectors writing into same bucket with different timezones

Collector version

v0.133.0

Environment information

Environment

Any

OpenTelemetry Collector configuration

exporters:
  awss3:
    s3uploader:
      region: "us-east-1"
      s3_bucket: "tmp"
      s3_prefix: "logs"
      s3_partition_format: "%Y/%m/%d/%H/%M" # <- this time will be local instead of UTC

Log output

Additional context

No response

Tip

_{React with 👍 to help prioritize this issue. Please use comments to provide useful context, avoiding +1 or me too, to help us triage it. Learn more here.}

[github-actions]

Pinging code owners:

• exporter/awss3: @atoulme @pdelewski @Erog38

See Adding Labels via Comments if you do not have permissions to add labels yourself.

[axw]

What would be your ideal solution @mymmrac?
Simply changing it UTC would be breaking, though the exporter is alpha so maybe that's OK.
It might be worth also introducing another setting to control the timezone.

[mymmrac]

For me, adding one more option with timezone should be fine (for example s3_partition_timezone), maybe it's worth checking what other components do with timezone to make a similar change

[mymmrac]

Note: My current solution was to change the timezone of my app by setting the environment variable TZ=UTC

[mymmrac]

From what I see some has settings to set timezone, some does the same and not provide timezone

For example https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/dorisexporter has timezone setting, and https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/opensearchexporter doesn't have it

[atoulme]

thanks for the bug report. Would you like to work on a fix?

[mymmrac]

Sure, I can add something similar to what is implemented in dorisexporter