Jeremy Voss
Jeremy Voss
Looking into this.
Some options for adding a timeout inside resource detectors: * Add subtype like ResourceDetectorWithTimeout * Unified Resource Detector Timeout env var * Add method to base class getTimeout() that can...
A couple more issues: * While processes run in parallel, timeout applies sequentially. Detectors waited for last can take the longest * We wait for all to finish before allowing...
For log records, here are some initial links: https://github.com/open-telemetry/opentelemetry-python/blob/main/opentelemetry-sdk/src/opentelemetry/sdk/_logs/_internal/__init__.py#L492 https://github.com/open-telemetry/opentelemetry-python/blob/main/opentelemetry-sdk/src/opentelemetry/sdk/_logs/_internal/__init__.py#L530 Need to confirm logData does no defaulting.
Confirmed emit does not seem to default timestamp. This also means that trying to export that LogRecord to console fails because it cannot format timestamp. LogRecord.to_json() does not expect None....
Looking into include_trace_context, it does not seem python has this to begin with. LoggingHandler.emit calls LoggingHandler._translate which uses `get_current_span().get_span_context()`. This may be an optional feature for if we want people...
Here is the [observed_timestamp](https://github.com/open-telemetry/opentelemetry-python/blob/main/opentelemetry-sdk/src/opentelemetry/sdk/_logs/_internal/__init__.py#L161) field
Seems we are NOT defaulting observed_timestamp to current time. We could change this in the sdk... ``` class LogRecord(APILogRecord): def __init__( ... ): if not observed_timestamp: observed_timestamp = int(time.time() *...
Use time.time_ns()
Take a look at https://github.com/open-telemetry/opentelemetry-python/pull/3608