mirror of
https://github.com/k3s-io/kubernetes.git
synced 2025-07-18 17:33:39 +00:00
Calling WaitForPodTerminatedInNamespace after testFlexVolume is useless because
the client pod that it waits for always gets deleted by testVolumeClient:
0fcc3dbd55/test/e2e/framework/volume/fixtures.go (L541-L546)
Worse, because WaitForPodTerminatedInNamespace treats "not found" as "must keep
polling", these two tests always kept waiting for 5 minutes:
Kubernetes e2e suite: [It] [sig-storage] Flexvolumes should be mountable
when non-attachable 6m4s
The only reason why these tests passed is that WaitForPodTerminatedInNamespace
used to return the "not found" API error. That is not guaranteed and about to
change.
223 lines
7.5 KiB
Go
223 lines
7.5 KiB
Go
/*
|
|
Copyright 2017 The Kubernetes Authors.
|
|
|
|
Licensed under the Apache License, Version 2.0 (the "License");
|
|
you may not use this file except in compliance with the License.
|
|
You may obtain a copy of the License at
|
|
|
|
http://www.apache.org/licenses/LICENSE-2.0
|
|
|
|
Unless required by applicable law or agreed to in writing, software
|
|
distributed under the License is distributed on an "AS IS" BASIS,
|
|
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
See the License for the specific language governing permissions and
|
|
limitations under the License.
|
|
*/
|
|
|
|
package storage
|
|
|
|
import (
|
|
"context"
|
|
"fmt"
|
|
"net"
|
|
"path"
|
|
|
|
"time"
|
|
|
|
"github.com/onsi/ginkgo/v2"
|
|
v1 "k8s.io/api/core/v1"
|
|
clientset "k8s.io/client-go/kubernetes"
|
|
"k8s.io/kubernetes/test/e2e/framework"
|
|
e2enode "k8s.io/kubernetes/test/e2e/framework/node"
|
|
e2epod "k8s.io/kubernetes/test/e2e/framework/pod"
|
|
e2eskipper "k8s.io/kubernetes/test/e2e/framework/skipper"
|
|
e2essh "k8s.io/kubernetes/test/e2e/framework/ssh"
|
|
e2etestfiles "k8s.io/kubernetes/test/e2e/framework/testfiles"
|
|
e2evolume "k8s.io/kubernetes/test/e2e/framework/volume"
|
|
"k8s.io/kubernetes/test/e2e/storage/utils"
|
|
admissionapi "k8s.io/pod-security-admission/api"
|
|
)
|
|
|
|
const (
|
|
driverDir = "test/e2e/testing-manifests/flexvolume/"
|
|
defaultVolumePluginDir = "/usr/libexec/kubernetes/kubelet-plugins/volume/exec"
|
|
// TODO: change this and config-test.sh when default flex volume install path is changed for GCI
|
|
// On gci, root is read-only and controller-manager containerized. Assume
|
|
// controller-manager has started with --flex-volume-plugin-dir equal to this
|
|
// (see cluster/gce/config-test.sh)
|
|
gciVolumePluginDir = "/home/kubernetes/flexvolume"
|
|
detachTimeout = 10 * time.Second
|
|
)
|
|
|
|
// testFlexVolume tests that a client pod using a given flexvolume driver
|
|
// successfully mounts it and runs
|
|
func testFlexVolume(ctx context.Context, driver string, config e2evolume.TestConfig, f *framework.Framework) {
|
|
tests := []e2evolume.Test{
|
|
{
|
|
Volume: v1.VolumeSource{
|
|
FlexVolume: &v1.FlexVolumeSource{
|
|
Driver: "k8s/" + driver,
|
|
},
|
|
},
|
|
File: "index.html",
|
|
// Must match content of examples/volumes/flexvolume/dummy(-attachable) domount
|
|
ExpectedContent: "Hello from flexvolume!",
|
|
},
|
|
}
|
|
e2evolume.TestVolumeClient(ctx, f, config, nil, "" /* fsType */, tests)
|
|
}
|
|
|
|
// installFlex installs the driver found at filePath on the node, and restarts
|
|
// kubelet if 'restart' is true. If node is nil, installs on the master, and restarts
|
|
// controller-manager if 'restart' is true.
|
|
func installFlex(ctx context.Context, c clientset.Interface, node *v1.Node, vendor, driver, filePath string) {
|
|
flexDir := getFlexDir(c, node, vendor, driver)
|
|
flexFile := path.Join(flexDir, driver)
|
|
|
|
host := ""
|
|
var err error
|
|
if node != nil {
|
|
host, err = e2enode.GetSSHExternalIP(node)
|
|
if err != nil {
|
|
host, err = e2enode.GetSSHInternalIP(node)
|
|
}
|
|
} else {
|
|
instanceWithPort := framework.APIAddress()
|
|
hostName := getHostFromHostPort(instanceWithPort)
|
|
host = net.JoinHostPort(hostName, e2essh.SSHPort)
|
|
}
|
|
|
|
framework.ExpectNoError(err)
|
|
|
|
cmd := fmt.Sprintf("sudo mkdir -p %s", flexDir)
|
|
sshAndLog(ctx, cmd, host, true /*failOnError*/)
|
|
|
|
data, err := e2etestfiles.Read(filePath)
|
|
if err != nil {
|
|
framework.Fail(err.Error())
|
|
}
|
|
cmd = fmt.Sprintf("sudo tee <<'EOF' %s\n%s\nEOF", flexFile, string(data))
|
|
sshAndLog(ctx, cmd, host, true /*failOnError*/)
|
|
|
|
cmd = fmt.Sprintf("sudo chmod +x %s", flexFile)
|
|
sshAndLog(ctx, cmd, host, true /*failOnError*/)
|
|
}
|
|
|
|
func uninstallFlex(ctx context.Context, c clientset.Interface, node *v1.Node, vendor, driver string) {
|
|
flexDir := getFlexDir(c, node, vendor, driver)
|
|
|
|
host := ""
|
|
var err error
|
|
if node != nil {
|
|
host, err = e2enode.GetSSHExternalIP(node)
|
|
if err != nil {
|
|
host, err = e2enode.GetSSHInternalIP(node)
|
|
}
|
|
} else {
|
|
instanceWithPort := framework.APIAddress()
|
|
hostName := getHostFromHostPort(instanceWithPort)
|
|
host = net.JoinHostPort(hostName, e2essh.SSHPort)
|
|
}
|
|
|
|
if host == "" {
|
|
framework.Failf("Error getting node ip : %v", err)
|
|
}
|
|
|
|
cmd := fmt.Sprintf("sudo rm -r %s", flexDir)
|
|
sshAndLog(ctx, cmd, host, false /*failOnError*/)
|
|
}
|
|
|
|
func getFlexDir(c clientset.Interface, node *v1.Node, vendor, driver string) string {
|
|
volumePluginDir := defaultVolumePluginDir
|
|
if framework.ProviderIs("gce") {
|
|
volumePluginDir = gciVolumePluginDir
|
|
}
|
|
flexDir := path.Join(volumePluginDir, fmt.Sprintf("/%s~%s/", vendor, driver))
|
|
return flexDir
|
|
}
|
|
|
|
func sshAndLog(ctx context.Context, cmd, host string, failOnError bool) {
|
|
result, err := e2essh.SSH(ctx, cmd, host, framework.TestContext.Provider)
|
|
e2essh.LogResult(result)
|
|
framework.ExpectNoError(err)
|
|
if result.Code != 0 && failOnError {
|
|
framework.Failf("%s returned non-zero, stderr: %s", cmd, result.Stderr)
|
|
}
|
|
}
|
|
|
|
func getHostFromHostPort(hostPort string) string {
|
|
// try to split host and port
|
|
var host string
|
|
var err error
|
|
if host, _, err = net.SplitHostPort(hostPort); err != nil {
|
|
// if SplitHostPort returns an error, the entire hostport is considered as host
|
|
host = hostPort
|
|
}
|
|
return host
|
|
}
|
|
|
|
var _ = utils.SIGDescribe("Flexvolumes", func() {
|
|
f := framework.NewDefaultFramework("flexvolume")
|
|
f.NamespacePodSecurityEnforceLevel = admissionapi.LevelBaseline
|
|
|
|
// note that namespace deletion is handled by delete-namespace flag
|
|
|
|
var cs clientset.Interface
|
|
var ns *v1.Namespace
|
|
var node *v1.Node
|
|
var config e2evolume.TestConfig
|
|
var suffix string
|
|
|
|
ginkgo.BeforeEach(func(ctx context.Context) {
|
|
e2eskipper.SkipUnlessProviderIs("gce", "local")
|
|
e2eskipper.SkipUnlessMasterOSDistroIs("debian", "ubuntu", "gci", "custom")
|
|
e2eskipper.SkipUnlessNodeOSDistroIs("debian", "ubuntu", "gci", "custom")
|
|
e2eskipper.SkipUnlessSSHKeyPresent()
|
|
|
|
cs = f.ClientSet
|
|
ns = f.Namespace
|
|
var err error
|
|
node, err = e2enode.GetRandomReadySchedulableNode(ctx, f.ClientSet)
|
|
framework.ExpectNoError(err)
|
|
config = e2evolume.TestConfig{
|
|
Namespace: ns.Name,
|
|
Prefix: "flex",
|
|
ClientNodeSelection: e2epod.NodeSelection{Name: node.Name},
|
|
}
|
|
suffix = ns.Name
|
|
})
|
|
|
|
ginkgo.It("should be mountable when non-attachable", func(ctx context.Context) {
|
|
driver := "dummy"
|
|
driverInstallAs := driver + "-" + suffix
|
|
|
|
ginkgo.By(fmt.Sprintf("installing flexvolume %s on node %s as %s", path.Join(driverDir, driver), node.Name, driverInstallAs))
|
|
installFlex(ctx, cs, node, "k8s", driverInstallAs, path.Join(driverDir, driver))
|
|
|
|
testFlexVolume(ctx, driverInstallAs, config, f)
|
|
|
|
ginkgo.By(fmt.Sprintf("uninstalling flexvolume %s from node %s", driverInstallAs, node.Name))
|
|
uninstallFlex(ctx, cs, node, "k8s", driverInstallAs)
|
|
})
|
|
|
|
ginkgo.It("should be mountable when attachable [Feature:Flexvolumes]", func(ctx context.Context) {
|
|
driver := "dummy-attachable"
|
|
driverInstallAs := driver + "-" + suffix
|
|
|
|
ginkgo.By(fmt.Sprintf("installing flexvolume %s on node %s as %s", path.Join(driverDir, driver), node.Name, driverInstallAs))
|
|
installFlex(ctx, cs, node, "k8s", driverInstallAs, path.Join(driverDir, driver))
|
|
ginkgo.By(fmt.Sprintf("installing flexvolume %s on master as %s", path.Join(driverDir, driver), driverInstallAs))
|
|
installFlex(ctx, cs, nil, "k8s", driverInstallAs, path.Join(driverDir, driver))
|
|
|
|
testFlexVolume(ctx, driverInstallAs, config, f)
|
|
|
|
// Detach might occur after pod deletion. Wait before deleting driver.
|
|
time.Sleep(detachTimeout)
|
|
|
|
ginkgo.By(fmt.Sprintf("uninstalling flexvolume %s from node %s", driverInstallAs, node.Name))
|
|
uninstallFlex(ctx, cs, node, "k8s", driverInstallAs)
|
|
ginkgo.By(fmt.Sprintf("uninstalling flexvolume %s from master", driverInstallAs))
|
|
uninstallFlex(ctx, cs, nil, "k8s", driverInstallAs)
|
|
})
|
|
})
|