engineyard / ey-cloud-recipes

A starter repo for custom chef recipes on EY's cloud platform. These are for reference, and do not indicate a supported status.
http://www.engineyard.com/products/cloud
Other
980 stars 291 forks source link

[delayed_job4] Update dj.erb #348

Closed radamanthus closed 6 years ago

radamanthus commented 7 years ago

Backport changes made in V5 to improve worker termination

radamanthus commented 6 years ago

This fixes #345

jegasega commented 6 years ago

Description of your patch

Recommended Release Notes

Ensure that dj worker processes do not become orphans

Estimated risk

Low

Components involved

DelayedJob custom chef recipe

Description of testing done

See QA instructions

QA Instructions

Test on configuration A_dj

Configuration A rails_activejob_example (delayed_job branch) App Unicorn Ruby 2.3 RubyGems 2.6.4 Postgres 9.4 US East Virginia Solo

Boot the test environment under the QA stack Enable the delayed_job recipe. Modify the recipe to install DelayedJob on the solo instance Modify the recipe and set a very low worker memory limit (e.g. low enough to always trigger the memory limit even with zero workload, e.g. 10MB). Upload custom cookbooks to the environment Run chef Observe the delayed_job processes by running ps -ef | grep elay Make sure:

jegasega commented 6 years ago

Tested the pull request against provided testing procedure. Everything works like expected. LGTM from my side.