[Twisted-Python] T9217 / PR1052: Wheels, wheels, and more wheels

[Glyph]

> On Aug 29, 2018, at 10:13 AM, Kyle Altendorf <sda at fstab.net <mailto:sda at fstab.net>> wrote:
> 
> Hi All,

Hi Kyle!

> I am working on ticket #9217 / PR #1051 to add lots more wheel generation to the Twisted CI.  I decided to give the cibuildwheel package a try and it made this process almost too easy (well... sort of :] ).  I've got AppVeyor-Windows, Travis-Linux, and Circle-OSX all building a variety of wheels for the supported Python versions and bit depths.  Travis doesn't save artifacts 'easily' so I went ahead and doubled up on Linux on Circle for now, though it's having some Docker issues at the moment and hasn't been successful yet.  For some reason in this one case the project directory isn't getting mounted into the container as expected.

Thanks for doing this!

> wheel links:
> https://github.com/twisted/twisted/pull/1051#issuecomment-416743261 <https://github.com/twisted/twisted/pull/1051#issuecomment-416743261> (and next comment)
> 
> Now that I've got the wheel builds happening I figured it'd be good to try them out on 'real' machines.  Turns out we get a failure on twisted.cred.test.test_strcred.SSHCheckerTests.test_isChecker for at least the two checks I've done so far (Windows and OSX).  I haven't done more than a cursory look at that yet, but it's on the list to understand and resolve.  More testing would of course be welcome.  Real world, just trial Twisted's own tests, whatever would be appreciated if you are interested.
> 
> failures with wheels:
> https://github.com/twisted/twisted/pull/1051#issuecomment-416977723 <https://github.com/twisted/twisted/pull/1051#issuecomment-416977723>

It's not clear to me that we have any test environments that accurately test what happens after you install from PyPI.  If we could build these wheels and then actually run tests on installing them that would be a big step forward in test fidelity!  But given the current state of the art, I'd be happy if they just got built from the same code that passed our tests in existing configurations.

> Overall, it's a bit unclear what the intended use of the various CI hosts are for Twisted.

Ideally, Travis would handle it all.

But Travis can't run multiple kernels, so we have buildbots for lots of alternate kernels, just to make sure.  Including BSD kernels, since those are actually substantially different where sockets are concerned.

And Travis macOS support is so slow that it was a serious blocker for a while, so the buildbot is filling in until Circle can take over.

And Travis can't do Windows, so that's where appveyor comes in.

There might still be various non-special-kernel testing things that buildbot does, but those are probably holdovers that, if they still have a reason to exist, it's exactly because Travis doesn't have artifact hosting.

Hopefully that's comprehensive?

> I hear that Travis OSX builds were really slow, but from what I can see Circle isn't doing any OSX (other than what I added).

The issue for addressing this is https://twistedmatrix.com/trac/ticket/9445 <https://twistedmatrix.com/trac/ticket/9445> if you'd like to pick up where Adi left off.  It's probably just resolving some conflicts, resubmitting for review, and bugging people to get it through the process :).

> There wasn't any artifact storage being used on Circle either.  So, I'm not sure if there was a reason to use Circle that went away, or...  But, not having to hook Travis up to S3 or somesuch for storage is quite nice so Circle wins at least in that category.

I think the big problem on Circle is capacity limits, but I don't know if that's just a macOS thing, or what.

> Now that I've got something rough in place, are there any opinions about how this should work?  I don't know the present release workflow so I don't know if we'd want an automatic push to PyPI on tags (probably not)

I definitely want this.  This is really the whole point, what we're trying to get to :).  Why don't you think we want this?

> or just artifacts on the build server to grab manually (would need some S3 or such for Travis, or Circle for Linux builds as well).  Anything else?  Do we want automated tests against the wheels?  cibuildwheel does have a feature for that though I haven't done anything with it yet.

Ideally we'd push the wheels on tag once the tests passed, but I'd be OK with just unconditionally just building / pushing on tag for starters.

> Anyways...  hello, thanks for Twisted, and I hope this work ends up saving some people some time.

I can't wait for fully automated releases :).
-------------- next part --------------
An HTML attachment was scrubbed...
URL: </pipermail/twisted-python/attachments/20180905/610c4203/attachment.html>