github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-08-19 00:31:00 +00:00
Code Issues Projects Releases Wiki Activity

Merge pull request #10035 from brendandburns/fix

Browse Source 
Update the addons test so it doesn't try to create clients on providers aren
This commit is contained in:
Alex Robinson 2015-06-29 11:45:09 -07:00
commit 99711263a1
1 changed files with 14 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
test/e2e/addon_update.go
View File

@ -196,6 +196,11 @@ var _ = Describe("Addon update", func() {
var namespace *api.Namespace
BeforeEach(func() {
// This test requires SSH, so the provider check should be identical to
// those tests.
if !providerIs("gce") {
return
}
var err error
c, err = loadClient()
Expect(err).NotTo(HaveOccurred())
@ -211,6 +216,11 @@ var _ = Describe("Addon update", func() {
})
AfterEach(func() {
// This test requires SSH, so the provider check should be identical to
// those tests.
if !providerIs("gce") {
return
}
if sshClient != nil {
// restart addon_update with the default options
sshExec(sshClient, "sudo /etc/init.d/kube-addons restart")
@ -226,14 +236,14 @@ var _ = Describe("Addon update", func() {
// WARNING: the test is not parallel-friendly!
It("should propagate add-on file changes", func() {
//these tests are long, so I squeezed several cases in one scenario
Expect(sshClient).NotTo(BeNil())
dir = namespace.Name // we use it only to give a unique string for each test execution
// This test requires SSH, so the provider check should be identical to
// those tests.
SkipUnlessProviderIs("gce")
//these tests are long, so I squeezed several cases in one scenario
Expect(sshClient).NotTo(BeNil())
dir = namespace.Name // we use it only to give a unique string for each test execution
temporaryRemotePathPrefix := "addon-test-dir"
temporaryRemotePath := temporaryRemotePathPrefix + "/" + dir // in home directory on kubernetes-master
defer sshExec(sshClient, fmt.Sprintf("rm -rf %s", temporaryRemotePathPrefix)) // ignore the result in cleanup