k8snetworkplumbingwg / multus-cni

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

remove vendoring, use Go modules #1287

Closed jaypipes closed 2 weeks ago

jaypipes commented 6 months ago

Removes the vendor/ directory and all vestiges of life before Go modules.

Issue k8snetworkplumbingwg/sriov-network-device-plugin#544

jaypipes commented 6 months ago

@dougbtv @s1061123 Hi! Any chance to look at this PR and/or run it through the gate tests please? Thanks much in advance!

dougbtv commented 5 months ago

Thanks for the proposed change, I'm going to bring it to the NPWG main meeting and discuss, feel free to join, I added to the agenda for next week @ https://docs.google.com/document/d/1oE93V3SgOGWJ4O1zeD1UmpeToa0ZiiO6LqRAmZBPFWM/edit

jaypipes commented 5 months ago

@dougbtv if you could approve the running of the workflows when you get a chance, that would be groovy, thanks! :)

adrianchiris commented 3 months ago

+1 on removing vendoring. we already did that did that for a bunch of projects in ngwg (sriov dp, cni, operator)

github-actions[bot] commented 3 weeks ago

This pull request is stale because it has been open 90 days with no activity. Remove stale label or comment or this will be closed in 7 days.