github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2026-07-18 04:54:54 +00:00
Code Issues Projects Releases Wiki Activity

Merge pull request #131378 from cuishuang/master

Browse Source 
fix: Fix non-constant format string in framework.Logf call
This commit is contained in:
Kubernetes Prow Robot
2025-05-04 19:31:56 -07:00
committed by GitHub
parent 359f36dfe0 d10eb86927
commit c87ed899e6
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
test/e2e/node/mount_propagation.go
View File

@@ -197,11 +197,11 @@ var _ = SIGDescribe("Mount propagation", func() {
// Higher pids are more likely to be created after the first one
kubeletPids := strings.Split(strings.TrimSuffix(kubeletPid, "\n"), " ")
enterKubeletMountNS := fmt.Sprintf("nsenter -t %s -m", kubeletPids[len(kubeletPids)-1])
framework.Logf(enterKubeletMountNS)
// Check that the master and host mounts are propagated to the container runtime's mount namespace
for _, mountName := range []string{"host", master.Name} {
cmd := fmt.Sprintf("%s cat \"%s/%s/file\"", enterKubeletMountNS, hostDir, mountName)
framework.Logf("running command '%s' on node '%s'", cmd, node.Name)
output, err := hostExec.IssueCommandWithResult(ctx, cmd, node)
framework.ExpectNoError(err, "host container namespace should see mount from %s: %s", mountName, output)
output = strings.TrimSuffix(output, "\n")
@@ -211,6 +211,7 @@ var _ = SIGDescribe("Mount propagation", func() {
// Check that the slave, private, and default mounts are not propagated to the container runtime's mount namespace
for _, podName := range []string{slave.Name, private.Name, defaultPropagation.Name} {
cmd := fmt.Sprintf("%s test ! -e \"%s/%s/file\"", enterKubeletMountNS, hostDir, podName)
framework.Logf("running command '%s' on node '%s'", cmd, node.Name)
output, err := hostExec.IssueCommandWithResult(ctx, cmd, node)
framework.ExpectNoError(err, "host container namespace shouldn't see mount from %s: %s", podName, output)
}