allow passing environment for sentry exporter

[fadeltd]

Component(s)

exporter/sentry

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

Hi, I'd like to be able to pass an environment for the sentry exporter as it is currently not supported, for easier sentry filtering

Describe the solution you'd like

exporters:
  sentry:
    dsn: https://key@host/path/42
    env: prod
    insecure_skip_verify: true

Describe alternatives you've considered

No response

Additional context

I can create a PR for this

[github-actions]

Pinging code owners:

• exporter/sentry: @AbhiPrasad

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

[AbhiPrasad]

This makes sense, but I think we probably want to selectively grab the environment based on the incoming data, and not do it with a blanket collector config.

FYI we also have: getsentry/sentry#40712

[fadeltd]

This makes sense, but I think we probably want to selectively grab the environment based on the incoming data, and not do it with a blanket collector config.

FYI we also have: getsentry/sentry#40712

Well, for my projects, each project has their own separate config for each environment, so I only need to set the dsn value and the env, and this would work. I'm not sure if this is the best approach for other projects

[AbhiPrasad]

I guess it's fair to just ship this for now to unblock you, but we should add documentation that this applies to every single event.

[fadeltd]

Hi, I'm reopening this since I just found out that because sentryexporter is using transport
https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/exporter/sentryexporter/transport.go#L36-L41, the environment value I set on sentry.ClientOptions isn't getting sent to the sentry

I need to actually attach the environment value on the transaction https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/exporter/sentryexporter/sentry_exporter.go#L437

		transaction.Environment = environment

any idea of how I can implement this with the same approach (using config)?

[github-actions]

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

Pinging code owners:

• exporter/sentry: @AbhiPrasad

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