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 Ruby 2.4, Add Ruby 2.6, Add k8s 1.17 #693

Merged
merged 6 commits into from
Feb 25, 2020
Merged

Drop Ruby 2.4, Add Ruby 2.6, Add k8s 1.17 #693

merged 6 commits into from
Feb 25, 2020

Conversation

dturn
Copy link
Contributor

@dturn dturn commented Feb 11, 2020

What are you trying to accomplish with this PR?
Add tests for K8s 1.17
Add tests for Ruby 2.6
Ruby 2.4 is EOL 3/31 might as well drop it 6 weeks early as long as I'm in this code.

How is this accomplished?
Update our pipeline file

What could go wrong?
Upset people who were still using 2.4. Though it's possible we don't cut a release before the 3/31 EoL date.

Sadly ruby 2.7 fails due to kuebclient ManageIQ/kubeclient#432

@dturn dturn requested review from DazWorrall, KnVerey and a team as code owners February 11, 2020 21:39
@dturn dturn changed the title Drop Ruby 2.4, Add Ruby 2.7, Add k8s 1.17 Drop Ruby 2.4, Add Ruby 2.6, Add k8s 1.17 Feb 11, 2020
@dturn dturn force-pushed the ruby-2-7 branch 2 times, most recently from 663c9cd to 53c7044 Compare February 11, 2020 22:41
@ghost ghost added the operations [ProdX] label Feb 13, 2020
env:
LOGGING_LEVEL: "4"
KUBERNETES_VERSION: v1.17-latest
RUBY_VERSION: "2.6"
Copy link
Contributor

Choose a reason for hiding this comment

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

How does this work? Isn't the container still circleci/ruby:2.5.7-node-browsers?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

.shopify-build/krane.yml Outdated Show resolved Hide resolved
@dturn dturn self-assigned this Feb 24, 2020
@dturn dturn requested a review from a team as a code owner February 25, 2020 18:31
@dturn dturn merged commit ebd94c4 into master Feb 25, 2020
@dturn dturn deleted the ruby-2-7 branch February 25, 2020 19:59
MrMarvin added a commit that referenced this pull request Mar 24, 2020
KnVerey pushed a commit that referenced this pull request Apr 13, 2020
* Drop ruby 2.4 and test using 2.6. 
* Smaller container
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.

None yet

3 participants