insomniacslk / dhcp

DHCPv6 and DHCPv4 packet library, client and server written in Go
BSD 3-Clause "New" or "Revised" License
685 stars 168 forks source link

Add MLNX-OS ZTPv6 vendor data parsing #503

Closed gepoggio closed 1 year ago

gepoggio commented 1 year ago

MLNX-OS sends DHCP information spread in different vendor options sub-options: https://docs.nvidia.com/networking/display/MLNXOSv3103002/Getting+Started#heading-RunningDHCP-ZTP

Enterprise number 33049 is registered to Mellanox: https://www.iana.org/assignments/enterprise-numbers/?q=33049

Adding support to parse the VendorOptions and return the standard VendorData struct.

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.08 :tada:

Comparison is base (4980196) 72.77% compared to head (0a77c47) 72.86%.

:exclamation: Current head 0a77c47 differs from pull request most recent head cb5419b. Consider uploading reports for the commit cb5419b to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #503 +/- ## ========================================== + Coverage 72.77% 72.86% +0.08% ========================================== Files 90 90 Lines 5742 5761 +19 ========================================== + Hits 4179 4198 +19 Misses 1391 1391 Partials 172 172 ``` | [Impacted Files](https://app.codecov.io/gh/insomniacslk/dhcp/pull/503?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=insomniac) | Coverage Δ | | |---|---|---| | [dhcpv6/ztpv6/parse\_vendor\_options.go](https://app.codecov.io/gh/insomniacslk/dhcp/pull/503?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=insomniac#diff-ZGhjcHY2L3p0cHY2L3BhcnNlX3ZlbmRvcl9vcHRpb25zLmdv) | `91.66% <100.00%> (+2.98%)` | :arrow_up: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.