Skip to content

Add ip6tables rule for DHCPv6 hop limit configuration#865

Merged
sknat merged 1 commit intomasterfrom
abasu-patch-hop-limit
Jan 30, 2026
Merged

Add ip6tables rule for DHCPv6 hop limit configuration#865
sknat merged 1 commit intomasterfrom
abasu-patch-hop-limit

Conversation

@aritrbas
Copy link
Collaborator

@aritrbas aritrbas commented Jan 24, 2026

Configure ip6tables mangle rule to set hop limit to 2 for DHCPv6 OUTPUT traffic from client (sport 546) to server (dport 547). This prevents VPP from dropping DHCPv6 SOLICIT/REQUEST packets when it decrements hop-limit by 1 during forwarding. Since clients generate SOLICIT/REQUEST with hop-limit=1, without this rule, VPP drops the packet (ip6 ttl <= 1) with ICMP time exceeded, causing DHCPv6 lease negotiation to fail.

  • The rule is checked for existence before adding to prevent duplicates since ip6tables does not auto-dedupe rules.
  • The rule is cleaned up during configuration restoration.

@aritrbas aritrbas force-pushed the abasu-patch-hop-limit branch from d86a5a8 to e7c77fc Compare January 24, 2026 07:14
@aritrbas aritrbas self-assigned this Jan 27, 2026
Copy link
Collaborator

@sknat sknat left a comment

Choose a reason for hiding this comment

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

Awesome ! Many thanks !

Configure ip6tables mangle rule to set hop limit to 2 for DHCPv6 OUTPUT
traffic from client (sport 546) to server (dport 547). This prevents VPP
from dropping DHCPv6 SOLICIT/REQUEST packets when it decrements hop-limit
by 1 during forwarding. Since clients generate SOLICIT/REQUEST with
hop-limit=1, without this rule VPP drops the packet (ip6 ttl <= 1)
with ICMP time exceeded, causing DHCPv6 lease negotiation to fail.

The rule is checked for existence before adding to prevent duplicates
since ip6tables does not auto-dedupe rules. The rule is also cleaned
up during configuration restoration.

Signed-off-by: Aritra Basu <aritrbas@cisco.com>
@aritrbas aritrbas force-pushed the abasu-patch-hop-limit branch from e7c77fc to 0fe96f3 Compare January 29, 2026 23:17
@sknat sknat merged commit 6364745 into master Jan 30, 2026
5 checks passed
@sknat sknat deleted the abasu-patch-hop-limit branch January 30, 2026 16:57
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.

2 participants