Commit Graph

33386 Commits

Author SHA1 Message Date
Kubernetes Submit Queue
a3e339b6d6 Merge pull request #30033 from caesarxuchao/fix-29975
Automatic merge from submit-queue

remove logging to t.lofg

Until release 1.6, golang's t.report() doesn't protect t.output with a read lock: https://github.com/golang/go/blob/release-branch.go1.6/src/testing/testing.go#L536-L538. It leads to the data race in https://github.com/kubernetes/kubernetes/issues/29975.

I think we can just remove the offending lines:
```
logWatcher1 := eventBroadcaster.StartLogging(t.Logf) // Prove that it is useful
```
It looks like the code just prints event to test log for human. The line is originally added by @lavalamp in https://github.com/kubernetes/kubernetes/pull/5535. (Thanks to @saad-ali for digging the history of the file)

Fix #29975. Marked as P0 because it fixes a P0 flake that blocks everyone.
2016-08-04 12:00:31 -07:00
Kubernetes Submit Queue
c24cf99fa6 Merge pull request #30055 from lixiaobing10051267/masterClonePath
Automatic merge from submit-queue

Replace with explicit kubernetes fork path

At other place in development.md, explicit kubernetes fork path has been existed:

    mkdir -p $GOPATH/src/k8s.io
    cd $GOPATH/src/k8s.io
    # Replace "$YOUR_GITHUB_USERNAME" below with your github username
    git clone https://github.com/$YOUR_GITHUB_USERNAME/kubernetes.git

the following is easy to be confused and can be replaced with same description:

    git clone https://path/to/your/fork .
2016-08-04 11:23:58 -07:00
Kubernetes Submit Queue
41abea5a96 Merge pull request #30053 from fejta/gke
Automatic merge from submit-queue

Set GOOGLE_APPLICATION_CREDENTIALS after activating

https://developers.google.com/identity/protocols/application-default-credentials

We should set this explicitly as https://golang.org/pkg/os/user/#User (which the oauth libraries use: [sdk.go] - [default.go] via [CreateGCECloud]) will ignore any `$HOME` overrides.

[CreateGCECloud]: fa95788e56/test/e2e/e2e.go (L75)
[sdk.go]: fa95788e56/vendor/golang.org/x/oauth2/google/sdk.go (L165)
[default.go]: fa95788e56/vendor/golang.org/x/oauth2/google/default.go (L101)

Mitigates https://github.com/kubernetes/kubernetes/issues/28514 and related to https://github.com/kubernetes/test-infra/issues/266
2016-08-04 11:23:53 -07:00
Kubernetes Submit Queue
4080ad770a Merge pull request #25968 from rrati/density-capacity-pods
Automatic merge from submit-queue

Added test to density that will run maximum capacity pods on nodes

Added a test to the Density Suite that will load the kubelets with their maximum capacity number of pods
2016-08-04 11:23:48 -07:00
Zhou Fang
0801c082a2 add cadvisor client v2 dependency 2016-08-04 10:51:44 -07:00
Chao Xu
2558c239b8 remove logWatcher to logf 2016-08-04 10:28:18 -07:00
Zhou Fang
637e0f91ce add +build linux to density_test, resource_usage_test and resource_collector 2016-08-04 10:21:01 -07:00
Buddha Prakash
49201f6923 Update Libcontainer's Cgroup Config: AllowAllDevices to be Nil 2016-08-04 10:05:30 -07:00
Matt Bruzek
b9ac078be7 Updating the util.sh script to work with latest version of juju. 2016-08-04 12:04:16 -05:00
Buddha Prakash
216d707f28 Bump Libcontainer to latest head 2016-08-04 10:04:14 -07:00
Kubernetes Submit Queue
5841f6e164 Merge pull request #29795 from xiangpengzhao/fix-makefile-cleanup
Automatic merge from submit-queue

Makefile cleanup

Keep the same style for each PHONY and rule.

cc @thockin
2016-08-04 09:37:25 -07:00
Kubernetes Submit Queue
689f06f936 Merge pull request #30080 from jszczepkowski/ps-rollback
Automatic merge from submit-queue

Reverted conversion of influx-db to PetSet.

```release-note
Reverted conversion of influx-db to Pet Set, it is now a Replication Controller.
```

Reverted conversion of influx-db to PetSet, it is now a Replication Controller and Persistent Volumes are no longer used.
2016-08-04 09:37:21 -07:00
Kubernetes Submit Queue
42a12a4cd6 Merge pull request #29978 from hodovska/sharedInformer-fixup
Automatic merge from submit-queue

SharedInformerFactory: usage and fixes

