Skip to content

Allow running any Ruby 2.3 version, upgrade to latest.#1675

Merged
monfresh merged 3 commits intomasterfrom
brody/ruby-2.3-latest
Sep 18, 2017
Merged

Allow running any Ruby 2.3 version, upgrade to latest.#1675
monfresh merged 3 commits intomasterfrom
brody/ruby-2.3-latest

Conversation

@brodygov
Copy link
Contributor

Allow running any ruby 2.3 version

Change the .ruby-version file to use 2.3 rather than 2.3.3. This may
require users to create an alias in their ruby version config to specify a
particular point release.

For example, in rbenv you would create a symlink from 2.3 to 2.3.5. If
you're using ruby-build and have rbenv-aliases installed, this will happen
automatically when you install new versions of ruby.

Also update the version in the Dockerfile and CircleCI config to not pin to a
specific point release.

This has no impact on login.gov servers, which don't use the .ruby-version
file. The Gemfile already allows any 2.3.* release after 2.3.3.

Enforce the use of ruby 2.3.5 or newer

https://www.ruby-lang.org/en/news/2017/08/29/multiple-vulnerabilities-in-rubygems/

https://github.com/18F/identity-devops-private/issues/502

Change the .ruby-version file to use `2.3` rather than `2.3.3`. This
may require users to create an alias in their ruby version config to
specify a particular point release.

For example, in rbenv you would create a symlink from `2.3` to `2.3.5`.
If you're using ruby-build and have rbenv-aliases installed, this will
happen automatically when you install new versions of ruby.

- https://github.com/tpope/rbenv-aliases

Also update the version in the Dockerfile and CircleCI config to not pin
to a specific point release.

This has no impact on login.gov servers, which don't use the
.ruby-version file. The Gemfile already allows any 2.3.* release after
2.3.3.
Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@monfresh monfresh left a comment

Choose a reason for hiding this comment

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

LGTM. Is there a corresponding devops issue that ensures the latest 2.3 release will be installed?

@monfresh monfresh merged commit 64cb474 into master Sep 18, 2017
@monfresh monfresh deleted the brody/ruby-2.3-latest branch September 18, 2017 15:19
@brodygov
Copy link
Contributor Author

Yes, we'll need to coordinate with https://github.com/18F/identity-devops/pull/618

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants