k8snetworkplumbingwg / whereabouts

A CNI IPAM plugin that assigns IP addresses cluster-wide
Apache License 2.0
273 stars 120 forks source link

Use newer contianernetworking/cni 100 type #411

Closed manuelbuil closed 4 months ago

manuelbuil commented 4 months ago

What this PR does / why we need it:

Given that we are now using contianernetworking/cni v1.1.2, we can move to the latest 100 type. This PR moves to the newer type: "github.com/containernetworking/cni/pkg/types/100". The biggest change is the removal of the Version field. Components using this new type basically process the Address to verify the type.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged): Fixes #

Special notes for your reviewer (optional):

coveralls commented 4 months ago

Pull Request Test Coverage Report for Build 8018475037

Details


Totals Coverage Status
Change from base Build 8018096162: -0.1%
Covered Lines: 1133
Relevant Lines: 1570

💛 - Coveralls