From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtpng3.m.smailru.net (smtpng3.m.smailru.net [94.100.177.149]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dev.tarantool.org (Postfix) with ESMTPS id 215574765E0 for ; Mon, 21 Dec 2020 20:01:20 +0300 (MSK) References: <20201210161832.729439-1-gorcunov@gmail.com> <20201210161832.729439-5-gorcunov@gmail.com> <3d201857-f809-c758-297c-e3e896fbf06c@tarantool.org> <20201216111606.GA14556@grain> <0bb6f2db-5ef8-9d71-be61-67e113c7e0ad@tarantool.org> <20201220182137.GC3139@grain> <20201220191654.GG3139@grain> From: Vladislav Shpilevoy Message-ID: <8e3f5b78-57c6-eefe-4a62-b3ee6ae561a3@tarantool.org> Date: Mon, 21 Dec 2020 18:01:18 +0100 MIME-Version: 1.0 In-Reply-To: <20201220191654.GG3139@grain> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Subject: Re: [Tarantool-patches] [PATCH v4 4/4] crash: report crash data to the feedback server List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Cyrill Gorcunov Cc: TML On 20.12.2020 20:16, Cyrill Gorcunov wrote: > On Sun, Dec 20, 2020 at 07:41:47PM +0100, Vladislav Shpilevoy wrote: >> >> The error above I checked on your branch. And it remains the same >> even with the latest commit which you didn't push yet. Because the >> feedback options still are not configured before box.cfg. It has >> nothing to do with the recursive crash. >> >> It means, as I said above, that you can't send a crash before >> box.cfg is done. Therefore you can simply not even try if the >> host is not configured yet. It solves the error message, and alongside >> solves the crash chain issue. You don't need environment variables >> for this. >> >> I would suggest not to rush responding to these emails, but give it >> some thought beforehand. >> >> It is actually strange, that feedback daemon does not work before >> box.cfg. We don't see any reports from tarantool instances which are >> used as an application server. But it has nothing to do with your >> patch anyway. > > The latest updates are in gorcunov/gh-5261-crash-report-4-rev2 Indeed, on this branch the crash is not attempted to be sent.