Sn0rt commented on code in PR #9983:
URL: https://github.com/apache/apisix/pull/9983#discussion_r1302455669
##########
t/plugin/opentelemetry.t:
##########
@@ -95,113 +77,34 @@ __DATA__
if code >= 300 then
ngx.status = code
end
- ngx.say(body)
+ ngx.say("done")
}
}
-
-
-
-=== TEST 2: trigger opentelemetry
--- request
-GET /opentracing
+GET /t
--- response_body
-opentracing
---- wait: 1
---- grep_error_log eval
-qr/opentelemetry export span/
---- grep_error_log_out
-opentelemetry export span
+done
-=== TEST 3: use default always_off sampler
---- config
- location /t {
- content_by_lua_block {
- local t = require("lib.test_admin").test
- local code, body = t('/apisix/admin/routes/1',
- ngx.HTTP_PUT,
- [[{
- "plugins": {
- "opentelemetry": {
- }
- },
- "upstream": {
- "nodes": {
- "127.0.0.1:1980": 1
- },
- "type": "roundrobin"
- },
- "uri": "/opentracing"
- }]]
- )
-
- if code >= 300 then
- ngx.status = code
- end
- ngx.say(body)
- }
- }
-
-
-
-=== TEST 4: not trigger opentelemetry
+=== TEST 2: trigger opentelemetry
--- request
GET /opentracing
+--- wait: 2
--- response_body
opentracing
---- grep_error_log eval
-qr/opentelemetry export span/
---- grep_error_log_out
-=== TEST 5: use trace_id_ratio sampler, default fraction = 0
---- config
- location /t {
- content_by_lua_block {
- local t = require("lib.test_admin").test
- local code, body = t('/apisix/admin/routes/1',
- ngx.HTTP_PUT,
- [[{
- "plugins": {
- "opentelemetry": {
- "sampler": {
- "name": "trace_id_ratio"
Review Comment:
keep this too
##########
t/plugin/opentelemetry.t:
##########
@@ -95,113 +77,34 @@ __DATA__
if code >= 300 then
ngx.status = code
end
- ngx.say(body)
+ ngx.say("done")
}
}
-
-
-
-=== TEST 2: trigger opentelemetry
--- request
-GET /opentracing
+GET /t
--- response_body
-opentracing
---- wait: 1
---- grep_error_log eval
-qr/opentelemetry export span/
---- grep_error_log_out
-opentelemetry export span
+done
-=== TEST 3: use default always_off sampler
---- config
- location /t {
- content_by_lua_block {
- local t = require("lib.test_admin").test
- local code, body = t('/apisix/admin/routes/1',
- ngx.HTTP_PUT,
- [[{
- "plugins": {
- "opentelemetry": {
Review Comment:
keep this configure is better
##########
t/plugin/opentelemetry.t:
##########
@@ -232,123 +135,34 @@ qr/opentelemetry export span/
if code >= 300 then
ngx.status = code
end
- ngx.say(body)
+ ngx.say("done")
}
}
-
-
-
-=== TEST 8: trigger opentelemetry
--- request
-GET /opentracing
+GET /t
--- response_body
-opentracing
---- wait: 1
---- grep_error_log eval
-qr/opentelemetry export span/
---- grep_error_log_out
-opentelemetry export span
-
-
-
-=== TEST 9: use parent_base sampler, default root sampler = always_off
---- config
- location /t {
- content_by_lua_block {
- local t = require("lib.test_admin").test
- local code, body = t('/apisix/admin/routes/1',
- ngx.HTTP_PUT,
- [[{
- "plugins": {
- "opentelemetry": {
- "sampler": {
- "name": "parent_base"
Review Comment:
keep this usage too.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
