public inbox for isar-users@googlegroups.com
 help / color / mirror / Atom feed
From: "'Jan Kiszka' via isar-users" <isar-users@googlegroups.com>
To: Zhihang Wei <wzh@ilbers.de>, 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 19:41:34 +0100	[thread overview]
Message-ID: <8817cccb-c763-4190-a283-310483cef09e@siemens.com> (raw)
In-Reply-To: <4ae04a9d-ba56-4036-b431-d851064ba893@ilbers.de>

On 26.03.26 18:01, Zhihang Wei wrote:
> 
> 
> 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".
> 

Upps, yes. I originally exposed ci_build.sh API, but the realized that
avocado is the better interface. And the only way to run individual tests.

> 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.

That's what we do, see help. It's just that some required switches for
avocado are preconfigured, specifically the unfortunately serialization.

Jan

-- 
Siemens AG, Foundational Technologies
Linux Expert Center

-- 
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/8817cccb-c763-4190-a283-310483cef09e%40siemens.com.

  reply	other threads:[~2026-03-26 18:41 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
2026-03-26 18:41     ` 'Jan Kiszka' via isar-users [this message]
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=8817cccb-c763-4190-a283-310483cef09e@siemens.com \
    --to=isar-users@googlegroups.com \
    --cc=cedric.hombourger@siemens.com \
    --cc=felix.moessbauer@siemens.com \
    --cc=jan.kiszka@siemens.com \
    --cc=wzh@ilbers.de \
    /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