Commit Graph

30222 Commits

Author SHA1 Message Date
k8s-merge-robot
6a388d4a0d Merge pull request #27072 from colhom/federation-e2e-docs
Automatic merge from submit-queue

Updating e2e docs with instructions on running federation tests

Last two commits are for review. Depends on #26951

\cc @madhusudancs @quinton-hoole @nikhiljindal 
 
[![Analytics](https://kubernetes-site.appspot.com/UA-36037335-10/GitHub/.github/PULL_REQUEST_TEMPLATE.md?pixel)]()
2016-06-09 19:14:11 -07:00
k8s-merge-robot
7c4c19f623 Merge pull request #26936 from yifan-gu/fix_selinux
Automatic merge from submit-queue

rkt: Fix incomplete selinux context string when the option is partial.

Fix "EmptyDir" e2e tests failures caused by #https://github.com/kubernetes/kubernetes/pull/24901

As mentioned in https://github.com/kubernetes/kubernetes/pull/24901#discussion_r61372312
We should apply the selinux context of the rkt data directory (/var/lib/rkt) when users do not specify all the selinux options.

Due to my fault, the change was missed during rebase, thus caused the regression.

After applying this PR, the e2e tests passed.
```
$ go run hack/e2e.go -v -test --test_args="--ginkgo.dryRun=false --ginkgo.focus=EmptyDir"
...
Ran 19 of 313 Specs in 199.319 seconds
SUCCESS! -- 19 Passed | 0 Failed | 0 Pending | 294 Skipped PASS
```

BTW, the test is removed because the `--no-overlay=true` flag will only be there on non-coreos distro.

cc @euank @kubernetes/sig-node
2016-06-09 19:14:08 -07:00
k8s-merge-robot
8c517e0aa7 Merge pull request #27135 from jlowdermilk/fix-upgrade-tests
Automatic merge from submit-queue

Fix GKE upgrade e2e util.

containers command group at HEAD no longer accepts --zone. Flag
has to be specified after subcommand group. Fix #27011
2016-06-09 18:32:53 -07:00
k8s-merge-robot
adc55115a0 Merge pull request #27125 from wojtek-t/protobufs_in_nettest
Automatic merge from submit-queue

Change webserver binary to be usable in large clusters.

Ref #27039

@zmerlynn
2016-06-09 18:32:50 -07:00
k8s-merge-robot
9ae6a008c9 Merge pull request #26952 from madhusudancs/fed-apiserver-admissioncontrol-todo
Automatic merge from submit-queue

Add a TODO to enable admission control in federation apiserver when the support is added.

Only the last commit here needs review.

Depends on #26951.

cc @kubernetes/sig-cluster-federation

[![Analytics](https://kubernetes-site.appspot.com/UA-36037335-10/GitHub/.github/PULL_REQUEST_TEMPLATE.md?pixel)]()
2016-06-09 18:32:46 -07:00
k8s-merge-robot
005011227e Merge pull request #26904 from madhusudancs/federation-apiserver-protocol
Automatic merge from submit-queue

Prefix scheme/protocol to federation API server URL.

cc @kubernetes/sig-cluster-federation 

[![Analytics](https://kubernetes-site.appspot.com/UA-36037335-10/GitHub/.github/PULL_REQUEST_TEMPLATE.md?pixel)]()
2016-06-09 18:32:42 -07:00
k8s-merge-robot
31e0be6048 Merge pull request #26820 from lavalamp/fix-docs2
Automatic merge from submit-queue

Stop checking in generated documents

Part of #26205

Auto generated docs are **NO LONGER CHECKED IN**, only placeholders.

To generate them, e.g. before exporting docs, run hack/generate-docs.sh.

hack/verify-generated-docs.sh ensures that generated docs are merely the
placeholder text.

hack/update-generated-docs.sh puts the placeholder text in the proper
places.

The old munge behavior is moved into hack/{update|verify}-munge-docs.sh.
2016-06-09 18:32:39 -07:00
k8s-merge-robot
cbde2ec8c2 Merge pull request #25987 from hpcloud/openstack-lbaas-v2
Automatic merge from submit-queue

LBaaS v2 Support for Openstack Cloud Provider Plugin

Resolves #19774.

This work is based on Gophercloud support for LBaaS v2 currently in review (this will have to merge first):
https://github.com/rackspace/gophercloud/pull/575

These changes includes the addition of a new loadbalancer configuration option:  **LBVersion**.  If this configuration attribute is missing or anything other than "v2", lbaas v1 implementation will be used.
2016-06-09 18:32:35 -07:00
k8s-merge-robot
e12c93feb9 Merge pull request #25737 from soltysh/scheduledjob_test
Automatic merge from submit-queue

ScheduledJob tests

This builds on top of #25569, so only the last two commits matter ([Revert commit a31ca0d and move batch/v2alpha1 tests to separate group](e7f6ba796e) and [ScheduledJob client and storage tests](c59c045a8c)). This also supersedes #25566 with simpler approach to testing, by testing single version at a time. 
@caesarxuchao I've reverted the changes I've did previously to enable multi-version tests and introduced new group where `batch/v2alpha1` is tested (see 1st commit). I've also added back the necessary tests for scheduledjob related stuff (2nd commit) that are already in queue (@erictune fyi - I can't stand having code without tests 😉).

[![Analytics](https://kubernetes-site.appspot.com/UA-36037335-10/GitHub/.github/PULL_REQUEST_TEMPLATE.md?pixel)]()
2016-06-09 18:32:31 -07:00
k8s-merge-robot
9b3a8a82f7 Merge pull request #21424 from aledbf/redis-alpine
Automatic merge from submit-queue

Update redis example to use alpine:3.4 as image

```
kubernetes/redis    v2                  40e584663e63        5 minutes ago       58.25 MB
kubernetes/redis    v1                  de309495e6c7        13 months ago       146 MB
```

<!-- Reviewable:start -->
---
This change is [<img src="http://reviewable.k8s.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](http://reviewable.k8s.io/reviews/kubernetes/kubernetes/21424)
<!-- Reviewable:end -->
2016-06-09 17:16:21 -07:00
Maciej Szulik
2ab885a53a Revert commit a31ca0dc98 and move batch/v2alpha1 tests to separate group in KUBE_TEST_API_VERSIONS 2016-06-10 01:11:56 +02:00
Colin Hom
5f7c7ad195 document federation e2e cli flow 2016-06-09 15:11:52 -07:00
Dawn Chen
c88c9584a9 Merge pull request #27147 from kubernetes/revert-27027-gci-network
Revert "GCI: add support for network plugin"
2016-06-09 13:39:57 -07:00
Dawn Chen
8207eddd99 Revert "GCI: add support for network plugin" 2016-06-09 13:24:05 -07:00
Jeff Lowdermilk
3567199432 Fix GKE upgrade e2e util.
containers command group at HEAD no longer accepts --zone. Flag
has to be specified after subcommand group.
2016-06-09 10:47:12 -07:00
Wojciech Tyczynski
12d1cc454c Change webserver binary to be usable in large clusters. 2016-06-09 16:01:01 +02:00
k8s-merge-robot
c9c4ada309 Merge pull request #26615 from jsafrane/gce-attach-tests
Automatic merge from submit-queue

GCE attach tests

Add basic tests for GCE attacher.

Looking at the code, it would deserve some refactoring as suggested in #25888, so mounting is not tested at all.
2016-06-09 06:00:56 -07:00
k8s-merge-robot
dd345fbf89 Merge pull request #26494 from caesarxuchao/enable-watchCache-integration-tests
Automatic merge from submit-queue

Enable WatchCache in test/integration/ tests

We already run cmd/integration/ with watch cache on. We should also run tests in test/integration/ with watch cache on.

@wojtek-t @lavalamp
2016-06-09 04:55:47 -07:00
k8s-merge-robot
c80f650b70 Merge pull request #26188 from brendandburns/test_main
Automatic merge from submit-queue

Add a custom main instead of the standard test main, to reduce stack …

Adds a custom test main handler (see: `TestMain` in https://golang.org/pkg/testing/ for details)

Partial fix for https://github.com/kubernetes/kubernetes/issues/25965

This does the standard timeout, but strips non-kubernetes stacks out of the stack trace (e.g. it filters things like:

```
goroutine 466 [IO wait, 7 minutes]:
net.runtime_pollWait(0x7fd74c4672c0, 0x72, 0xc821614000)
	/usr/local/go/src/runtime/netpoll.go:160 +0x60
net.(*pollDesc).Wait(0xc8215c21b0, 0x72, 0x0, 0x0)
	/usr/local/go/src/net/fd_poll_runtime.go:73 +0x3a
net.(*pollDesc).WaitRead(0xc8215c21b0, 0x0, 0x0)
	/usr/local/go/src/net/fd_poll_runtime.go:78 +0x36
net.(*netFD).Read(0xc8215c2150, 0xc821614000, 0x1000, 0x1000, 0x0, 0x7fd74c491050, 0xc820014058)
	/usr/local/go/src/net/fd_unix.go:250 +0x23a
net.(*conn).Read(0xc820a5a090, 0xc821614000, 0x1000, 0x1000, 0x0, 0x0, 0x0)
	/usr/local/go/src/net/net.go:172 +0xe4
net/http.noteEOFReader.Read(0x7fd74c465258, 0xc820a5a090, 0xc8215f0068, 0xc821614000, 0x1000, 0x1000, 0x405773, 0x0, 0x0)
	/usr/local/go/src/net/http/transport.go:1687 +0x67
net/http.(*noteEOFReader).Read(0xc8215ae1a0, 0xc821614000, 0x1000, 0x1000, 0xc82159ad1d, 0x0, 0x0)
	<autogenerated>:284 +0xd0
bufio.(*Reader).fill(0xc8202a2b40)
	/usr/local/go/src/bufio/bufio.go:97 +0x1e9
bufio.(*Reader).Peek(0xc8202a2b40, 0x1, 0x0, 0x0, 0x0, 0x0, 0x0)
	/usr/local/go/src/bufio/bufio.go:132 +0xcc
net/http.(*persistConn).readLoop(0xc8215f0000)
	/usr/local/go/src/net/http/transport.go:1073 +0x177
created by net/http.(*Transport).dialConn
	/usr/local/go/src/net/http/transport.go:857 +0x10a6
```

We may want to get even more aggressive in the future.

@kubernetes/sig-testing
2016-06-09 04:11:01 -07:00
k8s-merge-robot
4d430ba09c Merge pull request #25400 from jsafrane/devel/operation-error
Automatic merge from submit-queue

Add specific error type for "operation already exists" error.

PersistentVolume controller needs to know why scheduling a new operation has failed - if the operation was already running or some other error happened.
2016-06-09 03:28:16 -07:00
k8s-merge-robot
7932158540 Merge pull request #26974 from pmorie/log-spam
Automatic merge from submit-queue

Reduce log spam in dockertools manager.go

Fixes #26972

@kubernetes/sig-node
2016-06-09 02:43:30 -07:00
k8s-merge-robot
f1ccc2f1be Merge pull request #26354 from csrwng/fix_visitor_error
Automatic merge from submit-queue

kubectl: fix URLVisitor error message

The status of a failed fetch will usually include the code, resulting in a duplicate code in the error message:
```
unable to read URL "http://git:8080/ruby-hello-world.git", server reported 401 401 Unauthorized
```

This change rearranges them so at least the message doesn't stutter (and includes the code in case it's not part of the status):
```
unable to read URL "http://git:8080/ruby-hello-world.git", server reported 401 Unauthorized, status code 401
```

[![Analytics](https://kubernetes-site.appspot.com/UA-36037335-10/GitHub/.github/PULL_REQUEST_TEMPLATE.md?pixel)]()
2016-06-09 02:43:27 -07:00
k8s-merge-robot
0b6e0e2b02 Merge pull request #26749 from Clarifai/describe-svc
Automatic merge from submit-queue

AWS: kubectl get service should print hostnames for LB services

Fixes #21526

Also test wide outputs. We only guarantee the first IP to be fully printed
if multiple ingresses are present. For AWS, which has no ingress IPs, but
only hostnames, the ELB hostname will be truncated, unless -o=wide is
specified.
2016-06-09 02:43:23 -07:00
k8s-merge-robot
aa008ab097 Merge pull request #27094 from caseydavenport/cd-networkpolicy-validation-fix
Automatic merge from submit-queue

Fix NetworkPolicy validation bug

Fix bugs in NetworkPolicy resource (new in v1.3) validation.

Please add this to the v1.3 milestone.
2016-06-09 01:58:38 -07:00
k8s-merge-robot
29c5d6c721 Merge pull request #26848 from pmorie/wrap-volumes
Automatic merge from submit-queue

Wrap more comments in pkg/volume

Wrap some more comments in `pkg/volume`
2016-06-09 01:15:52 -07:00
k8s-merge-robot
a5cd171c2f Merge pull request #25064 from Clarifai/httpheaders-querystring
Automatic merge from submit-queue

Preserve query strings in HTTP probes instead of escaping them

Fixes a problem reported on Slack by devth.

```release-note
* Allow the use of query strings and URI fragments in HTTP probes
```

This might also preserve fragments, for those crazy enough to pass them.

I am using url.Parse() on the path in order to get path/query/fragment
and also deliberately avoiding the addition of more fields to the API.
2016-06-09 01:15:47 -07:00
k8s-merge-robot
5a5c490a43 Merge pull request #27057 from janetkuo/kubectl-run-server-discovery
Automatic merge from submit-queue

Don't ignore error when Discovery().ServerResources() in kubectl run

Ref #27014
This doesn't fix the flake directly, but expose the error happened when the flake happens. 

@kubernetes/kubectl @caesarxuchao 

[![Analytics](https://kubernetes-site.appspot.com/UA-36037335-10/GitHub/.github/PULL_REQUEST_TEMPLATE.md?pixel)]()
2016-06-08 23:49:04 -07:00
k8s-merge-robot
72beb65ee5 Merge pull request #27027 from andyzheng0831/gci-network
Automatic merge from submit-queue

GCI: add support for network plugin

I had run e2e against a cluster with both master and nodes on GCI a couple of times. The PR auto tests will cover the hybrid cluster with just master on GCI.

cc/ @roberthbailey @fabioy @kubernetes/goog-image
2016-06-08 23:01:19 -07:00
k8s-merge-robot
7913573f6d Merge pull request #27018 from mml/oidc
Automatic merge from submit-queue

Avoid pathname collisions in TestNewOICAuthProvider.
2016-06-08 22:11:34 -07:00
k8s-merge-robot
b0cd744b12 Merge pull request #26985 from bprashanth/image_exit
Automatic merge from submit-queue

Exit image puller subshell

Exit the subshell with 0 so even if the last docker pull fails the pod doesn't end up in the error state.
2016-06-08 21:13:16 -07:00
Dawn Chen
9b7a9932a1 Merge pull request #27108 from kubernetes/revert-26830-node-e2e-build
Revert "Only build essential targets for node e2e"
2016-06-08 20:15:26 -07:00
Dawn Chen
a51fdbb152 Revert "Only build essential targets for node e2e" 2016-06-08 20:02:09 -07:00
Dawn Chen
f54975a738 Merge pull request #27106 from fejta/pr
Wait for arc.getArchive() to complete before running tests
2016-06-08 19:42:28 -07:00
Erick Fejta
c1d915df5b Wait for arc.getArchive() to complete before running tests 2016-06-08 19:07:07 -07:00
Davide Agnello
bb1ae12906 Vendor/Godep updates 2016-06-08 18:17:42 -07:00
Davide Agnello
eb32bf55ef Adding support for lbaas-v2 in OpenStack Cloud Provider 2016-06-08 18:16:59 -07:00
Daniel Smith
1d25410c3b Run new update-generated-docs.sh script 2016-06-08 17:25:21 -07:00
Dawn Chen
8d510898f0 Merge pull request #26830 from timstclair/node-e2e-build
Only build essential targets for node e2e
2016-06-08 17:23:21 -07:00
Daniel Smith
adcd48ccea update documentation & hooks 2016-06-08 17:22:12 -07:00
Daniel Smith
8faa88626d Revamp doc generation
Auto generated docs are **NO LONGER CHECKED IN**, only placeholders.

To generate them, e.g. before exporting docs, run hack/generate-docs.sh.

hack/verify-generated-docs.sh ensures that generated docs are merely the
placeholder text.

hack/update-generated-docs.sh puts the placeholder text in the proper
places.

The old munge behavior is moved into hack/{update|verify}-munge-docs.sh.
2016-06-08 17:22:12 -07:00
Dawn Chen
34dcea3037 Merge pull request #27065 from timstclair/cadvisor-godeps
Bump cAdvisor to v0.23.3
2016-06-08 17:21:08 -07:00
k8s-merge-robot
6986a3b2e2 Merge pull request #27091 from mml/union-merge
Automatic merge from submit-queue

Mark known-flags to merge via union.
2016-06-08 17:11:02 -07:00
k8s-merge-robot
613bbbb131 Merge pull request #26667 from mml/skip-local-data
Automatic merge from submit-queue

Stop 'kubectl drain' deleting pods with local storage.

Kubectl drain will not continue if there are pods with local storage unless
forced with --delete-local-data.

Fixes #23972
2016-06-08 17:10:58 -07:00
Casey Davenport
6fe7da72b4 Fix NetworkPolicy validation bug 2016-06-08 16:31:29 -07:00
Rudi Chiarito
40f76a95cb AWS: kubectl get service should print hostnames for LoadBalancer services
Fixes #21526

Also test wide outputs. We only guarantee the first IP to be fully printed
if multiple ingresses are present. For AWS, which has no ingress IPs, but
only hostnames, the ELB hostname will be truncated, unless -o=wide is
specified.
2016-06-08 19:29:48 -04:00
k8s-merge-robot
54b352ae3a Merge pull request #26907 from smarterclayton/tolerate_quantity
Automatic merge from submit-queue

Resource quantity must support leading and trailing whitespace in JSON for back-compat

For backwards compatibility reasons, we must continue to support leading or trailing whitespace on Quantity values when deserialized from JSON.  We must also support numbers serialized into yaml (`cpu: 1`) and JSON (`"cpu": 1`)

Fixes #26898
2016-06-08 16:16:41 -07:00
Matt Liggett
52f6b27365 Mark known-flags to merge via union.
This avoids pointless merge conflicts in a simple text file.
Fixes #27088.
2016-06-08 15:59:35 -07:00
k8s-merge-robot
85b67f2f00 Merge pull request #26874 from wojtek-t/fix_retry_after
Automatic merge from submit-queue

Fix Retry-After in clients

Fix #26871
2016-06-08 15:21:53 -07:00
Matt Liggett
d09af4a1d6 Stop 'drain' deleting pods with local storage.
Unless forced with --delete-local-data.  Also a refactoring of the
kubectl drain logic that selects/rejects pods and produces error/warning
messages.
2016-06-08 14:59:03 -07:00
Colin Hom
782bf6e866 extract federation image push from push-ci-build.sh 2016-06-08 14:43:12 -07:00