k8snetworkplumbingwg / multus-cni

A CNI meta-plugin for multi-homed pods in Kubernetes
Apache License 2.0
2.41k stars 584 forks source link

clean up go.mod, get rid of client-go v1.5.2 #1353

Closed xrstf closed 2 weeks ago

xrstf commented 2 weeks ago

client-go 1.5.2 has been retracted and having it here as a dependency forces not just Multus, but everyone who imports Multus to do the same replace dance with tons of Kube dependencies.

From what I can see, there is simply no need to replace tons of dependencies. So I cleaned it up and go mod tidy and go build are happy.

coveralls commented 2 weeks ago

Coverage Status

coverage: 56.454% (-7.4%) from 63.857% when pulling 808185b10f6855d494999ca32cd3b9045328f184 on xrstf:master into e1a0d2a3fd4f490861ff1872840fdf0a69200747 on k8snetworkplumbingwg:master.