instana / python-sensor

:snake: Python Distributed Tracing & Metrics Sensor for Instana
https://www.instana.com/
MIT License
69 stars 31 forks source link

Is Python 3.10 supported? #455

Closed alexlang74 closed 1 year ago

alexlang74 commented 1 year ago

Problem Description

The github documentation says "Python > 2.7 is supported", but https://www.ibm.com/docs/en/instana-observability/current?topic=package-python-supported-components-versions#python-runtimes limits this to Python < 3.10. Which one is correct? If Python 3.10 is not supported yet - do you have a timeline for that? Thanks!

Minimal, Complete, Verifiable, Example

No response

Python Version

3.10

Python Modules

na

Python Environment

na
Ferenc- commented 1 year ago

Hi @alexlang74,

Where have you read on github, that "Python > 2.7 is supported" ? I had the codebase of this repo searched with multiple tools, and none of them could find anything like that on the current mater branch head. If there is such a statement, then let us know, because that is definitely incorrect, currently the minimum needed runtime verion, for the supported track is 3.7.

The documentation here is correct. Commercial support is only for < 3.10 at the moment. You can write an AHA ticket about it, or contact PM, if you want this to be prioritized.

BTW, if you are not using asyncio or tornado, then your experience on 3.10 should be comparable to the 3.9.

alexlang74 commented 1 year ago

Argh, I mistyped! of course 3.7, sorry! We're using FastAPI on PY 3.10, and want to use instana goodness...

Ferenc- commented 1 year ago

Argh, I mistyped! of course 3.7, sorry! We're using FastAPI on PY 3.10, and want to use instana goodness...

And would you run it on k8s, with the webhook? Which server would you use with FastAPI? uvicorn?

Ferenc- commented 1 year ago

A quick smoke test with the currently listed fastapi example unicorn on K8S with the image: python:3.10, didn't show any blockers. fastapi is predominantly covered by the ASGI middleware, and the fastapi specific code is minimal in the tracer.

alexlang74 commented 1 year ago

Thank you!!