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

Ruby 2.5.8 #669

Merged
merged 2 commits into from
May 22, 2020
Merged

Ruby 2.5.8 #669

merged 2 commits into from
May 22, 2020

Conversation

eitoball
Copy link
Member

@eitoball eitoball commented May 6, 2020

ruby 2.5.8 is supported on Elastic Beanstalk. (https://docs.aws.amazon.com/elasticbeanstalk/latest/platforms/platforms-supported.html#platforms-supported.ruby)

The following are blogs related to Ruby 2.5.x updates. I don't think that changes in Ruby 2.5.x don't break api application.

@eitoball eitoball self-assigned this May 6, 2020
@sasharevzin
Copy link
Collaborator

@eitoball Let me know what is the status of this PR, please

@eitoball eitoball marked this pull request as ready for review May 16, 2020 22:55
@eitoball
Copy link
Member Author

Let me know what is the status of this PR, please

I thought that PR was in draft which meaning WIP (work in progress). Anyway, I needed to time to check out changes in Ruby 2.5.x, and check availability in Elastic Beanstalk.

@sasharevzin
Copy link
Collaborator

Let me know what is the status of this PR, please

I thought that PR was in draft which meaning WIP (work in progress). Anyway, I needed to time to check out changes in Ruby 2.5.x, and check availability in Elastic Beanstalk.

Sure thing. I was poking around to see what needs to be done upgrade to rails 6 (#647) and seems like rails works with Ruby 2.5 >= so was curious about status :)

@matschaffer
Copy link
Contributor

@sasharevzin
Copy link
Collaborator

https://docs.aws.amazon.com/elasticbeanstalk/latest/platforms/platforms-supported.html#platforms-supported.ruby says 2.5.8

Thanks @matschaffer
@eitoball Im curious, shell we skip 2.5 and jump right to 2.6?

@matschaffer
Copy link
Contributor

2.6.6 in that case. If it works I’m 👍 we can deploy to dev as a final check

@eitoball
Copy link
Member Author

Im curious, shell we skip 2.5 and jump right to 2.6?

We could, but I wouldn't do. If something happens, it would be easier to investigate because changes are smaller.

@sasharevzin
Copy link
Collaborator

Im curious, shell we skip 2.5 and jump right to 2.6?

We could, but I wouldn't do. If something happens, it would be easier to investigate because changes are smaller.

Sorry to ask but where smaller changes? In ruby version or in the app?

@eitoball
Copy link
Member Author

Sorry to ask but where smaller changes? In ruby version or in the app?

In ruby. When upgrading ruby version, if something happens, it could be caused by ruby's incompatible changes.

@sasharevzin
Copy link
Collaborator

In ruby. When upgrading ruby version, if something happens, it could be caused by ruby's incompatible changes.

But the same thing applies to 2.5, no? I think if there is an issue, we can always rollback to 2.5 from 2.6. It worth trying. Plus 2.6 will have a longer life than 2.5, right?

@matschaffer
Copy link
Contributor

I think what he’s getting at is that smaller ruby version jumps will likely mean smaller required code changes (either in the app or gem upgrade). And less risk of getting something wrong or mixed up.

That said, each jump does require some additional beanstalk env juggling so if the code is easy jumping versions it could make for less aws work.

@auspicacious
Copy link
Contributor

It's probably more efficient to upgrade to 2.5, run for ~1 week, then upgrade to 2.6, than to continue to debate. Changing platforms is not too difficult in Elastic Beanstalk. I'll try to do that this week.

@sasharevzin
Copy link
Collaborator

@auspicacious Thank you for your input (really).
My last message at this PR. I promise :)
If we have a pretty solid happy path test coverage (~82% at the last run) than maybe we can give a try for 2.6? If it will break something we can always roll it back, write a test to reproduce the failure, fix the failure, and try again? What do you think?

@auspicacious
Copy link
Contributor

I am not making a judgement about which opinion about the risk is right. My opinion is that we should be able to deploy and iterate rapidly, even when changing platforms, and that is most important, so that we can accept small contributions from many people.

@matschaffer
Copy link
Contributor

matschaffer commented May 19, 2020

Since we have this PR, and it passes, and @auspicacious is willing to work on the upgrade.

I'll say let's go 2.5 first.

@sasharevzin - if you felt inclined to start a 2.6 PR based on this one, I don't see a problem there.

Though @eitoball had been doing ruby/rails/ruby/rails in alternating upgrades.

So if there's a rails version that makes sense to bump first maybe we should do that.

Wish I could give more specific advise, but I don't have the time to keep up on what ruby/rails combos are compatible these days.

@sasharevzin sasharevzin self-requested a review May 19, 2020 10:33
Copy link
Collaborator

@sasharevzin sasharevzin left a comment

Choose a reason for hiding this comment

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

👍

@auspicacious auspicacious merged commit 9ebff99 into master May 22, 2020
@auspicacious auspicacious deleted the ruby-2.5.8 branch May 22, 2020 12:48
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.

4 participants