SNOW-1462014: Ability to disable OTel span creation even if opentelemetry is installed

[mikeldking]

What is the current behavior?

If opentelemetry is installed, spans are automatically emitted via https://github.com/snowflakedb/snowpark-python/blob/29b3bba9a7bd13206221b6fc6ad9ed0fae853fe0/src/snowflake/snowpark/_internal/open_telemetry.py

What is the desired behavior?

Add the ability to disable span creation even if opentelemetry is installed so as to keep the trace hierarchy simpler.

How would this improve snowflake-snowpark-python?

Make it more configurable and more composable with other OTel instrumentations

References, Other Background

OTel gives you the ability to turn off different instrumentations like so: https://github.com/snowflakedb/snowpark-python/blob/29b3bba9a7bd13206221b6fc6ad9ed0fae853fe0/src/snowflake/snowpark/_internal/open_telemetry.py

[sfc-gh-yuwang]

Hi, do you just want to be able to disable all snowpark emit spans, or you also want to have control on what function should not emit span? For example, do you want snowpark to emit span on .collect() while not emitting span on .show() ?

[mikeldking]

@sfc-gh-yuwang thanks so much for responding! For now I think it's an overall switch-off (@trevor-laviale-arize might be able to chime in here)

[trevorlaviale]

Yeah I think just the ability to disable in general is good for now :)

[sfc-gh-yuwang]

Got it, will do this recently, I am thinking about having a environment variable to turn on/off snowpark opentelemetry

[trevorlaviale]

Amazing, thanks so much!

[mikeldking]

Thanks @sfc-gh-yuwang - env var makes a ton of sense

[sfc-gh-yuwang]

I want to do a quick update on this issue, we are now discussing with other teams who potentially could have the issue about a uniform solution across snowflake, we should have this solved within this month

[sfc-gh-jfreeberg]

@mikeldking , @trevor-laviale-arize -- Have you tried setting this env var? OTEL_PYTHON_DISABLED_INSTRUMENTATIONS ?

https://opentelemetry.io/docs/zero-code/python/configuration/#disabling-specific-instrumentations

[sfc-gh-yuwang]

I also find that opentelemetry has recently added a feature of trace config: https://github.com/open-telemetry/opentelemetry-specification/pull/3877 , can you also try it?

[mikeldking]

Sorry very late to this request. This wasn't specifically for me but something observed by one of our customers. I believe the above would work and thus I will close my request. Thanks for the help