Skip to content

Commit b8361f9

Browse files
committed
Revert "pkg/kubelet: Fix userns tests on Windows"
This reverts commit 74b069d. Now that pkg/kubelet/userns has a windows stub, there is no need to handle this here. Let's remove this code so if it's ever called from windows, we find an error and the bug surfaces immediately. I've added it yesterday as an abundance of caution, to make sure the tests go back to green in the master branch. But now that everything is green again, it should be safe to remove it. Signed-off-by: Rodrigo Campos <[email protected]>
1 parent 8d71f74 commit b8361f9

File tree

1 file changed

+0
-5
lines changed

1 file changed

+0
-5
lines changed

pkg/kubelet/kubelet_pods.go

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -131,11 +131,6 @@ func (kl *Kubelet) getKubeletMappings() (uint32, uint32, error) {
131131
return defaultFirstID, defaultLen, nil
132132
}
133133

134-
// Windows doesn't support user namespaces, let's return the default mappings.
135-
if runtime.GOOS == "windows" {
136-
return defaultFirstID, defaultLen, nil
137-
}
138-
139134
_, err := user.Lookup(kubeletUser)
140135
if err != nil {
141136
var unknownUserErr user.UnknownUserError

0 commit comments

Comments
 (0)