Commit Graph

28514 Commits

Author SHA1 Message Date
k8s-merge-robot
aade9eaabc Merge pull request #24878 from pwittrock/docsdoc
Automatic merge from submit-queue

How to update docs - doc

## Contributor guidelines

1. Please read our [contributor guidelines](https://github.com/kubernetes/kubernetes/blob/master/CONTRIBUTING.md).
1. See our [developer guide](https://github.com/kubernetes/kubernetes/blob/master/docs/devel/development.md).
1. Follow the instructions for [labeling and writing a release note for this PR](https://github.com/kubernetes/kubernetes/blob/master/docs/devel/pull-requests.md#release-notes) in the block below. (_this would also contain updated detail later about this new template_)

```release-note
* Use the release-note-* labels to set the release note state 
* Clear this block to use the PR title as the release note 
-OR-
* Enter your extended release note here
```
2016-05-09 23:18:22 -07:00
k8s-merge-robot
35aff74f9c Merge pull request #23121 from deromka/master
Automatic merge from submit-queue

Added vsphere support for vagrant

Since the native vsphere support (using govc library) requires admin permissions on ESX/vCenter, not everyone can have such permissions. So I'm adding a vsphere support using vagrant using vagrant-vsphere plugin
2016-05-09 23:18:19 -07:00
k8s-merge-robot
a57876b98c Merge pull request #25178 from zmerlynn/random_max_target_pools
Automatic merge from submit-queue

GCE: Allow node count to exceed GCE TargetPool maximums

```release-note
If the cluster node count exceeds the GCE TargetPool maximum (currently 1000),
randomly select which nodes are members of Kubernetes External Load Balancers.
```


[![Analytics](https://kubernetes-site.appspot.com/UA-36037335-10/GitHub/.github/PULL_REQUEST_TEMPLATE.md?pixel)]()

If we would exceeded the TargetPool API maximums, instead just
randomly select some subsection of the nodes to include in the TP
instead.
2016-05-09 23:18:15 -07:00
k8s-merge-robot
f9b8fd0c96 Merge pull request #25011 from zhouhaibing089/addclose
Automatic merge from submit-queue

followup to add http server close method

Fixes #25009, a follow up of https://github.com/kubernetes/kubernetes/pull/24595.
2016-05-09 22:32:02 -07:00
k8s-merge-robot
c4214f743f Merge pull request #24918 from Random-Liu/add-docker-operation-timeout
Automatic merge from submit-queue

Kubelet: Add docker operation timeout

For #23563.
Based on #24748, only the last 2 commits are new.

This PR:
1) Add timeout for all docker operations.
2) Add docker operation timeout metrics
3) Cleanup kubelet stats and add runtime operation error and timeout rate monitoring.
4) Monitor runtime operation error and timeout rate in kubelet perf.

@yujuhong 
/cc @gmarek Because of the metrics change.
/cc @kubernetes/sig-node
2016-05-09 21:51:52 -07:00
Zach Loafman
faf0c44429 GCE: Allow nodes to exceed target pool maximums
If we would exceeded the TargetPool API maximums, instead just
randomly select some subsection of the nodes to include in the TP
instead.
2016-05-09 21:44:58 -07:00
k8s-merge-robot
def7639457 Merge pull request #25245 from pmorie/kubelet/cadvisor
Automatic merge from submit-queue

Reduce kubelet LOC: extract cadvisor

Step 2 of #25028 

@yujuhong @kubernetes/sig-node
2016-05-09 21:09:42 -07:00
k8s-merge-robot
27f418848d Merge pull request #25099 from mikebrow/devel-tree-80col-updates
Automatic merge from submit-queue

devel/ tree minor edits

Address line wrap issue #1488. Also cleans up other minor editing issues in the docs/devel/* tree such as spelling errors, links,...

Signed-off-by: Mike Brown <brownwm@us.ibm.com>
2016-05-09 21:09:38 -07:00
k8s-merge-robot
088694fa72 Merge pull request #23343 from derekparker/self-hosted-kubelet-proposal
Automatic merge from submit-queue

docs: Self-hosted Kubelet proposal

Provides a proposal for changes needed with Kubernetes to allow for a
self-hosted Kubelet bootstrap.
2016-05-09 21:09:35 -07:00
Phillip Wittrock
c6aa507aae Address PR comments 2016-05-09 23:20:08 +00:00
k8s-merge-robot
ff339c77cf Merge pull request #25269 from janetkuo/kubectl-conventions
Automatic merge from submit-queue

Document that kubectl commands shouldn't have aliases

## Pull Request Guidelines

1. Please read our [contributor guidelines](https://github.com/kubernetes/kubernetes/blob/master/CONTRIBUTING.md).
1. See our [developer guide](https://github.com/kubernetes/kubernetes/blob/master/docs/devel/development.md).
1. Follow the instructions for [labeling and writing a release note for this PR](https://github.com/kubernetes/kubernetes/blob/master/docs/devel/pull-requests.md#release-notes) in the block below.

```release-note
* Use the release-note-* labels to set the release note state 
* Clear this block to use the PR title as the release note 
-OR-
* Enter your extended release note here
```

Ref https://github.com/kubernetes/kubernetes/issues/25180#issuecomment-217453402 @kubernetes/kubectl 

[![Analytics](https://kubernetes-site.appspot.com/UA-36037335-10/GitHub/.github/PULL_REQUEST_TEMPLATE.md?pixel)]()
2016-05-09 15:03:19 -07:00
k8s-merge-robot
8d336ad9ba Merge pull request #25087 from ihmccreery/skew
Automatic merge from submit-queue

Clarify supported version skew between masters, nodes, and clients

Per discussion on #22897.

cc @bgrant0607 @roberthbailey
2016-05-09 15:03:16 -07:00
k8s-merge-robot
90c02c6d96 Merge pull request #24813 from duglin/AddBuildPath
Automatic merge from submit-queue

Make it clear where the `build` scripts are located


Signed-off-by: Doug Davis <dug@us.ibm.com>
2016-05-09 15:03:12 -07:00
Phillip Wittrock
448cc8754d Address PR comments 2016-05-09 13:39:27 -07:00
Phillip Wittrock
163a4873c4 How to update docs - doc 2016-05-09 13:39:27 -07:00
k8s-merge-robot
8a81000b71 Merge pull request #25331 from hongchaodeng/w
Automatic merge from submit-queue

etcd3/watcher: fix goroutine leak if ctx is canceled

### Problem
In reflector.go, it could probably call Stop() without retrieving all results
from ResultChan(). See [here](https://github.com/kubernetes/kubernetes/blob/master/pkg/client/cache/reflector.go#L369). A potential leak is that when an error has happened, it could block on resultChan,
and then cancelling context in Stop() wouldn't unblock it.

### What's this PR?
This fixes the problem by making it also select ctx.Done and cancel context afterwards if error happened.
2016-05-09 12:12:15 -07:00
k8s-merge-robot
833cfc08b3 Merge pull request #24894 from david-mcmahon/cp2
Automatic merge from submit-queue

Update cherrypick doc for release notes to more accurately reflect current process.

## Contributor guidelines

1. Please read our [contributor guidelines](https://github.com/kubernetes/kubernetes/blob/master/CONTRIBUTING.md).
1. See our [developer guide](https://github.com/kubernetes/kubernetes/blob/master/docs/devel/development.md).
1. Follow the instructions for [labeling and writing a release note for this PR](https://github.com/kubernetes/kubernetes/blob/master/docs/devel/pull-requests.md#release-notes) in the block below.
2016-05-09 12:12:11 -07:00
Janet Kuo
b5ccc1465d Document that kubectl commands shouldn't have aliases 2016-05-09 11:19:52 -07:00
Mike Brown
a42ea536ba devel/ tree 80col wrap and other minor edits
Signed-off-by: Mike Brown <brownwm@us.ibm.com>
2016-05-09 12:42:50 -05:00
David McMahon
8c15651ff2 Update pull request and cherrypick docs for release notes to more accurately reflect current process. 2016-05-09 10:19:19 -07:00
k8s-merge-robot
e973b5d27a Merge pull request #25354 from wojtek-t/fix_protobuf_generator
Automatic merge from submit-queue

Fix script for updating protobufs.

Fix #25351

@thockin
2016-05-09 09:24:58 -07:00
Hongchao Deng
97f4647dc3 etcd3/watcher: fix goroutine leak if ctx is canceled
In reflector.go, it could probably call Stop() without retrieving all results
from ResultChan().
A potential leak is that when an error has happened, it could block on resultChan,
and then cancelling context in Stop() wouldn't unblock it.
This fixes the problem by making it also select ctx.Done and cancel context
afterwards if error happened.
2016-05-09 09:06:11 -07:00
k8s-merge-robot
58ccbffb82 Merge pull request #25303 from ingvagabund/hack-update-all-remove-dollar-symbol
Automatic merge from submit-queue

Hack update all remove dollar symbol

When not running ./hack/update in silent mode, the script fails due to undefined ``$Updating`` variable.
2016-05-09 08:36:10 -07:00
k8s-merge-robot
4642621baf Merge pull request #25348 from gmarek/cidr
Automatic merge from submit-queue

Make Kubemark assign cluster CIDRs
2016-05-09 07:32:36 -07:00
Wojciech Tyczynski
9a8e4f69cd Fix script for updating protobufs. 2016-05-09 15:57:26 +02:00
k8s-merge-robot
bd2b41ca67 Merge pull request #25130 from deads2k/create-guidelines
Automatic merge from submit-queue

create command guidance

`kubectl create <resource>` commands are multiplying (configmap, namespace, secret (2 flavors and a pull for a third), and serviceaccounts so far) and I think we should agree on the goal of those subcommands.

Right now, `kubectl run` is an easy entrypoint for new users or users who don't really care which resources are created.  It is possible to get stable output from `kubectl run`, but it really is intent-based: make this image go in some reasonable way.  At the other end of the spectrum, you can craft yaml files by hand get exactly the object you want, but that requires pretty deep API knowledge.  I think that `kubectl create <resource>` should be the starting point of the middle ground.

`kubectl create <resource>` commands should have just enough arguments to create valid objects and the expectation should be that users will use those objects as skeletons to tweak using `kubectl edit` or the future `kubectl set` to modify them after the fact.  Editing an already existing and valid object is a lot easier than creating one from scratch, so it reduces the barrier to entry.

@kubernetes/kubectl @smarterclayton @bgrant0607 @liggitt
2016-05-09 06:31:27 -07:00
k8s-merge-robot
6596ae0c61 Merge pull request #24838 from saad-ali/attachControllerOpMap
Automatic merge from submit-queue

Add data structure for managing go routines by name

This PR introduces a data structure for managing go routines by name. It prevents the creation of new go routines if an existing go routine with the same name exists. This will enable parallelization of the designs in https://github.com/kubernetes/kubernetes/issues/20262 and https://github.com/kubernetes/kubernetes/issues/21931 with sufficient protection to prevent starting multiple operations on the same volume.
2016-05-09 06:31:22 -07:00
gmarek
7981f479de Make Kubemark assign cluster CIDRs 2016-05-09 15:09:37 +02:00
Wojciech Tyczynski
3c089bff41 Merge pull request #25344 from wojtek-t/fix_dynamic_client_with_protobufs
Fix dynamic client in protobuf configuration
2016-05-09 15:06:43 +02:00
k8s-merge-robot
545d56a63b Merge pull request #24810 from derekwaynecarr/sources_cleanup
Automatic merge from submit-queue

Clean-up sources ready tracking in kubelet

moved sources ready tracking behind an interface, made it thread-safe.
2016-05-09 05:48:09 -07:00
deads2k
8e8e9cdce0 create command guidance 2016-05-09 08:14:48 -04:00
k8s-merge-robot
1a1229e206 Merge pull request #24795 from deads2k/use-all-attributes
Automatic merge from submit-queue

enable resource name and service account cases for impersonation

Adds the resource name check since that attribute was added for authorization.  Also adds a check against a separate resource for service accounts.  Allowing impersonation of service accounts to use a different resource check places control of impersonation with the same users to have the power to get the SA tokens directly.

@kubernetes/kube-iam 
@sgallagher FYI
2016-05-09 04:58:19 -07:00
Wojciech Tyczynski
ca5f18370a Fix dynamic client in protobuf configuration 2016-05-09 13:52:10 +02:00
k8s-merge-robot
2cf511b1f5 Merge pull request #24750 from derekwaynecarr/kubelet_eviction_flag_parsing
Automatic merge from submit-queue

Kubelet eviction flag parsers and tests

The first two commits are from https://github.com/kubernetes/kubernetes/pull/24559 that have achieved LGTM.  

The last commit is only part that is interesting, it adds the parsing logic to handle the flags, and reserves `pkg/kubelet/eviction` for eviction manager logic.
2016-05-09 04:15:04 -07:00
k8s-merge-robot
84db106ecb Merge pull request #24701 from euank/addon-ssh-user
Automatic merge from submit-queue

test/e2e/addon_update: Respect KUBE_SSH_USER

This change makes the e2e tests more consistent as the other ones all already respected this variable.

I didn't do the larger change of re-factoring it to use `framework.SSH` because getting the `scp` portion working there has some significant complexity. If I find time, I'd like to go back and do it since this test needs a little ❤️

cc @yifan-gu, @kubernetes/sig-testing
2016-05-09 03:31:48 -07:00
k8s-merge-robot
761fc00234 Merge pull request #25337 from wojtek-t/content_type_in_tests
Automatic merge from submit-queue

Enable using protobufs in e2e tests

ref #25132
2016-05-09 02:50:03 -07:00
k8s-merge-robot
87cf3a31bb Merge pull request #24544 from gmarek/heapster
Automatic merge from submit-queue

Add heapster to kubemark

cc @mwielgus
2016-05-09 02:49:59 -07:00
Wojciech Tyczynski
cb875e0f8c Enable using protobufs in e2e tests 2016-05-09 10:57:35 +02:00
Wojciech Tyczynski
a7be41d4fe Merge pull request #25332 from wojtek-t/fix_protobuf_failures
Fix protobuf failures
2016-05-09 10:23:20 +02:00
k8s-merge-robot
06e2a3543f Merge pull request #25310 from swagiaal/cloud-provder
Automatic merge from submit-queue

Add cloud-provider setting to local-up-cluster.sh
2016-05-09 01:08:19 -07:00
Wojciech Tyczynski
1ea0d46bb3 Set headers in Watch() and Stream() requests 2016-05-09 09:33:13 +02:00
Wojciech Tyczynski
ac5d2cec44 Test if headers are set 2016-05-09 09:31:49 +02:00
k8s-merge-robot
2f1a9dd65c Merge pull request #24696 from saad-ali/attachControllerSkeleton
Automatic merge from submit-queue

Add data structure for storing attach detach controller state.

This PR introduces the data structure for maintaining the in-memory state for the new attach/detach controller (#20262).
2016-05-08 23:44:38 -07:00
k8s-merge-robot
cd3ab5d82d Merge pull request #24655 from njuicsgz/master
Automatic merge from submit-queue

kube-proxy userspace, enable RR if connect endpoint which is session affinity fails

for issue #24648
2016-05-08 23:04:23 -07:00
k8s-merge-robot
fa95788e56 Merge pull request #24242 from thockin/godep_vendor_dir
Automatic merge from submit-queue

Move godeps to vendor/

This is a first-step towards glide support, maybe we don't want or need to take this, but it was easy to try.

This fails to compile, not sure why:

```
# k8s.io/kubernetes/pkg/apis/extensions/v1beta1
_output/local/go/src/k8s.io/kubernetes/pkg/apis/extensions/v1beta1/conversion_generated.go:2703: undefined: extensions.ClusterAutoscaler
_output/local/go/src/k8s.io/kubernetes/pkg/apis/extensions/v1beta1/conversion_generated.go:2703: undefined: ClusterAutoscaler
_output/local/go/src/k8s.io/kubernetes/pkg/apis/extensions/v1beta1/conversion_generated.go:2719: undefined: extensions.ClusterAutoscaler
_output/local/go/src/k8s.io/kubernetes/pkg/apis/extensions/v1beta1/conversion_generated.go:2719: undefined: ClusterAutoscaler
_output/local/go/src/k8s.io/kubernetes/pkg/apis/extensions/v1beta1/conversion_generated.go:2723: undefined: extensions.ClusterAutoscalerList
_output/local/go/src/k8s.io/kubernetes/pkg/apis/extensions/v1beta1/conversion_generated.go:2723: undefined: ClusterAutoscalerList
_output/local/go/src/k8s.io/kubernetes/pkg/apis/extensions/v1beta1/conversion_generated.go:3468: Convert_extensions_JobSpec_To_v1beta1_JobSpec redeclared in this block
	previous declaration at _output/local/go/src/k8s.io/kubernetes/pkg/apis/extensions/v1beta1/conversion.go:328
_output/local/go/src/k8s.io/kubernetes/pkg/apis/extensions/v1beta1/conversion_generated.go:3845: Convert_extensions_ScaleStatus_To_v1beta1_ScaleStatus redeclared in this block
	previous declaration at _output/local/go/src/k8s.io/kubernetes/pkg/apis/extensions/v1beta1/conversion.go:98
_output/local/go/src/k8s.io/kubernetes/pkg/apis/extensions/v1beta1/conversion_generated.go:4737: Convert_v1beta1_JobSpec_To_extensions_JobSpec redeclared in this block
	previous declaration at _output/local/go/src/k8s.io/kubernetes/pkg/apis/extensions/v1beta1/conversion.go:380
_output/local/go/src/k8s.io/kubernetes/pkg/apis/extensions/v1beta1/conversion_generated.go:5186: Convert_v1beta1_ScaleStatus_To_extensions_ScaleStatus redeclared in this block
	previous declaration at _output/local/go/src/k8s.io/kubernetes/pkg/apis/extensions/v1beta1/conversion.go:120
_output/local/go/src/k8s.io/kubernetes/pkg/apis/extensions/v1beta1/conversion_generated.go:2723: too many errors
!!! Error in /home/thockin/tmp/godep-vendor/src/k8s.io/kubernetes/hack/lib/golang.sh:417
```
2016-05-08 22:20:09 -07:00
k8s-merge-robot
5dd087040b Merge pull request #24511 from pbx0/kubelet-symlink
Automatic merge from submit-queue

cluster/images/hyperkube: create symlink for each server

Add a kubelet symlink so that the hyperkube image can appear as a kubelet image. https://github.com/kubernetes/kubernetes/issues/24510
2016-05-08 21:31:23 -07:00
Tim Hockin
2100016e0f Don't set the golang env for ginkgo
a) it doesn't need it

b) changing CWD to a path with symlinks breaks deep within ginkgo, where it
crafts a relative path to ../../../../../../platforms/amd64/whatever which then
traverses the physical path not the symlinked one, and breaks.
2016-05-08 20:32:09 -07:00
Tim Hockin
7aff0f1486 Rename codecgen "testdata" dir so ugorji works
Go mistreats "testdata" and can't find vendor/ dirs.
2016-05-08 20:32:09 -07:00
Tim Hockin
18e7a3eb24 Get rid of hack/after-build scripts
The build is now fast enough to not need them.
2016-05-08 20:32:06 -07:00
Tim Hockin
833770d9ac Fix verify-godeps 2016-05-08 20:30:39 -07:00