Merge pull request #96 from yangxikun/bugfix/rokety/duplicate_trace_ids

fix: duplicate trace ids, closes #95

docker-compose.yml

@@ -14,13 +14,13 @@ services:
     ports:
       - 80:80
   jaeger:
-    image: jaegertracing/all-in-one
+    image: jaegertracing/all-in-one:1
     ports:
       - 26686:16686
     networks:
       - opentelemetry-lua
   otel-collector:
-    image: otel/opentelemetry-collector-contrib-dev:latest
+    image: otel/opentelemetry-collector-contrib:0.98.0
     command: [ "--config=/etc/otel-collector-config.yaml" ]
     volumes:
       - ./otel-collector-config.yaml:/etc/otel-collector-config.yaml

lib/opentelemetry/util.lua

@@ -1,7 +1,29 @@
 local _M = {}
 
 --- Set randomseed
-math.randomseed(ngx.time() + ngx.worker.pid())
+local random_seed = function ()
+  local seed
+  local frandom = io.open("/dev/urandom", "rb")
+  if frandom then
+    local str = frandom:read(4)
+    frandom:close()
+    if str then
+      local s = 0
+      for i = 1, 4 do
+        s = 256 * s + str:byte(i)
+      end
+      seed = s
+    end
+  end
+
+  if not seed then
+    seed = ngx.now() * 1000 + ngx.worker.pid()
+  end
+
+  return seed
+end
+
+math.randomseed(random_seed())
 
 -- performance better, but may cause clock skew
 local function ngx_time_nano()

otel-collector-config.yaml

@@ -9,15 +9,15 @@ exporters:
     endpoint: "0.0.0.0:8889"
     const_labels:
       label1: value1
-  logging:
-    loglevel: debug
+  debug:
+    verbosity: detailed
 
   #zipkin:
   #  endpoint: "http://zipkin-all-in-one:9411/api/v2/spans"
   #  format: proto
 
-  jaeger:
-    endpoint: jaeger:14250
+  otlp/jaeger:
+    endpoint: jaeger:4317
     tls:
       insecure: true
 
@@ -37,8 +37,8 @@ service:
     traces:
       receivers: [otlp]
       processors: [batch]
-      exporters: [logging, jaeger]
+      exporters: [debug, otlp/jaeger]
     metrics:
       receivers: [otlp]
       processors: [batch]
-      exporters: [logging, prometheus]
+      exporters: [debug, prometheus]