[Tarantool-patches] [PATCH v3 0/4] crash dump: implement sending feedback

Cyrill Gorcunov gorcunov at gmail.com
Fri Dec 4 18:29:59 MSK 2020


Our feedback daemon sends only a few portions of usage
statistics. But crash dumps are pretty important for us
as well, because real users may catch a way more important
issues than our testing farm, it is simply impossible to
cover all possible scenarios.

For this sake, if crash happens we can send it to our
feedback server.

In this series we implement only base functionality and
may extend it later.

I didn't find yet a simple way to test this code anything
but manually.

Any comments are highly appreciated.

v2:
 - left for internal use
v3 (by Mons):
 - enable sending crash report by default but give
   an ability to disable it
 - use vfork when running subsequent exeutable of
   tarantool to send report, since plain execve won't
   produce local crashdump

issue https://github.com/tarantool/tarantool/issues/5261
branch gorcunov/gh-5261-crash-report-3

Cyrill Gorcunov (4):
  backtrace: allow to specify destination buffer
  errstat: add crash report base code
  crash: use errstat code in fatal signals
  cfg: configure crash report sending

 src/box/lua/feedback_daemon.lua |   7 +
 src/box/lua/load_cfg.lua        |   3 +
 src/lib/core/CMakeLists.txt     |   1 +
 src/lib/core/backtrace.cc       |  12 +-
 src/lib/core/backtrace.h        |   3 +
 src/lib/core/errstat.c          | 409 ++++++++++++++++++++++++++++++++
 src/lib/core/errstat.h          | 253 ++++++++++++++++++++
 src/main.cc                     |  83 ++++---
 test/box/admin.result           |   2 +
 test/box/cfg.result             |   4 +
 10 files changed, 735 insertions(+), 42 deletions(-)
 create mode 100644 src/lib/core/errstat.c
 create mode 100644 src/lib/core/errstat.h


base-commit: a14dad9543360c49c2cf8906a88e09271cad91b6
-- 
2.26.2



More information about the Tarantool-patches mailing list