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.

Sign up for GitHub

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

Jump to bottom

fix setting ulimit #6319

Merged
merged 2 commits into from
May 14, 2019
Merged

fix setting ulimit #6319

merged 2 commits into from
May 14, 2019

Conversation

Stebalien
Copy link
Member

Not sure why CI wasn't failing.

@Stebalien
fix ulimit tests
c318e34 
Not sure why CI wasn't failing.

License: MIT
Signed-off-by: Steven Allen <steven@stebalien.com>
@Stebalien Stebalien mentioned this pull request May 11, 2019
Closed
@Stebalien Stebalien requested a review from Kubuxu May 13, 2019 21:03
@Stebalien
ulimit: fix setting ulimit
bff8326 
License: MIT
Signed-off-by: Steven Allen <steven@stebalien.com>
@Stebalien Stebalien changed the title fix ulimit tests fix setting ulimit May 13, 2019
@Stebalien Stebalien added the kind/bug A bug in existing code (including security flaws) label May 13, 2019
@Stebalien Stebalien merged commit 96e6dd2 into master May 14, 2019
@Stebalien Stebalien deleted the fix/ulimit-tests branch May 14, 2019 16:36
@Stebalien Stebalien restored the fix/ulimit-tests branch May 30, 2019 22:34
@Stebalien Stebalien deleted the fix/ulimit-tests branch May 30, 2019 22:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@magik6k magik6k magik6k approved these changes

@Kubuxu Kubuxu Awaiting requested review from Kubuxu

Assignees
No one assigned
Labels
kind/bug A bug in existing code (including security flaws)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Stebalien @magik6k