From: John Kacur <jkacur@redhat.com>
To: Tomas Glozar <tglozar@redhat.com>
Cc: Clark Williams <williams@redhat.com>,
Linux RT Users <linux-rt-users@vger.kernel.org>
Subject: Re: [PATCH] rteval: Introduce E2E tests with output checking
Date: Thu, 18 Sep 2025 13:56:19 -0400 (EDT) [thread overview]
Message-ID: <6f2a6564-ecf0-e77e-2ccf-5250eddab97f@redhat.com> (raw)
In-Reply-To: <20250725090201.32656-1-tglozar@redhat.com>
On Fri, 25 Jul 2025, Tomas Glozar wrote:
> Currently, rteval has two kinds of tests:
> - Unit tests, embedded directly in code, and run by
> unit-tests/unittest.py.
> - End-to-end tests, implemented in Makefile targets runit, load, and
> sysreport.
>
> Introduce a new test suite in folder e2e-tests/ (analogically to
> unit-tests) that uses Test::Harness ("prove" command) together with a
> simple test engine adopted from RTLA.
>
> The test suite runs rteval in a temporary folder for each test case,
> collects its exit value and output, and validates both according to the
> test specification. grep is used to check the output, optionally with
> custom flags. rteval.conf is generated individually based on the test
> specification of each case.
>
> Three test sets are implemented in the commit:
> - "basic", running base checks without any specific modules or module
> options. Replaces "make runit".
> - "loads", testing various kinds of loads, including their options.
> - "measurement", testing both cyclictest and timerlat.
>
> The latter two check in rteval output whether the command run by rteval
> corresponds to the rteval options and to the specified load or
> measurement, if possible.
>
> "make runit" and "make load" targets are depracated with a warning;
> "make sysreport" is kept until sysreport tests are added to the new test
> suite.
>
Interesting.
Note the correct spelling is "deprecated"
I'm not sure we want to deprecate runit and loads, they have different
purposes. I often run in a non-rt environment and without tuned if I'm
just testing functionality and not performance, and I got this after
running make check (skipping some of the output before that)
# Output match failed: "Command: rtla timerlat hist -p1100 -P f:95 -u
-c[0-9|-]+ -E3500 --no-summary -T1 -t=trace.txt"
e2e-tests/measurement.t .. Failed 6/19 subtests
Test Summary Report
-------------------
e2e-tests/loads.t (Wstat: 0 Tests: 8 Failed: 1)
Failed test: 5
e2e-tests/measurement.t (Wstat: 0 Tests: 19 Failed: 6)
Failed tests: 13-14, 16-19
Files=3, Tests=33, 308 wallclock secs ( 0.06 usr 0.00 sys + 146.20 cusr
281.79 csys = 428.05 CPU)
Result: FAIL
make: *** [Makefile:22: check] Error 1
How do I identify which test is test number 5?
Am I failing tests because of performance reasons or because the tests
expect an environment different from mine?
Thanks
John Kacur
next prev parent reply other threads:[~2025-09-18 17:56 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-07-25 9:02 [PATCH] rteval: Introduce E2E tests with output checking Tomas Glozar
2025-09-18 17:56 ` John Kacur [this message]
2025-09-18 19:24 ` Crystal Wood
2025-09-26 11:12 ` Tomas Glozar
2025-11-04 12:57 ` Tomas Glozar
2025-09-18 19:22 ` Crystal Wood
2025-09-26 11:09 ` Tomas Glozar
2025-09-30 22:57 ` Crystal Wood
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=6f2a6564-ecf0-e77e-2ccf-5250eddab97f@redhat.com \
--to=jkacur@redhat.com \
--cc=linux-rt-users@vger.kernel.org \
--cc=tglozar@redhat.com \
--cc=williams@redhat.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