gy09535 commented on a change in pull request #3129:
URL: https://github.com/apache/apisix/pull/3129#discussion_r550386822



##########
File path: t/core/utils.t
##########
@@ -239,3 +236,32 @@ res:tell David to
 res:John and David
 res: and David
 res:John and \$me
+
+
+
+=== TEST 7:  resolve with search
+--- config
+    location /t {
+        content_by_lua_block {
+            local core = require("apisix.core")
+            local resolvers = {"223.5.5.5", "223.6.6.6"}
+            local search = {"google.com"}
+            local ndots = 5
+            core.utils.init_dns_proxy({
+                ndots = ndots,
+                search = search,
+                resolvers = resolvers,
+            })
+            local ip_info, err = core.utils.dns_parse("news.google.com")

Review comment:
       > Then how to proof the code can be used correctly in k8s pod? Say we 
need to resolve the etcd service with its short name like "myetcd", while 
actually the record in dns is "myetcd.ns.svc.cluster.local" as far as I know.
   
   It is my fault, It can work now.




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to