Skip to content

Conversation

@codefromthecrypt
Copy link
Contributor

@codefromthecrypt codefromthecrypt commented Sep 3, 2025

Screenshot 2025-09-05 at 12 58 12 PM

@codefromthecrypt
Copy link
Contributor Author

weird, in python with the code in the last commit, all good, except it is strange that it seems the ENV override to hush the warnings isn't applying (also happens in docker so something's up..)

$ dotenv run --no-override -- opentelemetry-instrument python main.py --mcp                                                              
/Users/codefromthecrypt/oss/observability-examples/genai-function-calling/openai-agents/.venv/lib/python3.13/site-packages/opentelemetry/sdk/_events/__init__.py:53: LogDeprecatedInitWarning: LogRecord init with `trace_id`, `span_id`, and/or `trace_flags` is deprecated since 1.35.0. Use `context` instead.
  log_record = LogRecord(
/Users/codefromthecrypt/oss/observability-examples/genai-function-calling/openai-agents/.venv/lib/python3.13/site-packages/opentelemetry/sdk/_events/__init__.py:53: LogDeprecatedInitWarning: LogRecord init with `trace_id`, `span_id`, and/or `trace_flags` is deprecated since 1.35.0. Use `context` instead.
  log_record = LogRecord(
<think>
Okay, the user asked for the latest version of Elasticsearch 8. I called the get_latest_elasticsearch_version function with major_version set to 8. The response came back as 8.19.3. So, the answer should be that the latest version is 8.19.3. I need to present this clearly to the user.
</think>

The latest version of Elasticsearch 8 is **8.19.3**.
Screenshot 2025-09-04 at 12 09 37 PM

In docker, however, it is getting the same auth error which made us disable tracing in the first place

$ docker compose run --build --rm genai-function-calling --mcp
--snip--

/usr/local/lib/python3.13/site-packages/opentelemetry/sdk/_events/__init__.py:53: LogDeprecatedInitWarning: LogRecord init with `trace_id`, `span_id`, and/or `trace_flags` is deprecated since 1.35.0. Use `context` instead.
  log_record = LogRecord(
/usr/local/lib/python3.13/site-packages/opentelemetry/sdk/_events/__init__.py:53: LogDeprecatedInitWarning: LogRecord init with `trace_id`, `span_id`, and/or `trace_flags` is deprecated since 1.35.0. Use `context` instead.
  log_record = LogRecord(
<think>
Okay, the user asked for the latest version of Elasticsearch 8. I called the get_latest_elasticsearch_version function with major_version set to 8. The response came back as 8.19.3. So, the answer should be that the latest version is 8.19.3. I need to present this clearly to the user.
</think>

The latest version of Elasticsearch 8 is **8.19.3**.
[non-fatal] Tracing client error 401: {
  "error": {
    "message": "Incorrect API key provided: unused. You can find your API key at https://platform.openai.com/account/api-keys.",
    "type": "invalid_request_error",
    "param": null,
    "code": "invalid_api_key"
  }
}

@anuraaga
Copy link
Collaborator

anuraaga commented Sep 4, 2025

@codefromthecrypt Sorry I pushed 7483cb3 but realized probably you were waiting for other stuff to rerecord the cassette. Feel free to blast that commit

anuraaga and others added 2 commits September 4, 2025 16:48
Signed-off-by: Adrian Cole <[email protected]>
@codefromthecrypt codefromthecrypt marked this pull request as ready for review September 5, 2025 05:06
@codefromthecrypt
Copy link
Contributor Author

thanks @anuraaga I re-recorded. I think we can leave these workarounds as branch deps and later clean up.

@anuraaga anuraaga merged commit c484abb into elastic:main Sep 5, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants