MINOR: opentracing: change the scope of the variable 'ot.uuid' from 'sess' to 'txn'
authorMiroslav Zagorac <mzagorac@haproxy.com>
Thu, 9 Sep 2021 12:19:25 +0000 (14:19 +0200)
committerWilly Tarreau <w@1wt.eu>
Sun, 12 Sep 2021 05:08:14 +0000 (07:08 +0200)
At the suggestion of Willy Tarreau, the scope of the 'ot.uuid' variable was
changed from 'sess' to 'txn', so it is now limited to the transaction only.

addons/ot/README-func
addons/ot/include/filter.h
addons/ot/src/scope.c
addons/ot/test/be/ot.cfg
addons/ot/test/cmp/ot.cfg
addons/ot/test/ctx/ot.cfg
addons/ot/test/fe/ot.cfg
addons/ot/test/sa/ot.cfg

index 273c7f9..a6cb21e 100644 (file)
@@ -144,7 +144,7 @@ context is created, and flags are set that define which analyzers are used.
       flt_ot_runtime_context_init(<err>) {
          flt_ot_pool_alloc() {
          }
-         /* Initializing and setting the variable 'sess.ot.uuid'. */
+         /* Initializing and setting the variable 'txn.ot.uuid'. */
          if (flt_ot_var_register(<err>) != -1) {
             flt_ot_var_set(<err>) {
             }
index c97a0cc..6d41b72 100644 (file)
@@ -22,7 +22,7 @@
 
 #define FLT_OT_FMT_NAME           "'" FLT_OT_OPT_NAME "' : "
 #define FLT_OT_FMT_TYPE           "'filter' : "
-#define FLT_OT_VAR_UUID           "sess", "ot", "uuid"
+#define FLT_OT_VAR_UUID           "txn", "ot", "uuid"
 #define FLT_OT_ALERT(f, ...)      ha_alert(FLT_OT_FMT_TYPE FLT_OT_FMT_NAME f "\n", ##__VA_ARGS__)
 
 #define FLT_OT_CONDITION_IF       "if"
index 80b0bc2..6492c27 100644 (file)
@@ -117,7 +117,7 @@ struct flt_ot_runtime_context *flt_ot_runtime_context_init(struct stream *s, str
 
 #ifdef USE_OT_VARS
        /*
-        * The HAProxy variable 'sess.ot.uuid' is registered here,
+        * The HAProxy variable 'txn.ot.uuid' is registered here,
         * after which its value is set to runtime context UUID.
         */
        if (flt_ot_var_register(FLT_OT_VAR_UUID, err) != -1)
index edd3f76..12cf0a1 100644 (file)
@@ -20,7 +20,7 @@
     ot-scope frontend_http_request
         extract "ot-ctx" use-headers
         span "HAProxy session" child-of "ot-ctx" root
-            baggage "haproxy_id" var(sess.ot.uuid)
+            baggage "haproxy_id" var(txn.ot.uuid)
         span "Client session" child-of "HAProxy session"
         span "Frontend HTTP request" child-of "Client session"
             tag "http.method" method
index 21b15dd..a16fc9e 100644 (file)
@@ -21,7 +21,7 @@
 
     ot-scope client_session_start
         span "HAProxy session" root
-            baggage "haproxy_id" var(sess.ot.uuid)
+            baggage "haproxy_id" var(txn.ot.uuid)
         span "Client session" child-of "HAProxy session"
         event on-client-session-start
 
index a06a4e0..a753f77 100644 (file)
@@ -57,7 +57,7 @@
     ot-scope client_session_start_1
         span "HAProxy session" root
             inject "ot_ctx_1" use-headers use-vars
-            baggage "haproxy_id" var(sess.ot.uuid)
+            baggage "haproxy_id" var(txn.ot.uuid)
         event on-client-session-start
 
     ot-scope client_session_start_2
index 11de828..6f384e8 100644 (file)
@@ -22,7 +22,7 @@
 
     ot-scope client_session_start
         span "HAProxy session" root
-            baggage "haproxy_id" var(sess.ot.uuid)
+            baggage "haproxy_id" var(txn.ot.uuid)
         span "Client session" child-of "HAProxy session"
         event on-client-session-start
 
index ae7413b..654c2fe 100644 (file)
@@ -55,7 +55,7 @@
 
     ot-scope client_session_start
         span "HAProxy session" root
-            baggage "haproxy_id" var(sess.ot.uuid)
+            baggage "haproxy_id" var(txn.ot.uuid)
         span "Client session" child-of "HAProxy session"
         acl acl-test-src-ip src 127.0.0.1
         event on-client-session-start if acl-test-src-ip