From: Sergey Kaplun via Tarantool-patches <tarantool-patches@dev.tarantool.org>
To: Alexander Turenko <alexander.turenko@tarantool.org>
Cc: tarantool-patches@dev.tarantool.org
Subject: Re: [Tarantool-patches] [PATCH 2/2] ci: run ARM64 jobs on GitHub-hosted runners
Date: Thu, 15 Oct 2026 10:13:01 +0300 [thread overview]
Message-ID: <atB8_Sfx9hOtl8CA@root> (raw)
In-Reply-To: <1791982697.240786681@f763.i.mail.ru>
On 14.10.26, Alexander Turenko wrote:
> > > after the patch we will use github runners and it is worth set a timeout
> > > for jobs,
> > >
> > > otherwise we might run out of time on the runners (AFAIK it is limited
> > > there).
> >
> > Agree here. IINM, the default is 6 hours [1], which is too much for our
> > tests.
> > The longest job is Valgrind job, which is running about 12 minutes.
> > I suppose the limit of 30 minutes is reasonable.
> >
> > <snipped>
> >
> > [1]: https://docs.github.com/en/actions/reference/workflows-and-actions/workflow-syntax#jobsjob_idtimeout-minutes
>
> I agree that it is good to have job timeouts.
>
> I’d let the maintainers to set appropriate timeouts.
> Personally, I neither use the jobs nor know details about them.
> And I don’t think that it becomes more urgent when we switch
> from 6 parallel runners to 60 (I’d say the opposite).
>
> WBR, Alexander Turenko
OK, lets do it in the separate patch.
--
Best regards,
Sergey Kaplun
prev parent reply other threads:[~2026-10-15 7:13 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-01 1:06 [Tarantool-patches] [PATCH 0/2] Use GitHub-hosted runners for Linux/ARM64 jobs Alexander Turenko via Tarantool-patches
2026-10-01 1:06 ` [Tarantool-patches] [PATCH 1/2] ci: use sudo to install packages if not root Alexander Turenko via Tarantool-patches
2026-10-02 13:18 ` Sergey Bronnikov via Tarantool-patches
2026-10-14 9:57 ` Sergey Kaplun via Tarantool-patches
2026-10-01 1:06 ` [Tarantool-patches] [PATCH 2/2] ci: run ARM64 jobs on GitHub-hosted runners Alexander Turenko via Tarantool-patches
2026-10-02 13:24 ` Sergey Bronnikov via Tarantool-patches
2026-10-14 10:46 ` Sergey Kaplun via Tarantool-patches
2026-10-14 12:58 ` Alexander Turenko via Tarantool-patches
2026-10-15 7:13 ` Sergey Kaplun via Tarantool-patches [this message]
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=atB8_Sfx9hOtl8CA@root \
--to=tarantool-patches@dev.tarantool.org \
--cc=alexander.turenko@tarantool.org \
--cc=skaplun@tarantool.org \
--subject='Re: [Tarantool-patches] [PATCH 2/2] ci: run ARM64 jobs on GitHub-hosted runners' \
/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
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox