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

GHI-229 - Sidekiq service memory usage limits #243

Open
wants to merge 2 commits into
base: next-release
Choose a base branch
from

Conversation

mithat-skyvera
Copy link
Contributor

@mithat-skyvera mithat-skyvera commented Apr 23, 2024

Description of your patch

Declared MemorySwapMax and MemoryHigh limit for the sidekiq service. Added environment variable "EY_SIDEKIQ_WORKER_THRESHOLD_MEMORY_MB" for MemoryHigh limit of the sidekiq service. It is by default 80% of the MemoryMax. Related issue #229

Recommended Release Notes

Environment variable for threshold memory limit of sidekiq service.

Estimated Risk

Low ( After these changes are applied all sidekiq workers will have these limits applied)

Components involved

All clients who use sidekiq in their application

Dependencies

ey-sidekiq

Description of testing done

Create a v7 rails environment with sidekiq
Add environment variables to enable sidekiq in the environment
Run command "systemctl status [sidekiq worker name]" on the instance that sidekiq is running.
Look at the memory row in the output, Max, SwapMax and High limits should be declared.

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.

1 participant