Path: csiph.com!news.mixmin.net!aioe.org!news.servidellagleba.it!bofh.it!news.nic.it!robomod From: Antonio Terceiro Newsgroups: linux.debian.maint.python Subject: Re: pybuild-autopkgtest (was: Notes from the DC22 Python Team BoF) Date: Tue, 26 Jul 2022 23:00:01 +0200 Message-ID: References: X-Mailbox-Line: From debian-python-request@lists.debian.org Tue Jul 26 20:54:05 2022 Old-Return-Path: X-Amavis-Spam-Status: No, score=-112.913 tagged_above=-10000 required=5.3 tests=[BAYES_00=-2, DKIMWL_WL_HIGH=-0.714, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, LDO_WHITELIST=-5, MURPHY_DRUGS_REL8=0.02, PGPSIGNATURE=-5, T_SCC_BODY_TEXT_LINE=-0.01, UNPARSEABLE_RELAY=0.001, USER_IN_DKIM_WELCOMELIST=-0.01, USER_IN_DKIM_WHITELIST=-100] autolearn=ham autolearn_force=no Mail-Followup-To: Antonio Terceiro , debian-python@lists.debian.org MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="zyATBGQa3Toy+E1P" Content-Disposition: inline X-Debian-User: terceiro X-Mailing-List: archive/latest/20120 List-ID: List-URL: List-Archive: https://lists.debian.org/msgid-search/YuBUWHZokBJmAQC0@debian.org Approved: robomod@news.nic.it Lines: 96 Organization: linux.* mail to news gateway Sender: robomod@news.nic.it X-Original-Date: Tue, 26 Jul 2022 17:53:44 -0300 X-Original-Message-ID: X-Original-References: <0239A172-64F5-4B2A-A903-F50760F50D3B@kitterman.com> Xref: csiph.com linux.debian.maint.python:14286 --zyATBGQa3Toy+E1P Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Jul 26, 2022 at 03:37:50PM +0000, Scott Kitterman wrote: >=20 >=20 > On July 26, 2022 2:50:19 PM UTC, Antonio Terceiro w= rote: > >On Mon, Jul 25, 2022 at 09:04:27AM +0100, Julian Gilbey wrote: > >> On Sat, Jul 23, 2022 at 07:52:19PM +0200, Louis-Philippe V=E9ronneau w= rote: > >> > =3D=3D pybuild improvements =3D=3D > >> >=20 > >> > getting the autopkgtest MR in would be great > >> >=20 > >> > https://salsa.debian.org/python-team/tools/dh-python/-/merge_request= s/27 > >> >=20 > >> > We need people to test this MR some more, although it seems fairly m= ature. > >> >=20 > >> > It might be a good idea to have a line in d/control to let us migrat= e from > >> > the existing autopkgtests running unit tests to the new automated MR. > >>=20 > >> I've been looking at this a bit more. I'm not sure what this last > >> paragraph means: the new "automated" autopkgtest will only be run if > >> the maintainer explicitly adds: > >>=20 > >> Testsuite: autopkgtest-pkg-pybuild > >>=20 > >> to debian/control (see the autodep8 MR at > >> https://salsa.debian.org/ci-team/autodep8/-/merge_requests/27/diffs - > >> it will never automatically detect a pybuild package). > >> And a maintainer would presumably only add that if they are also > >> removing their existing debian/tests/control (or want to run the > >> pybuild tests in addition). > >>=20 > >> An alternative would be for the autodep8 patch to try to determine > >> whether to run pybuild-autopkgtest. One approach could be: > >>=20 > >> if the package would run autopkgtest-pkg-python: > >> if debian/control does not contain an override_dh_auto_test stanza: > >> run pybuild-autopkgtest > >>=20 > >> Note, though, that if autodep8 is called, it will run all of the > >> detected tests. (At least that is what I believe happens from reading > >> /usr/bin/autodep8; I haven't double-checked this.) So, for example, > >> if a package specifies > >>=20 > >> Testsuite: autopkgtest-pkg-python > >>=20 > >> it will also run the autopkgtest-pkg-pybuild suite as it will be > >> detected as being a Python package, and vice versa. That is a > >> possible reason *not* to use the above suggestion, as it would > >> potentially run pybuild-autopkgtest even if not desired. > > > >I think the notes did not capture the consensus correctly. The point was > >that it should be possible to automate updating the `Testsuite:` field > >to run tests with pybuild-autopkgtest, and that we should probably do > >that across team packages with the help of some scripting. >=20 > I suppose this is fine as long as whoever does it fixes all the resulting= RC bugs. Well the idea is to only actually commit the change and upload the package with the new Testsuite value after ensuring that actually works, i.e. that the autopkgtest passes. We are not suggesting to do this blindly across the packages, sorry for not making that clear. --zyATBGQa3Toy+E1P Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCAAdFiEEst7mYDbECCn80PEM/A2xu81GC94FAmLgVFUACgkQ/A2xu81G C95wEBAAsZFQagvUNu5etn+w/wVJda8YU0l9kTKq1hTsyYb9QjfETYYq10/KO7+6 ePTdJG/sBcxICLK4fBAIQ8nkpESmEsog8A919qpUELLhkv9iSlW6vszJXd6XbjCW J+2lZUWy3qZcAceFIswAuNJEh6wTKXDe6rXlYo8S2q5Ua056qCZKqzOwd5v0e9gZ pCdsk6r4Q5NMCATl86IYneFhfnrMujArsHh3c+ZwUdEbQDBPlu9HjtwVKT3oJ+li CsfOZRHQDLFiRJlTonX3i0WCE2lSDFiZtCQ/aFfCBHGL/Jv7GE1b+XgGphpDG9BV mRsHGfk9ROCjY8V+djHjXqCSPhOawg8GzxKTgLVKu4+r2qjLJJADej+Wc55RUWAQ h7VamTK3HywDTOXApl0gx5l2t0NVgKeUOU6L6rIBaScLwvlT79+fMmNsac4mav4k 2qcKHb27TBiyVBtBCvsrLSzhxvpwt6jEGwLLAeiV1lDRP8aOXYGAaGxrr7J6UgD0 nXvEKlFwEPj6CJsdbmpinRjXKIe9Wftza2+makbJF2ATUZm5jgWwYJQZPfVob5Bb 4yTj5SRuHiAbYqXKcVRriJDhItIyczojLAp6Rf7bAZ122m7qjWZ8R0RTTTcoX9hL DZ9RiSCmaNs88KcPsi1T6lDPnOh42phEmmSVs6r4MhYwHelM2EE= =X8w9 -----END PGP SIGNATURE----- --zyATBGQa3Toy+E1P--