-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
DHCP_am: support multiple nameservers #4834
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
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #4834 +/- ##
==========================================
- Coverage 80.88% 80.50% -0.38%
==========================================
Files 368 368
Lines 90231 90236 +5
==========================================
- Hits 72980 72643 -337
- Misses 17251 17593 +342
🚀 New features to boost your workflow:
|
490c16f
to
4817a16
Compare
4817a16
to
7711b1f
Compare
result | ||
|
||
# 3 results depending on the Python version | ||
assert result in [ |
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.
Looks like it fails on Fedora Rawhide (with Python 3.14.0rc2):
###(006)=[failed] Check that the dhcpd alias is properly defined and documented
>>> assert dhcpd
>>> import IPython
>>>
>>> result = IPython.lib.pretty.pretty(dhcpd)
>>> result
'<function scapy.ansmachine.dhcpd(self, pool: scapy.base_classes.Net | List[str] = Net("192.168.1.128/25"), network: str = \'192.168.1.0/24\', gw: str = \'192.168.1.1\', nameserver: str | List[str] = None, domain: str | None = None, renewal_time: int = 60, lease_time: int = 1800, **kwargs)>'
>>>
>>> assert result in [
... '<function scapy.ansmachine.dhcpd(self, pool: Union[scapy.base_classes.Net, List[str]] = Net("192.168.1.128/25"), network: str = \'192.168.1.0/24\', gw: str = \'192.168.1.1\', nameserver: Union[str, List[str]] = None, domain: Optional[str] = None, renewal_time: int = 60, lease_time: int = 1800, **kwargs)>',
... '<function scapy.ansmachine.dhcpd(self, pool: Union[scapy.base_classes.Net, List[str]] = Net("192.168.1.128/25"), network: str = \'192.168.1.0/24\', gw: str = \'192.168.1.1\', nameserver: Union[str, List[str]] = None, domain: Union[str, NoneType] = None, renewal_time: int = 60, lease_time: int = 1800, **kwargs)>',
... '<function scapy.ansmachine.dhcpd(self, pool=Net("192.168.1.128/25"), network=\'192.168.1.0/24\', gw=\'192.168.1.1\', nameserver=None, domain=None, renewal_time=60, lease_time=1800, **kwargs)>',
... ]
AssertionError
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.
Thanks ! See befe516
This varies a LOT depending on the Python version hehe ^^
closes #4638
fixes #4431