Update Travis e2e environment

Use latest and greated Kubernetes and minikube release.
Now that Travis offer systemd-enabled distributions, we
can get rid of the `--bootstrapper=localkube` minikube hack.

Also, better error message when e2e tests fails due to
missing katafygio in the $PATH.

Author: bpineau

.travis.yml

@@ -2,14 +2,15 @@ language: go
 go:
   - "1.12.x"
 
-# only needed for e2e tests
+# needed for e2e tests (so we can start minikube)
 sudo: required
+dist: xenial # We need the systemd for the kubeadm and it's default from 16.04+
 
 env:
   global:
     # only needed for e2e tests
-    - K8S_VERSION=v1.10.0
-    - MINIKUBE_VERSION=v0.26.1
+    - K8S_VERSION=v1.15.0
+    - MINIKUBE_VERSION=v1.2.0
     - MINIKUBE_WANTREPORTERRORPROMPT=false
     - MINIKUBE_WANTUPDATENOTIFICATION=false
     - CHANGE_MINIKUBE_NONE_USER=true
@@ -22,8 +23,7 @@ before_script:
   - sudo curl -Lo /usr/local/bin/minikube https://github.com/kubernetes/minikube/releases/download/${MINIKUBE_VERSION}/minikube-linux-amd64
   - sudo chmod +x /usr/local/bin/kubectl /usr/local/bin/minikube
   - mkdir -p $HOME/.kube && touch $HOME/.kube/config
-  # remove the --bootstrapper flag once this issue is solved: https://github.com/kubernetes/minikube/issues/2704
-  - sudo -E /usr/local/bin/minikube start --vm-driver=none --bootstrapper=localkube --kubernetes-version=${K8S_VERSION}
+  - sudo -E /usr/local/bin/minikube start --vm-driver=none --kubernetes-version=${K8S_VERSION}
   # Fix the kubectl context, as it is often stale.
   - /usr/local/bin/minikube update-context
   # Wait for Kubernetes to be up and ready.

README.md

@@ -107,7 +107,7 @@ brew install bpineau/tap/katafygio
 
 You can also deploy with the provided [helm](https://helm.sh/) chart:
 ```shell
-helm install --name kf-backups --namespace kube-system assets/helm-chart/katafygio/
+helm install --name kf-backups assets/helm-chart/katafygio/
 ```
 
 ## See Also

assets/e2e_test.go

@@ -55,13 +55,21 @@ func TestE2E(t *testing.T) {
 
 func TestMain(m *testing.M) {
 	deleteResources()
-	_ = exec.Command("rm", "-rf", dumpPath)
+	err := exec.Command("rm", "-rf", dumpPath).Run()
+	if err != nil {
+		fmt.Printf("failed to clean test dump path %s: %v", dumpPath, err)
+		os.Exit(1)
+	}
 
 	ctx, cancel := context.WithCancel(context.Background())
 
 	go func() {
 		cmd := exec.CommandContext(ctx, "katafygio", "-e", dumpPath)
-		_ = cmd.Run()
+		err := cmd.Run()
+		if err != nil && err.Error() != "signal: killed" {
+			fmt.Printf("failed to spawn katafygio: %s", err.Error())
+			os.Exit(1)
+		}
 	}()
 
 	ret := m.Run()