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

Drop CLI Ruby 2.1.0 support, upgrade installer embedded Ruby to 2.5.0 #3272

Merged
merged 9 commits into from
Feb 20, 2018

Conversation

kke
Copy link
Contributor

@kke kke commented Feb 13, 2018

Ruby 2.1 release branch has been EOL since 2017-03-31

Dropping automated check run on ruby 2.1 and replacing with 2.5.0.

Also the MacOS .pkg installer now installs an embedded 2.5.0 ruby instead of 2.1.

@kke kke changed the title (wip) Drop Ruby 2.1.0 support, build installer with embedded ruby 2.5.0 Drop CLI Ruby 2.1.0 support, build installer with embedded ruby 2.5.0 Feb 13, 2018
@kke
Copy link
Contributor Author

kke commented Feb 13, 2018

(Built pkg in dropbox Temp/) (built without signing key)

@jakolehm
Copy link
Contributor

@kke we should probably check that plugin deps work with ruby 2.5?

@kke
Copy link
Contributor Author

kke commented Feb 14, 2018

2.2 is EOLing on 2018-03-31

@kke kke changed the title Drop CLI Ruby 2.1.0 support, build installer with embedded ruby 2.5.0 Drop CLI Ruby 2.1.0 support, upgrade installer embedded ruby to 2.5.0 Feb 16, 2018
@kke kke changed the title Drop CLI Ruby 2.1.0 support, upgrade installer embedded ruby to 2.5.0 Drop CLI Ruby 2.1.0 support, upgrade installer embedded Ruby to 2.5.0 Feb 16, 2018
@kke kke added this to the 1.5.0 milestone Feb 19, 2018
@SpComb
Copy link
Contributor

SpComb commented Feb 19, 2018

I would consider tagging a .dev or .pre on this branch to test if the travis/jenkins baked .pkg and .deb both work.

@SpComb
Copy link
Contributor

SpComb commented Feb 19, 2018

The 1.5.0.pre2 deb package LGTM, and at least the cloud plugin seems to work fine with ruby 2.5.

@kke kke mentioned this pull request Feb 19, 2018
tags: true
os: osx
repo: kontena/kontena
rvm: 2.4
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2.4 or 2.5?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did not want to change build environment in #3294 ... that's why 2.4 is still there. This does not affect target embedded ruby version.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

BTW wouldn't it be better to bake the server/agent images and CLI releases using the same ruby version used to run the e2e tests? That would be 2.4?

@kke
Copy link
Contributor Author

kke commented Feb 20, 2018

Builds seem to work fine, tested the osx build and seems like basic stuff like cloud plugin are working.

I think this is ready for merge.

@kke kke merged commit 5d36625 into master Feb 20, 2018
@kke kke deleted the chore/drop_ruby21 branch February 20, 2018 12:56
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.

3 participants