public inbox for isar-users@googlegroups.com
 help / color / mirror / Atom feed
From: Jan Kiszka <jan.kiszka@siemens.com>
To: Baurzhan Ismagulov <ibr@ilbers.de>
Cc: isar-users@googlegroups.com
Subject: Re: [PATCH 2/3] CI: make gitlab-ci build verbose instead of quiet
Date: Tue, 15 Oct 2019 13:31:24 +0200	[thread overview]
Message-ID: <1601e8f5-8219-f7f8-b687-dd21955de6fa@siemens.com> (raw)
In-Reply-To: <20191015104844.a6eodbfewfq23lqa@yssyq.m.ilbers.de>

On 15.10.19 12:48, Baurzhan Ismagulov wrote:
> On Sat, Sep 21, 2019 at 12:36:39PM +0200, Jan Kiszka wrote:
>>> -    - scripts/ci_build.sh -q -f
>>> +    - scripts/ci_build.sh -f
> ...
>> NACK, at least until non-verbose mode is fixed to not interleave the output
>> of parallel tasks.
> 
> I've enabled that on ci.isar-build.org, since it's the only way to see the
> concurrency. This may help debugging sporadic races. I agree that the output is
> very hard to read.

Sigh... If you really think you can read anything out of the resulting
logs in practice (I seriously doubt this), please confine this to your
private Jenkins runs. For gitlab-ci, we neither need this - because of
[1] - nor is it desired as I explained clearly. Believe me, I've
debugged a number of races seen in CI originally just recently.

Thanks,
Jan

> 
> IIRC, I had a discussion with DEBOS (?) people regarding the logs, but we
> didn't go into depth. One approach could be to always log everything (even
> messages that don't go to the console), e.g. into a database (with timestamps,
> task name and number, etc.), and have a tool to display and filter the output.
> 
> With kind regards,
> Baurzhan.
> 

[1]
https://github.com/ilbers/isar/commit/e5b7d62f91b233f85361d49871cf9a3dad5b2436

-- 
Siemens AG, Corporate Technology, CT RDA IOT SES-DE
Corporate Competence Center Embedded Linux

  reply	other threads:[~2019-10-15 11:31 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-09-19 17:04 [PATCH 0/3] various random style fixes Henning Schild
2019-09-19 17:04 ` [PATCH 1/3] ci_build: cosmetic change to increase readability Henning Schild
2019-10-14 14:11   ` Baurzhan Ismagulov
2019-09-19 17:04 ` [PATCH 2/3] CI: make gitlab-ci build verbose instead of quiet Henning Schild
2019-09-21 10:36   ` Jan Kiszka
2019-10-15 10:48     ` Baurzhan Ismagulov
2019-10-15 11:31       ` Jan Kiszka [this message]
2019-10-15 12:19         ` Baurzhan Ismagulov
2019-09-19 17:04 ` [PATCH 3/3] meta: rootfs: align the name of the apt preferences file for isar-apt Henning Schild
2019-10-15 10:41   ` Baurzhan Ismagulov
2019-09-19 17:38 ` [PATCH 0/3] various random style fixes Jan Kiszka

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1601e8f5-8219-f7f8-b687-dd21955de6fa@siemens.com \
    --to=jan.kiszka@siemens.com \
    --cc=ibr@ilbers.de \
    --cc=isar-users@googlegroups.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox