Re: revisiting CI setup
Bill Deegan <bill-cJFiu+DHMVC5azolltMz9laTQe2KTcn/@public.gmane.org> Mon, 16 Dec 2019 15:30:06 -0500
| Newsgroups | gmane.comp.programming.tools.scons.devel |
|---|---|
| Message-ID | <CAEyG4CF7SP3FB_H0LObPN-0h4nBiN3exgG1Sboj9hjBK8E2x0Q@mail.gmail.com> |
--===============7451300024578658720== Content-Type: multipart/alternative; boundary="000000000000630baf0599d8146d" --000000000000630baf0599d8146d Content-Type: text/plain; charset="UTF-8" I'd say since appveyor windows builds have sped up so much we can decrease the split per py type from /4 to maybe /2 or maybe /1 I think the max runtime per build is 1hr? On Mon, Dec 16, 2019 at 2:26 PM Mats Wichmann <[email protected]> wrote: > > time passes... so time to ask some questions again. > > > The travis build does two code-coverage runs (2.7 and 3.6), then seven > regular runs (2.7, 3.5, 3.6, 3.7, 3.8-dev, pypy2 and pypy3. > > The appveyor build, despite a recent caching hack that sped it up a lot, > is still much slower, and thus is asked to do less work, but has a more > complex scenario: it needs to test itself on three different appveyor > images, each provisioned with a different Visual Studio version, so what > it runs is: 2.7 on VS2017 image, 3.5 on VS2015 image, 3.6 on VS2017 > image (this is also a coverage run), 3.7 on VS2019 image. > > We're having some test problems due to evolution of Windows images - in > particular (there are github issues on this), clang tests do not run > properly, because before it always found the mingw clang, but in the > VS2019 image that is no longer installed by default, and the setup > doesn't work with a "native" clang - basically this exposed an untested > issue, rather than breaking something that was working. > > 3.8 is fully released now. only the Linux CI (travis) is running the > tests under 3.8, and it's using "3.8-dev" as opposed to an officially > released version of 3.8. Although it's early days yet, there ought to be > a 3.9-dev (last I checked, appveyor hadn't enabled that choice, but that > was a while back) > > what sets of these are actually important for checking that a PR does > not introduce testing regressions? > > I'm unconvinced the pypy tests add any value in this context. Maybe we > could move them to the buildbot setup instead? In particular, pypy3 has > never gained the performance of pypy2, and doesn't seem to be in active > development, and the performance is a killer for our use of it. > > We should be able to stop testing 2.7 once develoment flips to "4.0", > since it won't carry the compatibility promise any longer. > > Is there any other way to rearrange/improve this, including the > mentioned buildbot - things which aren't deemed critical to run on every > PR but which still would be useful to see the progress of could move to > buildbot.... > > _______________________________________________ > Scons-dev mailing list > [email protected] > https://pairlist2.pair.net/mailman/listinfo/scons-dev > --000000000000630baf0599d8146d Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"ltr"><div>I'd say since appveyor windows builds have sped u= p so much we can decrease the split per py type from /4 to maybe /2 or mayb= e /1 <br></div><div>I think the max runtime per build is 1hr?<br></div></di= v><br><div class=3D"gmail_quote"><div dir=3D"ltr" class=3D"gmail_attr">On M= on, Dec 16, 2019 at 2:26 PM Mats Wichmann <<a href=3D"mailto:mats@wichma= nn.us">[email protected]</a>> wrote:<br></div><blockquote class=3D"gmail_= quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,= 204);padding-left:1ex"><br> time passes... so time to ask some questions again.<br> <br> <br> The travis build does two code-coverage runs (2.7 and 3.6), then seven <br> regular runs (2.7, 3.5, 3.6, 3.7, 3.8-dev, pypy2 and pypy3.<br> <br> The appveyor build, despite a recent caching hack that sped it up a lot, <b= r> is still much slower, and thus is asked to do less work, but has a more <br= > complex scenario: it needs to test itself on three different appveyor <br> images, each provisioned with a different Visual Studio version, so what <b= r> it runs is: 2.7 on VS2017 image, 3.5 on VS2015 image, 3.6 on VS2017 <br> image (this is also a coverage run), 3.7 on VS2019 image.<br> <br> We're having some test problems due to evolution of Windows images - in= <br> particular (there are github issues on this), clang tests do not run <br> properly, because before it always found the mingw clang, but in the <br> VS2019 image that is no longer installed by default, and the setup <br> doesn't work with a "native" clang - basically this exposed a= n untested <br> issue, rather than breaking something that was working.<br> <br> 3.8 is fully released now.=C2=A0 only the Linux CI (travis) is running the = <br> tests under 3.8, and it's using "3.8-dev" as opposed to an of= ficially <br> released version of 3.8. Although it's early days yet, there ought to b= e <br> a 3.9-dev (last I checked, appveyor hadn't enabled that choice, but tha= t <br> was a while back)<br> <br> what sets of these are actually important for checking that a PR does <br> not introduce testing regressions?<br> <br> I'm unconvinced the pypy tests add any value in this context. Maybe we = <br> could move them to the buildbot setup instead? In particular, pypy3 has <br= > never gained the performance of pypy2, and doesn't seem to be in active= <br> development, and the performance is a killer for our use of it.<br> <br> We should be able to stop testing 2.7 once develoment flips to "4.0&qu= ot;, <br> since it won't carry the compatibility promise any longer.<br> <br> Is there any other way to rearrange/improve this, including the <br> mentioned buildbot - things which aren't deemed critical to run on ever= y <br> PR but which still would be useful to see the progress of could move to <br= > buildbot....<br> <br> _______________________________________________<br> Scons-dev mailing list<br> <a href=3D"mailto:[email protected]" target=3D"_blank">[email protected]= g</a><br> <a href=3D"https://pairlist2.pair.net/mailman/listinfo/scons-dev" rel=3D"no= referrer" target=3D"_blank">https://pairlist2.pair.net/mailman/listinfo/sco= ns-dev</a><br> </blockquote></div> --000000000000630baf0599d8146d-- --===============7451300024578658720== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Scons-dev mailing list [email protected] https://pairlist2.pair.net/mailman/listinfo/scons-dev --===============7451300024578658720==--