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

Projects
None yet
7 participants
@CAM-Gerlach
Copy link
Contributor

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

This comment has been minimized.

Copy link
Contributor 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

This comment has been minimized.

Copy link
Contributor

tirkarthi commented Mar 15, 2019

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

This comment has been minimized.

Copy link
Contributor 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

This comment has been minimized.

Copy link
Contributor

tirkarthi commented Mar 16, 2019

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

This comment has been minimized.

Copy link
Contributor Author

CAM-Gerlach commented Mar 16, 2019

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

5 checks passed

Azure Pipelines PR #20190315.46 succeeded
Details
bedevere/issue-number Issue number 36307 found
Details
bedevere/news "skip news" label found
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@miss-islington

This comment has been minimized.

Copy link

miss-islington commented Mar 18, 2019

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 added a commit that referenced this pull request Mar 18, 2019

[2.7] bpo-36307: Travis: upgrade to Xenial environment (GH-12356)
(cherry picked from commit 74ae50e)

Co-authored-by: CAM Gerlach <CAM.Gerlach@Gerlach.CAM>
@bedevere-bot

This comment has been minimized.

Copy link

bedevere-bot commented Mar 18, 2019

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

methane added a commit that referenced this pull request Mar 18, 2019

[3.7] bpo-36307: Travis: upgrade to Xenial environment (GH-12356)
(cherry picked from commit 74ae50e)

Co-authored-by: CAM Gerlach <CAM.Gerlach@Gerlach.CAM>
@bedevere-bot

This comment has been minimized.

Copy link

bedevere-bot commented Mar 18, 2019

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

bpo-36307: Travis: upgrade to Xenial environment (GH-12356)
(cherry picked from commit 74ae50e)

Co-authored-by: CAM Gerlach <CAM.Gerlach@Gerlach.CAM>

methane added a commit that referenced this pull request Mar 19, 2019

bpo-36307: Travis: upgrade to Xenial environment (GH-12356)
(cherry picked from commit 74ae50e)

Co-authored-by: CAM Gerlach <CAM.Gerlach@Gerlach.CAM>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.