Merge pull request #113209 from luozhiwenn/personal/etc-host

ensure etc-host file permission is 644 whatever umask is
2023-05-16 01:41:35 -07:00
parent a31899a6fd 76c8765bda
commit 03b2e84183
1 changed files with 5 additions and 1 deletions
--- a/pkg/kubelet/kubelet_pods.go
+++ b/pkg/kubelet/kubelet_pods.go
@@ -344,7 +344,11 @@ func ensureHostsFile(fileName string, hostIPs []string, hostName, hostDomainName
 		hostsFileContent = managedHostsFileContent(hostIPs, hostName, hostDomainName, hostAliases)
 	}

-	return os.WriteFile(fileName, hostsFileContent, 0644)
+	hostsFilePerm := os.FileMode(0644)
+	if err := os.WriteFile(fileName, hostsFileContent, hostsFilePerm); err != nil {
+		return err
+	}
+	return os.Chmod(fileName, hostsFilePerm)
 }

 // nodeHostsFileContent reads the content of node's hosts file.