Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 26 Feb 2024 16:25:45 +0000
From:      Igor Ostapenko <igor.ostapenko@pm.me>
To:        Paul Floyd <paulf2718@gmail.com>
Cc:        freebsd-hackers@freebsd.org
Subject:   Re: Add jail execution environment support to the FreeBSD test suite
Message-ID:  <p1gWDrJkoYRyleu_fr9Sj4M4G68SXbats7uxlQLuQNm0uIn3hrGP5Sf4x9VsRyvEHouVbVHB_6H0KOAz9JfYCwiNvIHFY5F9u13tYmcZoyA=@pm.me>
In-Reply-To: <bda8840f-5450-48c8-9721-cc782c2c98ac@gmail.com>
References:  <2bjQNp1msrv-_AqyamMun6kY-SCqbgPm3Q7DqVQHAYlqvFkiE1i85svfIT-QQdUG1cg3cKippyTyv8Z-5nbLu4WaMutgZQ7KT-YYo_5Pbro=@pm.me> <188b3537-11b1-4169-aae2-070d349b6016@gmail.com> <5XGsuxCWojiK7f1wBuoC73PWMtjVB3oCvsy1vHQtNr5UHC_2LN8zNGczKwpQEs4Pcc8g1G3Rir4uIrQ9iUxKbWmonsEu5f_mHzldy4vuYk4=@pm.me> <bda8840f-5450-48c8-9721-cc782c2c98ac@gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Friday, February 23rd, 2024 at 7:57 PM, Paul Floyd <paulf2718@gmail.com>=
 wrote:
> On 23-02-24 16:08, Igor Ostapenko wrote:
>=20
> > On Friday, February 23rd, 2024 at 10:30 AM, Paul Floyd paulf2718@gmail.=
com wrote:
> >=20
> > > Will there be an option to turn this off globally?
> > >=20
> > > I sometimes run test suits with Valgrind. I'm not aware of any option=
 to
> > > do that cleanly so I just do it with brute force: run kyua and all
> > > children under Valgrind.
> > >=20
> > > valgrind --trace-children=3Dyes -q kyua test -k Kyuafile
> > >=20
> > > Adding jails into the mix is likely to give worse results. I did try
> > > using an x86 jail as an alternative to VirtualBox but I gave up as I =
got
> > > too many test failures.
> >=20
> > Thanks for your attention and mention of your use case.
> >=20
> > I've added this point to my TODO. I guess it could look as the followin=
g:
> >=20
> > valgrind --trace-children=3Dyes -q kyua -v execenv=3Dhost test -k Kyuaf=
ile
>=20
>=20
> Sounds good, but I'd need to test in order to make sure. I don't run
> that kind of test very often.
>=20
> > Does it look like something that would cover your production needs?
>=20
>=20
> It's not to test FreeBSD for prod, it's to test Valgrind is working with
> all those tests (any FreeBSD errors picked up are a bonus).
>=20

Paul, it would be appreciated if you have time to test the patch. The gener=
al
test plan is covered here: https://reviews.freebsd.org/D42350, but specific=
ally
for your case the following extra check could be run in addition:

valgrind --trace-children=3Dyes -q kyua -v execenv=3Dhost test -k /usr/test=
s/sys/netpfil/pf/Kyuafile

It should report that pass_block:* tests are skipped due to the given
"-v execenv=3Dhost" runtime configuration.

I hope current implementation meets your expectations.


Best regards, Igor.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?p1gWDrJkoYRyleu_fr9Sj4M4G68SXbats7uxlQLuQNm0uIn3hrGP5Sf4x9VsRyvEHouVbVHB_6H0KOAz9JfYCwiNvIHFY5F9u13tYmcZoyA=>