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

V0.2.2 + back-ported [plugins/bridge: clean masquerading rules] #202

Closed
wants to merge 6 commits into from

Conversation

tmrts
Copy link

@tmrts tmrts commented May 11, 2016

Backports #195 to v0.2.2, which is a fix for iptable rule leaks in rkt (See rkt#2565)

steveej and others added 6 commits April 22, 2016 19:36
This script uses rkt and a fedora image to build release tarballs.
For two reasons:

1. They're not functional within rkt
2. They rebuild the binaries dynamically
In the Add command we set up masquerading rules that didn't have a
corresponding clean-up code in Del.

Add the clean-up code.
@iaguis
Copy link
Contributor

iaguis commented May 11, 2016

I think we need #177 too to fix rkt/rkt#1765 for the nspawn flavor.

@tmrts
Copy link
Author

tmrts commented May 11, 2016

@iaguis Alright I'll turn this into v0.2.3, do you have any more fixes in mind @jonboulle @alban?

@jonboulle
Copy link
Contributor

nope, not that I can think of

@jonboulle
Copy link
Contributor

I think @steveej had concerns about the #177 patch though..

@jonboulle
Copy link
Contributor

Let's hold off on that one for now. #2565 is a lot more critical.

@jonboulle
Copy link
Contributor

@tmrts github is confusing me - is this against v0.2.2, even though it says "tmrts wants to merge 6 commits into containernetworking:master from tmrts:v0.2.2"?

@tmrts
Copy link
Author

tmrts commented May 11, 2016

@jonboulle I specifically selected a new branch, I'll make sure that's the case before merging

@tmrts
Copy link
Author

tmrts commented May 11, 2016

@jonboulle as it turns out, I can't even push a PR to a non-existant branch without write access.

@steveej
Copy link
Member

steveej commented May 11, 2016

Closing in favor of #203

@steveej steveej closed this May 11, 2016
@tmrts tmrts deleted the v0.2.2 branch May 11, 2016 16:18
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