-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[dhcp6relay] Support relaying Relay-Forward message #9887
Conversation
src/dhcp6relay/src/relay.h
Outdated
@@ -15,7 +15,7 @@ | |||
|
|||
#define RELAY_PORT 547 | |||
#define CLIENT_PORT 546 | |||
#define HOP_LIMIT 32 | |||
#define HOP_LIMIT 8 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why changing hop limit?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Correcting hop limit according to rfc: https://datatracker.ietf.org/doc/html/rfc8415#:~:text=8.%20%20%20Section%207%20(%22DHCP%20Constants%22)%20was%20primarily%20updated%20to%20add%0A%20%20%20%20%20%20%20%20constants%20from%20RFC%204242%20and%20RFC%207083.%20%20Note%20that%20the%20default%0A%20%20%20%20%20%20%20%20HOP_COUNT_LIMIT%20value%20was%20reduced%20from%2032%20to%208.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add a comment accordingly?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated
@kellyyeh - Can we have this fix in 202111 branch? |
Why I did it
Allow dhcp6relay to relay DHCPv6 Relay messages.
How I did it
Encapsulate Relay-Forward message in a new Relay-Forward message. No changes needed on the Relay-Reply from server.
How to verify it
Send a Relay Forward message, check if a new Relay Forward message was constructed and sent to the servers.
Tested with sonic-net/sonic-mgmt#5051
Which release branch to backport (provide reason below if selected)
Description for the changelog
A picture of a cute animal (not mandatory but encouraged)