fix(ollama): Implemented meter in the instrumentation

[adharshctr]

• I have added tests that cover my changes.
• If adding a new instrumentation or changing an existing one, I've added screenshots from some observability platform showing the change.
• PR name follows conventional commits format: feat(instrumentation): ... or fix(instrumentation): ....
• (If applicable) I have updated the documentation accordingly.

---

Important

Added metrics collection to Ollama instrumentation using histograms for token usage and operation duration.

• Metrics:
  • Added token_histogram and duration_histogram to track token usage and operation duration in __init__.py.
  • Introduced _create_metrics() to initialize histograms for metrics.
  • Added is_metrics_enabled() to check if metrics collection is enabled.
• Functions:
  • Updated _wrap() and _awrap() to record metrics using histograms.
  • Modified _set_response_attributes() to record token usage in token_histogram.
  • Updated _accumulate_streaming_response() and _aaccumulate_streaming_response() to pass token_histogram.
• Instrumentation:
  • Updated OllamaInstrumentor._instrument() to initialize and pass histograms to wrapped methods.

^{This description was created by for 1276b31. It will automatically update as commits are pushed.}

[ellipsis-dev]

❌ Changes requested. Reviewed everything up to 1276b31 in 3 minutes and 7 seconds

More details

• Looked at 279 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 4 drafted comments based on config settings.

1. packages/opentelemetry-instrumentation-ollama/opentelemetry/instrumentation/ollama/__init__.py:446

• Draft comment:
  Tuple assignment mistake: When metrics are disabled, the tuple is set to three values (None, None, None), while only two (token_histogram, duration_histogram) are expected.
• Reason this comment was not posted:
  Marked as duplicate.

2. packages/opentelemetry-instrumentation-ollama/opentelemetry/instrumentation/ollama/__init__.py:237

• Draft comment:
  Potential bug: Using 'res' outside the loop in _accumulate_streaming_response may be undefined if the response is empty.
• Reason this comment was not posted:
  Comment was on unchanged code.

3. packages/opentelemetry-instrumentation-ollama/opentelemetry/instrumentation/ollama/__init__.py:237

• Draft comment:
  Verify the use of the dictionary union operator (res | accumulated_response) in streaming responses. Ensure that the merging behavior is as intended.
• Reason this comment was not posted:
  Confidence changes required: 50% <= threshold 50%
  None

4. packages/opentelemetry-instrumentation-ollama/opentelemetry/instrumentation/ollama/__init__.py:241

• Draft comment:
  The function name '_aaccumulate_streaming_response' contains an extra 'a' at the beginning. Consider renaming it (for example, to '_accumulate_streaming_response_async') to improve readability and follow naming conventions.
• Reason this comment was not posted:
  Comment was on unchanged code.

Workflow ID: wflow_ysXShdjstHS5VGQs

---

Want Ellipsis to fix these issues? Tag @ellipsis-dev in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

[nirga]

Thanks @adharshctr! Do you mind adding tests for this?

[adharshctr]

Thanks @adharshctr! Do you mind adding tests for this?

Added