[tarantool-patches] Re: [PATCH v3 4/4] box-tap:syslog remote destination test

  • From: Vladimir Davydov <vdavydov.dev@xxxxxxxxx>
  • To: Olga Arkhangelskaia <arkholga@xxxxxxxxxxxxx>
  • Date: Wed, 25 Jul 2018 19:43:57 +0300

Please fold this patch in patch 1.

On Tue, Jul 24, 2018 at 08:11:52PM +0300, Olga Arkhangelskaia wrote:

@@ -507,5 +507,45 @@ os.exit(res)
 ]]
 test:is(run_script(code), 0, "unix socket syslog log configuration")
 
+--
+-- Check syslog remote configuration
+--
+code = [[
+local tap = require('tap')
+local socket = require('socket')
+local os = require('os')
+local test = tap.test("syslog_remote")
+local log = require('log')
+local errno = require('errno')
+
+addr = '127.0.0.1'
+port = 1000 + math.random(32768)
+
+sc = socket('AF_INET', 'SOCK_DGRAM', 'udp')
+local attempt = 0
+while attempt < 10 do
+if not sc:bind (addr, port) then
+    port = 1000 + math.random(32768)
+    attempt = attempt + 1
+    else
+     break

Tab instead of spaces. Please fix.

+    end
+end
+sc:bind(addr, port)
+
+local opt = string.format("syslog:server=%s:%u,identity=tarantool", addr, 
port)
+local res = 1
+if pcall(box.cfg,{log = opt, log_level = 5}) then

pcall is not needed.

+    log.info('Test syslog destination')
+    while (sc:readable(1)) do
+        buf = sc:recv(1000)
+        if buf:match('Test syslog destination') then res = 0 end
+    end

Hmm, here you use a different way of detecting the test string in the
log rather than in patch 3. Please be consistent if possible and use the
same method in both tests.

+end
+sc:close()
+os.exit(res)
+]]
+test:is(run_script(code), 0, "remote syslog log configuration")
+
 test:check()
 os.exit(0)

Other related posts:

  1. Home
  2. tarantool-patches
  3. Archive
  4. 07-2018