fix: Add csv as a dependency for Ruby 3.4

[michaelherold]

csv has been a default gem for a long while. In Ruby 3.4, it will stop being a default gem and will need to be explicitly required as a dependency.

warning: /opt/rubies/3.3.1/lib/ruby/3.3.0/csv.rb was loaded from the standard library, but will no longer be part of the default gems since Ruby 3.4.0. Add csv to your Gemfile or gemspec. Also contact author of opentelemetry-exporter-otlp-0.26.1 to add csv into its gemspec.

This change adds it as a dependency for the OTLP exporter since it makes use of the library. It uses the same version constraint as the Zipkin exporter for consistency.

See eb89cd6 for a similar change.

[kaylareopelle]

Thanks for submitting this PR, @michaelherold! CSV is no longer a dependency of opentelemetry-exporter-otlp. If you update the gem to version 0.26.2 or above, the warning should go away.

[michaelherold]

Hi @kaylareopelle! I double checked when I submitted and this is where the require happens. I haven't vetted when or where it is used, though; perhaps it's not used anywhere but the require was left behind?

[kaylareopelle]

Ah, thank you! That link was very helpful. That file is for the opentelemetry-exporter-otlp-metrics gem, not the opentelemetry-exporter-otlp gem.

It looks like we're not using the CSV library in gem anymore, so I think the require was left behind.

Could you update this PR to remove the require 'csv' line from exporter/otlp-metrics/lib/opentelemetry/exporter/otlp/metrics_exporter.rb?

[github-actions]

👋 This pull request has been marked as stale because it has been open with no activity. You can: comment on the issue or remove the stale label to hold stale off for a while, add the keep label to hold stale off permanently, or do nothing. If you do nothing this pull request will be closed eventually by the stale bot

[kaylareopelle]

Hi, @michaelherold - The require 'csv' line will be removed by #1670

The warning will still appear on opentelemetry-exporter-otlp-0.26.1. The warning should no longer be reported on versions 0.26.2+ versions of that gem, as well as opentelemetry-exporter-otlp-metrics (when it is released).

[kaylareopelle]

Closing, as #1670 has been merged.