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

runc: Add patches from runc #2871 #1546

Merged
merged 1 commit into from
May 3, 2021

Conversation

zmrow
Copy link
Contributor

@zmrow zmrow commented Apr 30, 2021

Issue number:
N/A

Description of changes:

This change adds patches from [0] to mitigate an issue customers were
seeing where pods could not be scheduled because allocated IP addresses
were not being released.

[0]- https://github.com/opencontainers/runc/pull/2871

If runc-rc94 gets released first, we'll use that instead.

Testing done:
A Bottlerocket user helped test a build with this change and confirmed that their pod scheduling issues were resolved

Terms of contribution:

By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.

@zmrow zmrow requested a review from bcressey April 30, 2021 16:13
@zmrow zmrow changed the title runc: Add patches from runc/#2871 runc: Add patches from runc #2871 Apr 30, 2021
@bcressey bcressey requested a review from samuelkarp April 30, 2021 18:11
This change adds patches from [0] to mitigate an issue customers were
seeing where pods could not be scheduled because allocated IP addresses
were not being released.
[0]- opencontainers/runc#2871
@zmrow zmrow force-pushed the runc-seccomp-patch branch from 3940ee3 to 6c3e4e1 Compare April 30, 2021 18:40
@zmrow
Copy link
Contributor Author

zmrow commented Apr 30, 2021

^ Update the commit message for the terrible line wrapping.

@zmrow zmrow merged commit ab5b94f into bottlerocket-os:develop May 3, 2021
@zmrow zmrow deleted the runc-seccomp-patch branch May 3, 2021 16:32
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.

4 participants