Merge pull request #44966 from a-robinson/insecure

Automatic merge from submit-queue Fix cockroachdb statefulset test read/write commands Explicitly specifying `--insecure` is required on insecure clusters, which started being enforced in a very recent release. In 2 weeks we'll have a stable image version that we can reliably pin the relevant statefulset yaml file to in order to avoid stupid failures like this. I'm really sorry for the flakes! **What this PR does / why we need it**: It fixes the currently broken statefulset test suite - https://storage.googleapis.com/k8s-gubernator/triage/index.html?job=gci-gce-statefulset&test=CockroachDB **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: N/A **Special notes for your reviewer**: N/A **Release note**: ```release-note NONE ``` @kow3ns

Merge pull request #44966 from a-robinson/insecure
c2595909 · Kubernetes Submit Queue · GitHub · 963e0565 · 4af0e640 · c2595909
Commit c2595909 authored Apr 27, 2017 by Kubernetes Submit Queue Committed by GitHub Apr 27, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

statefulset.go test/e2e/statefulset.go +1 -1

No files found.
--- a/test/e2e/statefulset.go
+++ b/test/e2e/statefulset.go
@@ -720,7 +720,7 @@ func (c *cockroachDBTester) name() string {
 }

 func (c *cockroachDBTester) cockroachDBExec(cmd, ns, podName string) string {
-	cmd = fmt.Sprintf("/cockroach/cockroach sql --host %s.cockroachdb -e \"%v\"", podName, cmd)
+	cmd = fmt.Sprintf("/cockroach/cockroach sql --insecure --host %s.cockroachdb -e \"%v\"", podName, cmd)
 	return framework.RunKubectlOrDie(fmt.Sprintf("--namespace=%v", ns), "exec", podName, "--", "/bin/sh", "-c", cmd)
 }