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
feat(ulimits): additional ulimit options #13394
Conversation
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
3176d1c
to
2dca8ad
Compare
Can you add some tests in for these new options please? |
Hi @johnmccrae there are additions for the |
@chrisminton Thanks for pointing those out. I think we're good with kitchen tests, but see notes for additional cases/fields. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
one small bug.
2dca8ad
to
096f34d
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
@chrisminton Can you add the test that Phil requested? |
096f34d
to
db190db
Compare
@chrisminton Looks like the Buildkite build is broken due to an old configuration file. Can you rebase on main, please? |
- add as, cpu, maxlogins, msgqueue, rss, sigpending Signed-off-by: Chris Minton <chris.minton@sainsburys.co.uk>
Signed-off-by: Chris Minton <chris.minton@sainsburys.co.uk>
db190db
to
54c0872
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Approving workflows and tagging for community PR review. |
Description
user_ulimit
resourceRelated Issue
Types of changes
Checklist: