Commit Graph

160 Commits (034046e9319067a37a97780cacde3dbd8bd5cf6c)

Author SHA1 Message Date
priyawadhwa 034046e931 Add integration test to make sure minikube starts and stops with cont… (#3394)
* Add integration test to make sure minikube starts and stops with containerd
* Skip testing containerd if using none driver
2018-12-05 10:23:36 -08:00
Thomas Stromberg a402c11057 Remove check for obsolete kube-dns label & validate kubernetes.default rather than localhost 2018-12-04 12:46:59 -08:00
Balint Pato 47579273d8
even longer timeout + timeout error is retriable now (#3370)
* even longer timeout + timeout error is retriable now
* fix for debugging when tunnel test has issues
2018-11-27 13:58:26 -08:00
Thomas Strömberg e0df20f294 Restrict TestDocker commands to 10 minutes of total runtime. (#3350)
* Restrict TestDocker commands to 10 minutes of total runtime.
* Hide unneccesarily scary delete error
2018-11-27 08:24:28 -08:00
Balint Pato 4b11d63da1
debugging improvements and flakiness reduction (#3363)
Small improvements to debugging and reducing integration test flakiness: 
* added logging to kubeadm init in the SSHExecutor
* increasing timeout for RBAC creation via the kubernetes client set 
* only log tunnel output to test logger
2018-11-26 10:18:28 -08:00
Balint Pato a681603daa
Fix `minikube tunnel` for hyperkit (#3349)
This change is a bit of a hack to make IP routing work over the virtio-net interface - it forces the virtio-net interface (bridge100) created by hyperkit/xhyve to reset via removing and readding the single member of the interface. Without this "reset" packets are just simply lost most of the time, despite a completely healthy IP route.

This error was also the reason for a large percentage of our integration test failures. While reducing the flakiness of tunnel related tests I also increased the timeout requirement and adding retries to the nginx request in TestTunnel.
2018-11-21 13:39:48 -08:00
Thomas Stromberg f0b358f2b4 Remove extra log statements 2018-11-13 12:42:00 -08:00
Thomas Stromberg 8771eae0ae testDashboard: add timeout around reading URL from stdout.
This test typically takes <30s to execute, so 180s should leave plenty
of margin for error.
2018-11-07 16:40:45 -08:00
Thomas Stromberg 1b254773ef Call t.Parallel() after t.Skip() to avoid confusion while debugging. 2018-11-07 16:40:21 -08:00
Thomas Stromberg c9abdf46a7 Skip testTunnel on non-Windows platforms if route requires sudo password. 2018-11-07 16:39:28 -08:00
Thomas Strömberg 2ec03ebf61
Merge pull request #3260 from tstromberg/disable-mount-osx
Disable mount tests in Darwin to workaround panic: timeout bug
2018-10-19 17:03:08 -07:00
Thomas Strömberg 2621bb3a27
Merge pull request #3187 from tstromberg/consistent_err_formatting
Globally replace %s format directives with %v for error strings
2018-10-19 16:44:46 -07:00
Thomas Strömberg 43cdeb9242
Merge pull request #3219 from tstromberg/docker-env
Fix broken TestFunctional/EnvVars test when run under non-bash shell.
2018-10-19 16:19:54 -07:00
Thomas Stromberg 88ab68305b Add runtime import for Darwin check. 2018-10-19 16:16:32 -07:00
Thomas Stromberg 1f968bc893 More merge conflicts. 2018-10-19 14:20:17 -07:00
Thomas Stromberg 13e580b588 Fix merge breakage. 2018-10-19 14:16:03 -07:00
Thomas Stromberg 72e596507a Merge branch 'master' into consistent_err_formatting 2018-10-19 14:08:35 -07:00
Balint Pato ae9f4b20c7
minikube tunnel (#3015)
This commit introduces a new command, `minikube tunnel`, a LoadBalancer emulator functionality, that must be run with root permissions.

This command:

* Establishes networking routes from the host into the VM for all IP ranges used by Kubernetes.
* Enables a cluster controller that allocates IPs to services external `LoadBalancer` IPs.
* Cleans up routes and IPs when stopped (Ctrl+C), when `minikube` stops, and when `minikube tunnel` is ran with the `--cleanup` flag
2018-10-18 11:01:47 -07:00
Thomas Stromberg f20bf4d3b2 Disable mount tests in Darwin to workaround panic: timedout bug
These tests reliably cause a "panic: timed out" condition due to
hitherto unknown reasons. See issue #3200 for context.

dlorenc mentioned that this test never worked, so when I re-enabled it
recently for all platforms in af61bf790c,
it introduced a persistent test failure for darwin.

I'll leave issue #3200 open until we are able to re-enable this test.
2018-10-17 14:58:40 -07:00
Thomas Stromberg 07015bf43c Improve wording. 2018-10-05 14:02:04 -07:00
Thomas Stromberg 3d3926fd73 Remove scary looking log messages that distract from real failures. 2018-10-05 14:01:26 -07:00
Thomas Stromberg 633bd80dcb Merge branch 'master' into docker-env 2018-10-05 11:59:19 -07:00
Thomas Stromberg dc6d1bab8d Merge branch 'master' into dashboard_on_demand 2018-10-04 13:38:49 -07:00
Fernando Diaz 2ebdf5e1eb Enhance the Ingress Controller Addon (#3099)
- Updates Ingress-Controller Version to 0.19.0
- Adds Service Account for Ingress-Controller
- Adds Support for Prometheus
- Fixes bug with TCP/UDP ConfigMaps not Loading
- Adds more resource limits to default-backend
- Use new ingress class name
- Use app.kubernetes.io/xxxxxxxxxxx labels
2018-10-04 16:13:53 -04:00
Thomas Stromberg 534325e88f Use %v for errors instead of %s 2018-10-03 21:57:17 -07:00
Thomas Stromberg 57054acfb6 Add missing reference to cmd. 2018-10-03 15:12:40 -07:00
Thomas Stromberg c079bb1934 Update integration tests to handle newly persistant dashboard. 2018-10-03 15:06:30 -07:00
Thomas Stromberg df54c6a5b4 Use "kubectl proxy" instead of a NodePort to expose the dashboard.
This provides an additional level of security, by enforcing host checking, applying port randomization, and requiring explicit user intent to expose the service to the host.
2018-10-02 22:25:45 -07:00
Thomas Strömberg 4b1ee9cffb
Merge branch 'master' into directives 2018-09-28 19:20:49 -07:00
Thomas Stromberg 5ea3ef88ec Revert "Remove lint issues from integration tests, mostly by adding error"
This reverts commit af61bf790c.
2018-09-28 17:00:55 -07:00
Thomas Stromberg 3706b3b8ac Fix dangling format issues. 2018-09-28 16:48:37 -07:00
Thomas Stromberg 8540e3a038 Consistently use %v for formatting error messages. 2018-09-28 16:18:22 -07:00
Thomas Stromberg 055c8002b1 Replace %s format directives with %v for errors.
Previously we were mixed between the two forms. This commit picks %v,
which is consistent with the Kubernetes code base. They both effectively
do the same thing in this case, though %v works with any object, and %s
only with string objects.
2018-09-28 16:05:27 -07:00
Thomas Stromberg 6d93bd5ffe Wait for k8s-app=kube-dns pods to be Running before testing.
This effectively reverts 0a5efe156c - whose behavior was apparently correct. Both coredns and kubedns share k8s-app=kube-dns labels in our current environment.
2018-09-28 15:08:33 -07:00
Thomas Stromberg af61bf790c Remove lint issues from integration tests, mostly by adding error
handlers.
2018-09-28 11:57:48 -07:00
Thomas Stromberg 349ea5a06a Fix broken TestFunctional/EnvVars test when run under non-bash shell.
Make future failures to this test easier to debug. Here's an example of
the test failure I ran into:

    --- FAIL: TestFunctional/EnvVars (0.58s)
        cluster_env_test.go:36: SetEnvFromEnvCmdOutput: Error: No variables were parsed from docker-env output:
            set -gx DOCKER_TLS_VERIFY "1";
            set -gx DOCKER_HOST "tcp://192.168.39.199:2376";
            set -gx DOCKER_CERT_PATH "/usr/local/google/home/tstromberg/.minikube/certs";
            set -gx DOCKER_API_VERSION "1.35";
            # Run this command to configure your shell:
            # eval (minikube docker-env)
2018-09-26 17:53:42 -07:00
Thomas Stromberg 45379c61c2 Fix T.Log call containing a formatting directive. 2018-09-26 15:38:18 -07:00
dlorenc 86f44ad4ff Manual changes for go-containerregistry update. 2018-08-22 20:24:46 -05:00
dlorenc 20b4020019 Also disable persistence test. 2018-08-21 09:12:40 -07:00
dlorenc 0a5efe156c Don't look for DNS by name. It might be coredns or kubedns.
Signed-off-by: dlorenc <dlorenc@google.com>
2018-08-21 09:12:40 -07:00
dlorenc 232080ae0c Remove some more vendored k8s. 2018-07-26 08:23:21 -07:00
Aaron Prindle a4606583e4 remove all localkube references from code, change /var/lib/localkube to /var/lib/minikube 2018-07-24 12:36:42 -07:00
dlorenc 80e934b845 Fix e2e test compilation.
Signed-off-by: dlorenc <dlorenc@google.com>
2018-06-13 14:13:38 -07:00
dlorenc a2218d2e7e Add two more kubeadm preflight ignores to make e2e tests pass. 2018-06-12 13:48:29 -07:00
dlorenc 6b4c545a0f Deflake the dashboard test.
This was a bit overkill.
2018-06-08 10:22:43 -07:00
kairen d7c90bc3e5 Fix integration test for Ingress 2018-05-08 11:27:26 +02:00
Yongkun Anfernee Gui c07258b45a Fix more permissions 2018-02-09 14:32:53 -08:00
kairen 08f4677511 Change kubernetes-dashboard from RC to Deployment 2018-01-10 02:15:07 +08:00
dlorenc 030a4c747f Manual changes for 1.9 update. 2017-12-21 14:15:48 -08:00
kairen 797925f183 Update ingress controller, and fix integration test 2017-12-12 21:43:39 -08:00