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

Optimization: Upgrade Puma to 5.0.2 #10928

Merged
merged 1 commit into from Oct 20, 2020
Merged

Optimization: Upgrade Puma to 5.0.2 #10928

merged 1 commit into from Oct 20, 2020

Conversation

mstruve
Copy link
Contributor

@mstruve mstruve commented Oct 19, 2020

What type of PR is this? (check all applicable)

  • Bug Fix
  • Optimization

Description

The problems we were seeing in Heroku with the original Puma 5 gem upgrade look to have been resolved.

In addition, this broke our container builds. Since upgrading I rebuilt and launched a new Docker container without an issue.

Related Tickets & Documents

https://github.com/orgs/forem/projects/6#card-45908891

QA Instructions, Screenshots, Recordings

Build and launch a docker container.

alt_text

@pr-triage pr-triage bot added the PR: draft bot applied label for PR's that are a work in progress label Oct 19, 2020
@mstruve mstruve marked this pull request as ready for review October 19, 2020 17:12
@mstruve mstruve requested a review from a team October 19, 2020 17:12
@mstruve mstruve requested a review from a team as a code owner October 19, 2020 17:12
@mstruve mstruve requested review from jacobherrington and removed request for a team October 19, 2020 17:12
@pr-triage pr-triage bot added PR: unreviewed bot applied label for PR's with no review and removed PR: draft bot applied label for PR's that are a work in progress labels Oct 19, 2020
Copy link
Contributor

@benhalpern benhalpern left a comment

Choose a reason for hiding this comment

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

lgtm

@pr-triage pr-triage bot added PR: partially-approved bot applied label for PR's where a single reviewer approves changes and removed PR: unreviewed bot applied label for PR's with no review labels Oct 19, 2020
@benhalpern
Copy link
Contributor

This is good to go @mstruve if you want to merge it to observe any little changes to prod that might happen.

@mstruve mstruve merged commit 3cc9b20 into master Oct 20, 2020
@mstruve mstruve deleted the mstruve/puma-5 branch October 20, 2020 21:33
@pr-triage pr-triage bot added PR: merged bot applied label for PR's that are merged and removed PR: partially-approved bot applied label for PR's where a single reviewer approves changes labels Oct 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: merged bot applied label for PR's that are merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants