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

elapsed-time attribute in DHCPv6 SOLICIT #281

Closed
aleheux-tc opened this issue Sep 3, 2024 · 7 comments
Closed

elapsed-time attribute in DHCPv6 SOLICIT #281

aleheux-tc opened this issue Sep 3, 2024 · 7 comments
Assignees
Labels
bug Something isn't working fixed ipoe IPoE

Comments

@aleheux-tc
Copy link

Describe the bug

Not entirely sure if this is a bug or me being a n00b:

While testing our DHCPv6 server's fail-over abilities I noticed that the elapsed-time attribute remains 0.
In this scenario the DHCPv6 server is not actually responding to any requests so bngblaster will keep trying to send SOLICITs every 5 seconds.

The transaction-id attribute remains constant and rfc8415 says:

  elapsed-time         The amount of time since the client began its
                       current DHCP transaction.  This time is
                       expressed in hundredths of a second
                       (10^-2 seconds).  A 2-octet field containing
                       an unsigned integer.

Which suggests to me that it should increase.

Version (bngblaster -v):

bngblaster -v

Version: 0.9.5
Compiler: GNU (9.4.0)
IO Modes: packet_mmap_raw (default), packet_mmap, raw

JSON configuration:

{
}

Steps to reproduce the behavior:

  1. turn off DHCP server
  2. start bngblaster
  3. observe DHCP requests with tcpdump or wireshark

Expected behavior

I expect the elapsed-time attribute to increase (which in this case signals to the standby DHCP server that something's amiss)

Screenshots

If applicable, add screenshots to help explain your problem.

Additional context

Add any other context about the problem here.

@aleheux-tc aleheux-tc added the bug Something isn't working label Sep 3, 2024
@GIC-de
Copy link
Member

GIC-de commented Sep 3, 2024

The elapsed time is currently not incrementing, but let me see if this can be done quickly.

@aleheux-tc
Copy link
Author

aleheux-tc commented Sep 4, 2024

A bit more background on why the elapsed time attribute is useful:

From the Kea DHCP manual:

max-ack-delay - is one of the parameters controlling partner failure-detection. When communication with the partner is interrupted, the server examines the values of the secs field (DHCPv4) or Elapsed Time option (DHCPv6), which denote how long the DHCP client has been trying to communicate with the DHCP server. This parameter specifies the maximum time in milliseconds for the client to try to communicate with the DHCP server, after which this server assumes that the client failed to communicate with the DHCP server (is “unacked”). The default value of this parameter is 10000.

max-unacked-clients - specifies how many “unacked” clients are allowed (see max-ack-delay) before this server assumes that the partner is offline and transitions to the partner-down state. The special value of 0 is allowed for this parameter, which disables the failure-detection mechanism. In this case, a server that can’t communicate with its partner over the control channel assumes that the partner server is down and transitions to the partner-down state immediately. The default value of this parameter is 10.

When the "elapsed time" is always 0 the max-unacked-clients will always remain zero.

@GIC-de
Copy link
Member

GIC-de commented Sep 4, 2024

Could you try with latest dev branch (see b427699)!

@GIC-de GIC-de added ipoe IPoE fixed in-progress Work in progress and removed in-progress Work in progress labels Sep 4, 2024
@aleheux-tc
Copy link
Author

@GIC-de
Copy link
Member

GIC-de commented Sep 4, 2024

There was a regression from a ISIS change, please try again.

@aleheux-tc
Copy link
Author

aleheux-tc commented Sep 4, 2024 via email

@GIC-de
Copy link
Member

GIC-de commented Sep 4, 2024

Thanks for conformation, fix will be included in next release planned for today or latest tomorrow.

@GIC-de GIC-de closed this as completed in b427699 Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed ipoe IPoE
Projects
None yet
Development

No branches or pull requests

2 participants