Follow-up for #26709
2016-08-04 09:00:23 -07:00
Zach Loafman
963a05ec72 AWS/GCE: Rework use of master name
* Add a pillar for hostname (because even if there's a good Salt
function for it, I don't trust it to return the short hostname)
* Move INITIAL_ETCD_CLUSTER to just the GCE turn-up
* Remove the master_name, which isn't needed as a pillar
2016-08-04 08:46:36 -07:00
Robert Rati
fb72b50135 Added test to density that will run maximum capacity pods on all nodes 2016-08-04 11:46:22 -04:00
Kubernetes Submit Queue
20444ac84d Merge pull request #30057 from kubernetes/docs/devel-readme
Automatic merge from submit-queue

Improve developer README
2016-08-04 08:25:52 -07:00
Kubernetes Submit Queue
04733432db Merge pull request #25595 from erictune/sj-proposal
Automatic merge from submit-queue

Scheduled Job proposal elaboration
2016-08-04 08:25:47 -07:00
Kubernetes Submit Queue
8da97dff86 Merge pull request #30011 from janetkuo/map-concurrent-write-read
Automatic merge from submit-queue

Fix the map concurrent read/write issue in deployment controller

Fixes #29960

@lavalamp @kubernetes/deployment
2016-08-04 08:25:43 -07:00
derekwaynecarr
611c127f0d kubelet eviction manager support for min-reclaim 2016-08-04 11:17:44 -04:00
Jerzy Szczepkowski
c8d920ea6c Reverted conversion of influx-db to PetSet.
Reverted conversion of influx-db to PetSet.
2016-08-04 17:08:46 +02:00
derekwaynecarr
4c37a813df Move NamespaceLifecycle to use shared informers 2016-08-04 11:01:09 -04:00
Kubernetes Submit Queue
8ab06a3f86 Merge pull request #29958 from ronnielai/dep
Automatic merge from submit-queue

Updated cadvisor version

cc @derekwaynecarr
2016-08-04 07:50:19 -07:00
Eric Tune
87cba77e34 ScheduledJob: proposal updates 2016-08-04 07:29:06 -07:00
Bryan Boreham
bd8f4ff8cf Send SIGTERM on Pdeathsig to allow child to clean up
Also fix the comment to better describe the effect of this code.
2016-08-04 15:14:47 +01:00
codejuan
b038355c81 fixed regex error, which replace 2 line in template file 2016-08-04 22:13:45 +08:00
Kubernetes Submit Queue
d6854cbb6b Merge pull request #30062 from gambol99/spelling
Automatic merge from submit-queue

spelling mistakes

- fixing the spelling mistakes in aws.go
2016-08-04 07:13:19 -07:00
Kubernetes Submit Queue
0e71c0c808 Merge pull request #30031 from andreykurilin/apiserver_typo
Automatic merge from submit-queue

Fix various typos in apiserver pkg
2016-08-04 07:13:14 -07:00
Kubernetes Submit Queue
c2340870c6 Merge pull request #29952 from fabianofranz/handle_container_terminated_pod_running_condition
Automatic merge from submit-queue

Handle container terminated but pod still running in conditions

Sometimes when you have a pod with more than one container, and the container runs and terminates really fast, `PodContainerRunning` can go into a state where the pod indicates it's still running, but the container is already terminated. Handle that condition by returning `ErrContainerTerminated` when it happens.
2016-08-04 07:13:08 -07:00
Kubernetes Submit Queue
33239c1e6f Merge pull request #29948 from mbruzek/juju-kubedns-update
Automatic merge from submit-queue

Replacing skydns with kubedns for the juju cluster. #29720

```release-note
* Updating the cluster/juju provider to use kubedns in place of skydns.
```
2016-08-04 06:38:18 -07:00
Bryan Boreham
1307799a66 Run 'etcd2' if it exists, in preference to 'etcd'
This is to accommodate CoreOS, where 'etcd' is a very old version
2016-08-04 14:37:11 +01:00
Kubernetes Submit Queue
3e5b40d9a5 Merge pull request #29947 from ixdy/go-bindata-cross-image
Automatic merge from submit-queue

Install go-bindata in cross-build image

Another follow-up to #25584.

We need `go-bindata` to create `test/e2e/generated`, and downloading it with `go get` at build time is painful for a variety of reasons. We can just include it in the cross-build image and not worry about it, especially as it updates very infrequently.

This fixes `hack/update-generated-protobuf.sh` as well.

cc @jayunit100 @soltysh
2016-08-04 06:03:29 -07:00
Kubernetes Submit Queue
b9e808622d Merge pull request #27792 from jeffvance/e2e-prebind
Automatic merge from submit-queue

pv e2e refactor and pre-bind test

refactored persistentvolume e2e so that multiple It() tests can be run. Added one test case for pre-binding, but the overall structure of the test should allow additional test cases to be more easily added.
2016-08-04 05:26:43 -07:00
deads2k
0beca8e6be replace improper create quota alias 2016-08-04 08:02:14 -04:00
Kubernetes Submit Queue
16454277aa Merge pull request #29930 from ericchiang/rbac-validation-dont-mix-non-resource-urls-and-resources
Automatic merge from submit-queue

rbac validation: rules can't combine non-resource URLs and regular resources

This PR updates the validation used for RBAC to prevent rules from mixing non-resource URLs and regular resources.

For example the following is no longer valid

```yml
kind: ClusterRole
apiVersion: rbac.authorization.k8s.io/v1alpha1
metadata:
  name: admins
rules:
  - apiGroups: ["*"]
    resources: ["*"]
    verbs: ["*"]
    nonResourceURLs: ["*"]
```

And must be rewritten as so.

```yml
kind: ClusterRole
apiVersion: rbac.authorization.k8s.io/v1alpha1
metadata:
  name: admins
rules:
  - apiGroups: ["*"]
    resources: ["*"]
    verbs: ["*"]
  - nonResourceURLs: ["*"]
    verbs: ["*"]
``` 

It also:
* Mandates non-zero length arrays for required resources.
* Mandates non-resource URLs only be used for ClusterRoles (not namespaced Roles).
* Updates the swagger validation so `verbs` are the only required field in a rule. Further validation is done by the server.

Also, do we need to bump the API version?

Discussed by @erictune and @liggitt  in #28304

Updates kubernetes/features#2

cc @kubernetes/sig-auth 

Edit:
* Need to update the RBAC docs if this change goes in.
2016-08-04 04:52:51 -07:00
Tamer Tas
f027b7a896 Add ImageService stub for rkt CRI shim 2016-08-04 14:22:22 +03:00
Kubernetes Submit Queue
1933462c7b Merge pull request #29925 from ronnielai/container-gc
Automatic merge from submit-queue

Delete containers when pod is evicted

#29803
2016-08-04 04:20:02 -07:00
Kubernetes Submit Queue
0e55753dcc Merge pull request #29920 from thockin/verify-godeps-results
Automatic merge from submit-queue

Make verify-godeps check the results

Verify the results of godep restore/save.  When forced to run this finds a handful of real deltas that need to be resolved.

Fixes #29402

@rmmh because you wrote  #29571 which is the major culprit
2016-08-04 03:37:44 -07:00
Kubernetes Submit Queue
10b0f58f51 Merge pull request #30060 from gmarek/schedulerPredicates-json
Automatic merge from submit-queue

Fix JSON-based scheduler predicates tests

Fixes two completely broken tests. cc @kevin-wangzefeng @jayunit100
2016-08-04 02:55:55 -07:00
Kubernetes Submit Queue
05b12d4656 Merge pull request #30058 from wojtek-t/increase_namespaces_in_load
Automatic merge from submit-queue

Increase number of namespaces in load test

Ref #27595
2016-08-04 02:23:35 -07:00
Kubernetes Submit Queue
2ff8280123 Merge pull request #29915 from wojtek-t/prepare_for_controller_ref_in_scheduler
Automatic merge from submit-queue

Prepare for using "ControllerRef" in scheduler

This is part of a PR that I already have to avoid a bunch of rebases in the future (controller ref probably won't happen in 1.4 release).

@davidopp
2016-08-04 02:23:31 -07:00
Rohith
0da5f50b03 - fixing the spelling mistakes 2016-08-04 10:17:59 +01:00
lixiaobing10051267
fdd9ced105 Replace with explicit kubernetes fork path 2016-08-04 17:01:49 +08:00
gmarek
2a0d3fbcfb Fix JSON-based scheduler predicates tests 2016-08-04 10:55:27 +02:00
Ke Zhang
3973856ac2 replenish quota for service optimize, add testcase 2016-08-04 16:55:05 +08:00
Kubernetes Submit Queue
5273ac9b94 Merge pull request #29912 from wojtek-t/enable_pod_affinity
Automatic merge from submit-queue

Enable PodAffinity by default in scheduler

Ref #26144

@gmarek - FYI
2016-08-04 01:51:17 -07:00
Michal Fojtik
e220f95da7 Do not re-initialize the output on label selector conversion 2016-08-04 10:18:42 +02:00
Kubernetes Submit Queue
c8ea7af912 Merge pull request #29890 from dchen1107/test1
Automatic merge from submit-queue

Resolve docker-daemon cgroup issue for both systemd and non-systemd node for node e2e tests

Fixed https://github.com/kubernetes/kubernetes/issues/29827

cc/ @coufon this should unblock your pr: #29764

I validated both containervm image and coreos image, and works as expected. 

This is also required for adding gci image to node e2e test infrastructure.
2016-08-04 01:17:01 -07:00
Wojciech Tyczynski
7c4b797fb6 Increase number of namespaces in load test 2016-08-04 10:13:47 +02:00
Tamer Tas
7f97c7c615 Improve Developer README 2016-08-04 11:13:26 +03:00
Dominika Hodovska
305342c48d Use shared informer factory in controllers 2016-08-04 10:10:03 +02:00