Steve Flanders comments

Results 19 comments of


                                            Steve Flanders

Add OpenTelemetry distributed tracing

Hey @arey this PR brings distributed tracing to the microservices application. I can update the README. The collector is used to forward the received trace data to Zipkin.

Add OpenTelemetry distributed tracing

Took a look at the README and it does not say much about Zipkin -- added Otel Collector. Collector was added so users could switch to a different backend if...

cmd/agent: opencensus receiver w/ jaeger|zipkin exporters no traces

@DazWilkin nice find -- yes the agent/collector configuration is known to be inconsistent. This will be addressed in the OpenTelemetry Service.

security/all: fuzz input formats with any parsing

@tigrannajaryan @pjanotti do we plan to do this in OpenTelemetry Service?

exporter/stackdriver: error posting too frequent when >1 pod reports same metrics,

@rghetia what is the latest on this?

Flaky test: TestExportProtocolViolations_nodelessFirstMessage

@pjanotti is this still applicable?

Send to multiple exporters from tracer

@c24t this appears to work, thanks! note, there are other parts of the library that also need to be updated. For example, opencensus/trace/ext/flask/flask_middleware.py

build: TestExportProtocolViolations_nodelessFirstMessage is flaky

@tigrannajaryan is this still applicable?

oc receiver StopTraceReception and StopMetricsReception don't actually stop reception

@pjanotti does this need to move to OpenTelemetry Service?

Attributes and Links limit may be exporter concern, not the runtime

Agreed -- one of the design decisions for OC service was to set and forget client libraries, but with the current implementation that is not possible (already aware of someone...