puppet-jira
puppet-jira copied to clipboard
voxpupuli
Apply jvm_nofiles_limit to systemd service
Services started with systemd on CentOS 7, and possibly other platforms ignore ulimit unless the LimitNOFILE parameter is set in the systemd unit file.
https://bugzilla.redhat.com/show_bug.cgi?id=754285
Hey @root-expert or @bastelfreak - is it possible to get this fix in before the next release of the module?
This PR addresses the open file limits health check failure that could arise on systemd instances. https://confluence.atlassian.com/jirakb/health-check-open-files-limit-838555405.html
Hello @nambrosch and @jmcnatt!
The patch seems okay but a unit test should be added before merging this. I think the unit test can be added in jira_service_spec file.
@root-expert Thanks for reviewing this, I don't think the service spec file needs to be updated since adding LimitNOFILE adds any new requirements, it's part of systemd, and it uses the value of a puppet variable, $jira::jvm_nofiles_limit, that already exists.
@nambrosch The unit tests for config files are mostly used to test something is render correctly to the file and it won't get broken in the future by some other contribution :smile:
@root-expert It's not obvious to me what should be added. Any guidance that can be provided would be appreciated.
@nambrosch @root-expert would this be as simple as adding a content line in ./spec/classes/jira_service_spec.rb?
Hmm, on Centos stream8, Ubuntu 18.04 and 20.04 this actually reduces the ulimit from 1048576 to 16384. Is this a breaking change then?
@nambrosch could you please rebase your branch against voxpupuli:master?
all set. there are still some failing CI jobs but I don't believe they have anything to do with this PR.