-
-
Notifications
You must be signed in to change notification settings - Fork 110
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
DHCP update range issue #149
Comments
Hey! I'm assuming the gateway for this interface is x.x.x.193? If so, one difference between the API validation and the UI validation is the API will not allow the DHCP range to start with the gateway's address even though the UI does allow this. Doing so in the past had caused some issues with automated testing and to my understanding, including the gateway's address in the DHCP pool has no practical use case. I've tested this and bumping the start range to x.x.x.194 will allow the change from the API. If I'm mistaken and there is a legitimate use case for including the gateway's address in the DHCP pool I'm open to changing this. Again this is all under the assumption that the gateway is x.x.x.193, if that's not the case let me know and I can take another look. Thanks! |
Correct the gateway ip address is x.x.x.193 So ideally to not waste any ips, we would set the pool start and end to x.x.x.193 |
Gotcha, that is a fair use case and does seem to be the only way such a configuration can exist. I will work on a fix for this and have it included in a patch. Time permitting, I should be able to have that released next week. I appreciate the insights. Let me know if you have any questions in the meantime. Thanks again! |
Error Message: (2022)
Endpoint:
PUT /api/v1/services/dhcpd
Tested subnet details:
Subnet:
x.x.x.192
Mask:
255.255.255.192
Available range:
x.x.x.193 - x.x.x.254
When start range is set to x.x.x.193 and/or end range is set to x.x.x.193 it will cause the error while the UI have no issues.
The text was updated successfully, but these errors were encountered: