Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bpo-36307: Upgrade Travis CI to Xenial and rm obsolete sudo:false #12356

Merged
merged 1 commit into from
Mar 18, 2019

Conversation

CAM-Gerlach
Copy link
Member

@CAM-Gerlach CAM-Gerlach commented Mar 15, 2019

Currently, the .travis.yml config file still has a sudo: false key, which previously triggered the Container-based environment on Travis. However, that was removed in December 2018, and so the key is now obsolete. Furthermore, the builds are still running in the Trusty (Ubuntu 14.04) environment, which will be at the end of its 5-year development lifecycle in April 2019 (next month). Therefore, sudo: false should be removed, and the presumably, if possible, the distribution should be upgraded to the modern dist: xenial (Ubuntu 16.04).

https://bugs.python.org/issue36307

@CAM-Gerlach
Copy link
Member Author

CAM-Gerlach commented Mar 15, 2019

Okay, looks like everything's green with the same number of skips; aside from the allowed failures, which fail in the same way as on other PRs with the previous infrastructure. As a minor, 2-line meta-change to the CI infrastructure, I didn't think this merited a NEWS entry, but I'm happy to add one if desired.

@tirkarthi
Copy link
Member

There were Openssl compatibility issues in the past and xenial used to be less stable. I would request waiting for people working closely with buildbots to review this before merging and also backporting since this changes base image of one of primary CI. See also some discussion in https://bugs.python.org/issue34009 .

@CAM-Gerlach
Copy link
Member Author

CAM-Gerlach commented Mar 15, 2019

Thanks @tirkarthi .

There were Openssl compatibility issues in the past

To note, at least according to the issue you linked, these issues were reported with Trusty, whereas the OpenSSL version distributed with Xenial is compatible. Therefore, we could switch back to the distro-provided version, though it makes sense to keep it building its own version to demonstrate how to do so, exercise that functionality and ensure compatibility with the latest OpenSSL.

While were at it, should we also bump the version up to the latest 1.1.1b, which is the current LTS branch (with 1.0.2 EoL in Dec. 2019 and 1.1.0 EoL in a few months, in September)?

xenial used to be less stable

We will indeed want to confirm this with the relevant parties, as this is not a trivial change, but I do note just based on the thread, the one individual who commented about Xenial not working reliably later took it back, saying it only required extra configuration (which is no longer the case). However, there may be other issues of which I'm aware (we ourselves haven't had any since spyder-ide/spyder#8802 , but obviously we're not exactly CPython).

@tirkarthi
Copy link
Member

Thanks for the details. I was curious and just used search where the thread came up. I too would be happy to have it clarified along with backporting by someone else for the upgrade.

@CAM-Gerlach
Copy link
Member Author

I searched for related terms as well, but for whatever reason didn't find that particular thread. Thanks for the heads up.

@methane methane merged commit 74ae50e into python:master Mar 18, 2019
@miss-islington
Copy link
Contributor

Thanks @CAM-Gerlach for the PR, and @methane for merging it 🌮🎉.. I'm working now to backport this PR to: 2.7, 3.7.
🐍🍒⛏🤖

methane pushed a commit that referenced this pull request Mar 18, 2019
(cherry picked from commit 74ae50e)

Co-authored-by: CAM Gerlach <[email protected]>
@bedevere-bot
Copy link

GH-12404 is a backport of this pull request to the 2.7 branch.

methane pushed a commit that referenced this pull request Mar 18, 2019
(cherry picked from commit 74ae50e)

Co-authored-by: CAM Gerlach <[email protected]>
@bedevere-bot
Copy link

GH-12405 is a backport of this pull request to the 3.7 branch.

methane added a commit that referenced this pull request Mar 19, 2019
(cherry picked from commit 74ae50e)

Co-authored-by: CAM Gerlach <[email protected]>
methane added a commit that referenced this pull request Mar 19, 2019
(cherry picked from commit 74ae50e)

Co-authored-by: CAM Gerlach <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants