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

Remove ruby 2.1 support #5220

Merged
merged 6 commits into from
Aug 22, 2016
Merged

Remove ruby 2.1 support #5220

merged 6 commits into from
Aug 22, 2016

Conversation

lamont-granquist
Copy link
Contributor

Signed-off-by: Lamont Granquist lamont@scriptkiddie.org

@coderanger
Copy link
Contributor

Hrm, so this is going to make Travis testing a lot harder for me :-/ Not sure if that's a reason to block it but test matrix management is going to be even worse than it is now.

@lamont-granquist
Copy link
Contributor Author

lamont-granquist commented Aug 17, 2016

the whack-a-mole of gems dropping support for 2.1 already is only going to get worse, and for berkshelf it is already pretty intractable to get ruby 2.3/2.4 support while supporting 2.1 (i.e. next [major] version of berkshelf is gonna drop 2.1 support and that is already staged in master and ready to release)

and ruby 2.2 has been passing clean on travis for ages and is a pretty suitable ruby to test 12.x on if you need a single ruby to test on (which will inherently get some tests incorrect). i'm reasonably certain we had clean tests on 2.2 long before we ever managed to get windows and omnibus-chef onto 2.1

for a pattern to use to test old chef versions against the version of chef they were released against you can copy this approach:

https://github.com/chef-cookbooks/compat_resource/blob/master/.travis.yml

now that i look at that i think we can inject ruby 2.1.9 into the old rubies and modern bundler will actually be able to figure it out so i think that the old crap there can all be simplified a bit.

@coderanger
Copy link
Contributor

Yeah, that's what I think I need to move to, just need to write some helpers to generate the travis matrix for each project because I have so many. Probably not an issue for anyone else though so 👍

@lamont-granquist
Copy link
Contributor Author

looks like we need to wait until ruby-2.3 lands in -current to make the kitchen tests happy

@lamont-granquist
Copy link
Contributor Author

right so the way that i've used kitchen-appbundle-updater over kitchen-dokken means we're using the docker image with the prebaked chef-client rather than installing current from omnitruck.

Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
@lamont-granquist lamont-granquist merged commit c737f81 into master Aug 22, 2016
@lamont-granquist lamont-granquist deleted the lcg/drop-2.1-support branch August 22, 2016 18:56
@mwrock mwrock changed the title remove ruby 2.1 support Remove ruby 2.1 support Sep 9, 2016
@thommay thommay added Type: Enhancement Adds new functionality. and removed Enhancement labels Jan 25, 2017
@chef chef locked and limited conversation to collaborators Nov 16, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Type: Enhancement Adds new functionality.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants