#3824 create example programattic trace for koji-cli
Closed 5 months ago by mikem. Opened 2 years ago by hmodi.
hmodi/koji otel  into  master

file modified
+12
@@ -42,6 +42,18 @@ 

  from koji_cli.commands import *  # noqa: F401, F403

  from koji_cli.lib import categories, get_epilog_str, greetings, warn

  

+ try:

+     from otel_extensions import TelemetryOptions, init_telemetry_provider

+ 

+     # Configure OpenTelemetry tracing wrapper.

+     otel_options = TelemetryOptions(

+             OTEL_SERVICE_NAME="koji-cli",

+             OTEL_EXPORTER_OTLP_ENDPOINT="http://localhost:4318",

+             OTEL_EXPORTER_OTLP_PROTOCOL="http/protobuf",

+         )

+     init_telemetry_provider(otel_options)

+ except ImportError:

+     print("Cannot import OpenTelemetry dependencies, skipping OpenTelemetry initialization")

  

  def register_plugin(plugin):

      """Scan a given plugin for handlers

file modified
+6
@@ -25,6 +25,12 @@ 

  from koji.util import md5_constructor, to_list

  from koji.xmlrpcplus import xmlrpc_client

  

+ try:

+     from opentelemetry.instrumentation.requests import RequestsInstrumentor

+ 

+     RequestsInstrumentor().instrument()

+ except ImportError:

+     print("Cannot import RequestsInstrumentor, skipping 'requests' instrumentation")

  

  # for compatibility with plugins based on older version of lib

  # Use optparse imports directly in new code.

@@ -0,0 +1,12 @@ 

+ opentelemetry-api==1.18.0

+ opentelemetry-exporter-otlp==1.18.0

+ opentelemetry-exporter-otlp-proto-common==1.18.0

+ opentelemetry-exporter-otlp-proto-grpc==1.18.0

+ opentelemetry-exporter-otlp-proto-http==1.18.0

+ opentelemetry-instrumentation==0.39b0

+ opentelemetry-instrumentation-requests==0.39b0

+ opentelemetry-proto==1.18.0

+ opentelemetry-sdk==1.18.0

+ opentelemetry-semantic-conventions==0.39b0

+ opentelemetry-util-http==0.39b0

+ otel-extensions==0.2.4

Signed-off-by: Harsh Modi hmodi@redhat.com

This is just an example PR.

rebased onto 3e5b49e

2 years ago

Pull-Request has been closed by mikem

5 months ago