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

Removed advanced config option to switch between "puma" and "thin" #19304

Conversation

yrudman
Copy link
Contributor

@yrudman yrudman commented Sep 17, 2019

Removed advanced config option to switch between "puma" and "thin"

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1752870

@miq-bot add-label core, technical debt

@yrudman
Copy link
Contributor Author

yrudman commented Sep 17, 2019

@miq-bot add-label ivanchuk/yes

@miq-bot
Copy link
Member

miq-bot commented Sep 17, 2019

Checked commit yrudman@8ba1f61 with ruby 2.4.6, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 🏆

@@ -1014,7 +1014,6 @@
:prefetch_max_per_worker_dequeue: 100
:prefetch_min_per_worker_dequeue: 10
:prefetch_stale_threshold: 30.seconds
:rails_server: puma
Copy link
Member

Choose a reason for hiding this comment

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

Do we need a data migration to drop the setting if it was changed previously?

Copy link
Contributor Author

@yrudman yrudman Sep 18, 2019

Choose a reason for hiding this comment

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

we will have not used record without migration.
Migration PR created: ManageIQ/manageiq-schema#416

@yrudman
Copy link
Contributor Author

yrudman commented Sep 18, 2019

@miq-bot add-label changelog/yes

@kbrock kbrock merged commit 69d5c32 into ManageIQ:master Sep 20, 2019
@kbrock kbrock added this to the Sprint 121 Ending Sep 30, 2019 milestone Sep 20, 2019
@kbrock kbrock self-assigned this Sep 20, 2019
@yrudman yrudman deleted the remove-config-option-to-switch-between-puma-and-thin branch September 20, 2019 19:09
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.

4 participants