github/multus-cni
1
0
Fork 1
mirror of https://github.com/k8snetworkplumbingwg/multus-cni.git synced 2026-08-09 09:05:34 +00:00
Code Issues Projects Releases Wiki Activity

fix(test): use unique temp file path in pkg/server tests

Browse Source 
Both pkg/multus and pkg/server tests used /tmp/foo.multus.conf as the
readiness indicator file. Since go test ./... runs packages in parallel,
pkg/multus AfterEach would delete the file while pkg/server STATUS/GC
tests were polling for it, causing intermittent failures.

Rename to /tmp/foo.multus.server-test.conf to avoid the collision.
This commit is contained in:
Basavaraju-G
2026-06-28 14:18:52 +00:00
parent db30741512
commit f80583bddc
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
pkg/server/thick_cni_test.go
View File

@@ -92,7 +92,7 @@ var _ = Describe(suiteName, func() {
})
Context("STATUS and GC commands without pod context", func() {
const configPath = "/tmp/foo.multus.conf"
const configPath = "/tmp/foo.multus.server-test.conf"
var (
cniServer *Server
@@ -160,7 +160,7 @@ var _ = Describe(suiteName, func() {
containerID = "123456789"
ifaceName = "eth0"
podName = "my-little-pod"
configPath = "/tmp/foo.multus.conf"
configPath = "/tmp/foo.multus.server-test.conf"
)
var (
@@ -222,7 +222,7 @@ var _ = Describe(suiteName, func() {
containerID = "123456789"
ifaceName = "eth0"
podName = "my-little-pod"
configPath = "/tmp/foo.multus.conf"
configPath = "/tmp/foo.multus.server-test.conf"
)
var (
@@ -366,7 +366,7 @@ func referenceConfig(thickPluginSocketDir string) string {
"name": "node-cni-network",
"type": "multus",
"daemonSocketDir": "%s",
"readinessindicatorfile": "/tmp/foo.multus.conf",
"readinessindicatorfile": "/tmp/foo.multus.server-test.conf",
"defaultnetworkwaitseconds": 3,
"delegates": [{
"name": "weave1",