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

Make HairpinMode configurable #71

Merged
merged 1 commit into from
Jun 8, 2022
Merged

Conversation

spowelljr
Copy link
Contributor

@spowelljr spowelljr commented Jun 7, 2022

HairpinMode was hardcoded to none, resulting in some CNIs failing to hairpin when desired.

Added a hairpin-mode flag that allows the user to choose between the three valid options. The default stays as none.

Error when invalid value passed:
"test123" is not a valid hairpin-mode value, must be one of: ["promiscuous-bridge", "hairpin-veth", "none"]

@evol262 evol262 merged commit 5600fb0 into Mirantis:master Jun 8, 2022
@spowelljr spowelljr deleted the fixHairPin branch June 8, 2022 14:56
jandubois added a commit to lima-vm/alpine-lima that referenced this pull request Jun 8, 2022
Changes relevant for Lima are:

* Fix not reporting memory usage: Mirantis/cri-dockerd#66
* Make HairpinMode configurable: Mirantis/cri-dockerd#71

Signed-off-by: Jan Dubois <jan.dubois@suse.com>
jandubois added a commit to rancher-sandbox/rancher-desktop-wsl-distro that referenced this pull request Jun 8, 2022
Changes relevant for Lima are:

* Fix not reporting memory usage: Mirantis/cri-dockerd#66
* Make HairpinMode configurable: Mirantis/cri-dockerd#71

Signed-off-by: Jan Dubois <jan.dubois@suse.com>
evol262 added a commit that referenced this pull request Aug 5, 2022
Make HairpinMode configurable
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.

None yet

2 participants