Skip to content

Commit 6ec00e5

Browse files
committed
update sources
1 parent 2253bc3 commit 6ec00e5

File tree

5 files changed

+13
-11
lines changed

5 files changed

+13
-11
lines changed

ddtrace/auto.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,3 +19,7 @@ def main():
1919
directly.
2020
"""
2121
import ddtrace.bootstrap.sitecustomize # noqa:F401
22+
from ddtrace.internal.telemetry import telemetry_writer
23+
24+
25+
telemetry_writer.add_configuration("instrumentation_source", "ddtrace.auto", "code")

ddtrace/bootstrap/sitecustomize.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -165,8 +165,7 @@ def _(threading):
165165
else:
166166
log.debug("additional sitecustomize found in: %s", sys.path)
167167

168-
telemetry_writer.add_configuration("ddtrace_bootstrapped", True, "unknown")
169-
telemetry_writer.add_configuration("ddtrace_auto_used", "ddtrace.auto" in sys.modules, "unknown")
168+
telemetry_writer.add_configuration("instrumentation_source", "ddtrace.run", "code")
170169
# Loading status used in tests to detect if the `sitecustomize` has been
171170
# properly loaded without exceptions. This must be the last action in the module
172171
# when the execution ends with a success.

ddtrace/settings/_config.py

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@
1414

1515
from ddtrace.internal.serverless import in_azure_function
1616
from ddtrace.internal.serverless import in_gcp_function
17+
from ddtrace.internal.telemetry import telemetry_writer
1718
from ddtrace.internal.telemetry import validate_otel_envs
1819
from ddtrace.internal.utils.cache import cachedmethod
1920

@@ -775,10 +776,7 @@ def _set_config_items(self, items):
775776
item_names.append(key)
776777
item = self._config[key]
777778
item.set_value_source(value, origin)
778-
if self._telemetry_enabled:
779-
from ddtrace.internal.telemetry import telemetry_writer
780-
781-
telemetry_writer.add_configuration(item._name, item.value(), item.source())
779+
telemetry_writer.add_configuration(item._name, item.value(), item.source())
782780
self._notify_subscribers(item_names)
783781

784782
def _reset(self):
@@ -868,3 +866,6 @@ def _lower(self, value):
868866

869867

870868
config = Config()
869+
# This is used to track the source of the instrumentation.
870+
# By default it is set to "manual" but can be updated to ssi, ddtrace.auto, or ddtrace.run.
871+
telemetry_writer.add_configuration("instrumentation_source", "manual", "code")

lib-injection/sources/sitecustomize.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -431,7 +431,7 @@ def _inject():
431431
)
432432
# Track whether library injection was successful
433433
ddtrace.config._lib_was_injected = True
434-
ddtrace.internal.telemetry.telemetry_writer.add_configuration("lib_injection_successful", True, "code")
434+
ddtrace.internal.telemetry.telemetry_writer.add_configuration("instrumentation_source", "ssi", "code")
435435
except Exception as e:
436436
TELEMETRY_DATA.append(
437437
create_count_metric(

tests/telemetry/test_writer.py

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -159,8 +159,7 @@ def test_app_started_event(telemetry_writer, test_agent_session, mock_time):
159159
{"name": "DD_TRACE_WRITER_INTERVAL_SECONDS", "origin": "unknown", "value": 1.0},
160160
{"name": "DD_TRACE_WRITER_MAX_PAYLOAD_SIZE_BYTES", "origin": "unknown", "value": 20 << 20},
161161
{"name": "DD_TRACE_WRITER_REUSE_CONNECTIONS", "origin": "unknown", "value": False},
162-
{"name": "ddtrace_auto_used", "origin": "unknown", "value": False},
163-
{"name": "ddtrace_bootstrapped", "origin": "unknown", "value": False},
162+
{"name": "instrumentation_source", "origin": "code", "value": "manual"},
164163
{"name": "profiling_enabled", "origin": "default", "value": "false"},
165164
{"name": "data_streams_enabled", "origin": "default", "value": "false"},
166165
{"name": "appsec_enabled", "origin": "default", "value": "false"},
@@ -527,8 +526,7 @@ def test_app_started_event_configuration_override(test_agent_session, run_python
527526
{"name": "_DD_APPSEC_DEDUPLICATION_ENABLED", "origin": "default", "value": True},
528527
{"name": "_DD_IAST_LAZY_TAINT", "origin": "default", "value": False},
529528
{"name": "_DD_TRACE_WRITER_LOG_ERROR_PAYLOADS", "origin": "default", "value": False},
530-
{"name": "ddtrace_auto_used", "origin": "unknown", "value": True},
531-
{"name": "ddtrace_bootstrapped", "origin": "unknown", "value": True},
529+
{"name": "instrumentation_source", "origin": "code", "value": "ddtrace.auto"},
532530
{"name": "python_build_gnu_type", "origin": "unknown", "value": sysconfig.get_config_var("BUILD_GNU_TYPE")},
533531
{"name": "python_host_gnu_type", "origin": "unknown", "value": sysconfig.get_config_var("HOST_GNU_TYPE")},
534532
{"name": "python_soabi", "origin": "unknown", "value": sysconfig.get_config_var("SOABI")},

0 commit comments

Comments
 (0)