Unverified Commit c2848da1 authored by Kubernetes Submit Queue's avatar Kubernetes Submit Queue Committed by GitHub

Merge pull request #64556 from ingvagabund/e2e_node-containerized-make-kubeletConfigPath-rw

Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Mount the kubeletConfigPath rw when running containerized node e2e tests The kubelet needs to create dynamic-kubelet-config directory under the kubeletConfigPath when initialing dynamic config directory. **What this PR does / why we need it**: Without the patch the https://k8s-testgrid.appspot.com/sig-node-kubelet#kubelet-containerized-conformance-aws-e2e-rhel fails. **Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*: Fixes # **Special notes for your reviewer**: Started to occur after merging https://github.com/kubernetes/kubernetes/pull/64275 **Release note**: ```release-note NONE ```
parents 74b224eb ee830211
...@@ -219,7 +219,7 @@ func (e *E2EServices) startKubelet() (*server, error) { ...@@ -219,7 +219,7 @@ func (e *E2EServices) startKubelet() (*server, error) {
// if we will generate a kubelet config file, we need to mount that path into the container too // if we will generate a kubelet config file, we need to mount that path into the container too
if genKubeletConfigFile { if genKubeletConfigFile {
cmdArgs = append(cmdArgs, "-v", filepath.Dir(kubeletConfigPath)+":"+filepath.Dir(kubeletConfigPath)+":ro") cmdArgs = append(cmdArgs, "-v", filepath.Dir(kubeletConfigPath)+":"+filepath.Dir(kubeletConfigPath)+":rw")
} }
cmdArgs = append(cmdArgs, hyperkubeImage, "/hyperkube", "kubelet", "--containerized") cmdArgs = append(cmdArgs, hyperkubeImage, "/hyperkube", "kubelet", "--containerized")
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment