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

document cilium requirement when enabling wireguard #290

Merged
merged 2 commits into from
Dec 16, 2024
Merged

document cilium requirement when enabling wireguard #290

merged 2 commits into from
Dec 16, 2024

Conversation

enneitex
Copy link
Contributor

@enneitex enneitex commented Dec 6, 2024

Adding requirement for cilium + wireguard setup.
This port is documented in cilium doc

@enneitex enneitex requested a review from a team as a code owner December 6, 2024 17:18
brandond
brandond approved these changes Dec 6, 2024
View reviewed changes
Copy link
Member

@brandond brandond left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cc @manuelbuil

@manuelbuil
Copy link
Contributor

Thanks! Could you just say Wireguard, like in the other CNI plugin? I don't think the "transparent encryption" part adds any important detail

@manuelbuil manuelbuil merged commit ff370cd into rancher:main Dec 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@manuelbuil manuelbuil manuelbuil approved these changes

@brandond brandond brandond approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@enneitex @manuelbuil @brandond