Skip to content

Commit ccfaef2

Browse files
authored
Merge pull request #157 from adrianchiris/bump-rdma-cni
chore: bump rdma-cni to v1.5.0
2 parents 0581aa5 + 3f97c04 commit ccfaef2

File tree

3 files changed

+7
-7
lines changed

3 files changed

+7
-7
lines changed

go.mod

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ require (
66
github.com/containernetworking/cni v1.3.0
77
github.com/containernetworking/plugins v1.8.0
88
github.com/gofrs/flock v0.13.0
9-
github.com/k8snetworkplumbingwg/rdma-cni v1.4.0
9+
github.com/k8snetworkplumbingwg/rdma-cni v1.5.0
1010
github.com/k8snetworkplumbingwg/sriovnet v1.2.0
1111
github.com/onsi/ginkgo/v2 v2.27.2
1212
github.com/onsi/gomega v1.38.2

go.sum

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,8 @@ github.com/google/uuid v1.6.0 h1:NIvaJDMOsjHA8n1jAhLSgzrAzy1Hgr+hNrb57e+94F0=
3232
github.com/google/uuid v1.6.0/go.mod h1:TIyPZe4MgqvfeYDBFedMoGGpEw/LqOeaOT+nhxU+yHo=
3333
github.com/joshdk/go-junit v1.0.0 h1:S86cUKIdwBHWwA6xCmFlf3RTLfVXYQfvanM5Uh+K6GE=
3434
github.com/joshdk/go-junit v1.0.0/go.mod h1:TiiV0PqkaNfFXjEiyjWM3XXrhVyCa1K4Zfga6W52ung=
35-
github.com/k8snetworkplumbingwg/rdma-cni v1.4.0 h1:S4Dt8Oi8ndh6jRMfkhcIb6HXZUPgiqQJ0gmkaTGAZ2Q=
36-
github.com/k8snetworkplumbingwg/rdma-cni v1.4.0/go.mod h1:3LdblK1KkPEsXVSG4lD6iWTR4FhoJpjlW3u/gsz5Zuo=
35+
github.com/k8snetworkplumbingwg/rdma-cni v1.5.0 h1:sJm7/Smob34Hhffhz13ID7YPVH+N8flf8M/qFpNQJ8E=
36+
github.com/k8snetworkplumbingwg/rdma-cni v1.5.0/go.mod h1:om5VpWt5RG014xy48f/IsueLPJ+KWrXnpLSsa2nkWCg=
3737
github.com/k8snetworkplumbingwg/sriovnet v1.2.0 h1:6ELfAxCB1dvosGUy3DVRmfH+HWTzmPD3W67HKQvMR1M=
3838
github.com/k8snetworkplumbingwg/sriovnet v1.2.0/go.mod h1:jyWzGe6ZtYiPq6ih6aXCOy6mZ49Y9mNyBOLBBXnli+k=
3939
github.com/kr/pretty v0.3.1 h1:flRD4NNwYAUpkphVc1HcthR4KEIFJ65n8Mw5qdRn3LE=

pkg/utils/rdma.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -33,9 +33,9 @@ func MoveRdmaDevToNs(rdmaDev string, targetNs ns.NetNS) error {
3333

3434
// Move RDMA device to namespace
3535
func MoveRdmaDevToNsPci(pciDev string, targetNs ns.NetNS) (string, error) { // (hostRdmaDev, error)
36-
rdmaDevs, err := rdmaManager.GetRdmaDevsForPciDev(pciDev)
37-
if err != nil || len(rdmaDevs) == 0 {
38-
return "", fmt.Errorf("failed to get RDMA devices for PCI device: %s. %v", pciDev, err)
36+
rdmaDevs := rdmaManager.GetRdmaDevsForPciDev(pciDev)
37+
if len(rdmaDevs) == 0 {
38+
return "", fmt.Errorf("failed to get RDMA devices for PCI device: %s. No RDMA devices found", pciDev)
3939
}
4040

4141
if len(rdmaDevs) != 1 {
@@ -47,7 +47,7 @@ func MoveRdmaDevToNsPci(pciDev string, targetNs ns.NetNS) (string, error) { // (
4747
// Move RDMA device to container namespace
4848
rdmaDev := rdmaDevs[0]
4949

50-
err = MoveRdmaDevToNs(rdmaDev, targetNs)
50+
err := MoveRdmaDevToNs(rdmaDev, targetNs)
5151
if err != nil {
5252
return "", fmt.Errorf("failed to move RDMA device %s to namespace. %v", rdmaDev, err)
5353
}

0 commit comments

Comments
 (0)