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

Added Oracle Cloud ipv6 docs configuration #1088

Closed
wants to merge 1 commit into from

Conversation

szymonwilczek
Copy link

No description provided.

@topi314
Copy link
Member

topi314 commented Aug 16, 2024

this guide also uses tunnelbroker
how does this differ compared to https://lavalink.dev/configuration/ipv6/tunnelbroker.html?
I'd rather not have 2 copies of the same thing
if oracle cloud needs some extra commands then it should rather first redirect to the general tunnelbroker guide

@szymonwilczek
Copy link
Author

Main configuration in tunnelbroker guide uses example on netplan, which isn't configured as primary network interface for oracle. I thought the creation of images-like tutorial would be cool anyway

@topi314
Copy link
Member

topi314 commented Aug 16, 2024

how about you create a generic non netplan tunnelbroker variant then?
I think that would make more sense to cover as many setups as possible

@szymonwilczek
Copy link
Author

that would be great. I'll take a look at it when I have some time. i think we can close pr in that case

@topi314
Copy link
Member

topi314 commented Aug 16, 2024

you can keep this pr open and just push changes as you make those, but it's your choice

@topi314
Copy link
Member

topi314 commented Aug 16, 2024

your commits will get squashed into one at the end

@szymonwilczek
Copy link
Author

when I have some changes ready, i'll reopen again

- For `IP Protocol` change to: **ICMP**
So your fully configured `Ingress Rule` should look like this:

![image](https://github.com/user-attachments/assets/ef467326-05e2-4f3a-ac30-ae33726b66f4)
Copy link
Contributor

Choose a reason for hiding this comment

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

Wouldn't allowing the Echo Request ICMP packet (type 8) rather than all be enough? TunnelBroker's FAQ seems to suggest that.

Copy link
Author

Choose a reason for hiding this comment

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

Oh jesus, right. I completely forgot to exclude the All from the image. There was never meant to be the All since it's allowing too much.

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.

3 participants