search

k8snetworkplumbingwg / multus-cni

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

Update DRA integration info and fix e2e test #1313

Closed vasrem closed 4 months ago

vasrem commented 4 months ago

With Kubernetes 1.31, the DRA API will change (1,2). The dra example implementation has already been updated to some extent to reflect those changes. Current e2e tests fail because of that.

With this PR, we add additional info in the docs to reflect the versions of Kubernetes that multus supports DRA and also we fix the e2e test by pinning it to the original DRA example implementation.

Next step is to adjust the multus DRA integration to the new DRA API.

Note: The currently opened PRs will need to be rebased on top of master once this PR is merged to ensure that e2e test is passing.

coveralls commented 4 months ago

Coverage Status

coverage: 63.364% (+0.2%) from 63.116% when pulling 04fb8190fe611a7e2ebafc5384c5c734b61d13ee on vasrem:chore/update-dra-e2e into 85877341745ea2184456079a73ad771c27e2dc28 on k8snetworkplumbingwg:master.

dougbtv commented 4 months ago

Let's try to follow up next maintainer's meeting and see what needs to change for 1.31 compatibility, too, thanks for this fix!!