Update aggregation_temporality to be an enum

[kaylareopelle]

The aggregation_temporality option in the Sum and ExplicitBucketHistogram aggregations is currently open-ended. The only spec'd options for this are :delta and :cumulative. Let's update the assignment to use an enum instead.

Relates to #1555

[github-actions]

👋 This issue 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 issue will be closed eventually by the stale bot.

[kaylareopelle]

@xuan-cao-swi - I looked into this a little more after the SIG. The class I was thinking of as an example is is OpenTelemetry::Trace::Status. In the spec, the Set Status API only has three valid options. The class is used like this:

opentelemetry-ruby/api/lib/opentelemetry/trace/tracer.rb

Line 40 in e654d73

span&.status = Status.error("Unhandled exception of type: #{e.class}")

The class method is always called to get a valid status.

@arielvalentin, is something like this what you were initially thinking of when you recommended an enum for aggregation temporality or did you have something else in mind?

[xuan-cao-swi]

Thanks for the example! I'd like to contribute on this based on the structure of OpenTelemetry::Trace::Status.

[kaylareopelle]

@xuan-cao-swi, sounds great! I've assigned you to the issue.

[kaylareopelle]

Resolved by #1769