public inbox for isar-users@googlegroups.com
 help / color / mirror / Atom feed
From: Zhihang Wei <wzh@ilbers.de>
To: Jan Kiszka <jan.kiszka@siemens.com>,
	isar-users <isar-users@googlegroups.com>
Cc: Felix Moessbauer <felix.moessbauer@siemens.com>,
	Cedric Hombourger <cedric.hombourger@siemens.com>
Subject: Re: [PATCH 7/9] CONTRIBUTING: Switch to new run-tests.sh
Date: Thu, 26 Mar 2026 18:01:18 +0100	[thread overview]
Message-ID: <4ae04a9d-ba56-4036-b431-d851064ba893@ilbers.de> (raw)
In-Reply-To: <0707415b2a9875c8101e609e215af9f2f20bdb53.1774254639.git.jan.kiszka@siemens.com>



On 3/23/26 09:30, 'Jan Kiszka' via isar-users wrote:
> From: Jan Kiszka <jan.kiszka@siemens.com>
>
> This massively simplifies the way to invoke the testsuite.
>
> It also obsoletes the old ci_setup.sh script, so drop it.
>
> Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
> ---
>   CONTRIBUTING.md     | 72 ++++++++++-----------------------------------
>   scripts/ci_setup.sh | 32 --------------------
>   2 files changed, 15 insertions(+), 89 deletions(-)
>   delete mode 100755 scripts/ci_setup.sh
>
> diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
> index 07621232..71edabde 100644
> --- a/CONTRIBUTING.md
> +++ b/CONTRIBUTING.md
> @@ -77,63 +77,21 @@ Plan merges to `master` so that both fit the two-week window; short extensions s
>   
>      * Seemingly unaffected boards still build.
>   
> -   * It's highly suggested to test your patchset before submitting it to the mailing
<snip>
> -    Active developers may request from maintainers an account on isar-build.org
> -    to analyze CI logs or to launch their own CI builds there.
> +   * It's highly suggested to test your patchset before submitting it to the
> +     mailing list by launching CI tests scripts:
> +
> +     ```
> +     scripts/run-tests -T dev
> +     ```
Here it should be "-t dev".

 From my point of view, it would be better to expose the avocado command
to users. They should be aware that avocado is used here and can more
easily understand the commands sent later to reproduce issues on CI.

Zhihang

> +     Perform the above steps over the branch that contain your patches.
> +
> +     Be also mindful of community-provided resources such as deb.debian.org or
> +     snapshot.debian.org and consider using a caching proxy in your setup to
> +     reduce traffic as much as possible.
> +
> +     Active developers may request from maintainers an account on isar-build.org
> +     to analyze CI logs or to launch their own CI builds there.
>   
>   4. Structure patches logically, in small increments.

-- 
You received this message because you are subscribed to the Google Groups "isar-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
To view this discussion visit https://groups.google.com/d/msgid/isar-users/4ae04a9d-ba56-4036-b431-d851064ba893%40ilbers.de.

  reply	other threads:[~2026-03-26 17:01 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-03-23  8:30 [PATCH 0/9] Improve testsuite executability, basic GitHub CI 'Jan Kiszka' via isar-users
2026-03-23  8:30 ` [PATCH 1/9] Update kas-container to 5.2 'Jan Kiszka' via isar-users
2026-03-23  8:30 ` [PATCH 2/9] testsuite: Provide a new kas-based test container and wrapper script 'Jan Kiszka' via isar-users
2026-03-23  8:30 ` [PATCH 3/9] testsuite: Install avocado-framework-plugin-varianter-yaml-to-mux in test-container 'Jan Kiszka' via isar-users
2026-03-23  8:30 ` [PATCH 4/9] testsuite: Add squid to test-container 'Jan Kiszka' via isar-users
2026-03-23  8:30 ` [PATCH 5/9] ci: Add github workflow for building and deploying test-container 'Jan Kiszka' via isar-users
2026-03-23  8:30 ` [PATCH 6/9] ci: Switch gitlab-ci to new test-container 'Jan Kiszka' via isar-users
2026-03-23  8:30 ` [PATCH 7/9] CONTRIBUTING: Switch to new run-tests.sh 'Jan Kiszka' via isar-users
2026-03-26 17:01   ` Zhihang Wei [this message]
2026-03-26 18:41     ` 'Jan Kiszka' via isar-users
2026-03-23  8:30 ` [PATCH 8/9] testsuite: Drop KFAIL from trixie tests 'Jan Kiszka' via isar-users
2026-03-23  8:30 ` [PATCH 9/9] ci: Add github test job to github workflow 'Jan Kiszka' via isar-users

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=4ae04a9d-ba56-4036-b431-d851064ba893@ilbers.de \
    --to=wzh@ilbers.de \
    --cc=cedric.hombourger@siemens.com \
    --cc=felix.moessbauer@siemens.com \
    --cc=isar-users@googlegroups.com \
    --cc=jan.kiszka@siemens.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