[tarantool-patches] Re: [PATCH 1/2] Test for valid syslog configuraton

  • From: Vladimir Davydov <vdavydov.dev@xxxxxxxxx>
  • To: Olga Arkhangelskaia <arkholga@xxxxxxxxxxxxx>
  • Date: Tue, 17 Jul 2018 17:10:32 +0300

[ Please add an extra new line after a quote when replying to an email -
  this improves readability ]

On Tue, Jul 17, 2018 at 05:07:32PM +0300, Olga Arkhangelskaia wrote:

Hmm, the test doesn't pass on certain distros (see travis). I guess,
this is because they don't have syslog configured. What about wrapping
box.cfg in pcall and checking the status? Since we just want to check
that there's no crash, this should do.  
I'll do that way.
Or alternatively you can test
this by setting a custom syslog destination (you have to test it anyway
in your other patches AFAIR).  
Do you mean in syslog destination series?

Yes.

I had a thought about it, but did not do in v2. Will keep in mind

Up to you.

Other related posts:

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