Commit Graph

7797 Commits

Author SHA1 Message Date
Minhan Xia
6da0c11063 add dockershim checkpoint node e2e test 2017-06-29 13:26:09 -07:00
Kubernetes Submit Queue
f505c386d3 Merge pull request #47488 from k82cn/k8s_47382
Automatic merge from submit-queue (batch tested with PRs 47656, 47488)

Checked whether balanced Pods were created.

**What this PR does / why we need it**:
In #47382, it seems balanced Pods were not created; this PR will check whether balanced Pods were created, and show logs.

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: partially fixes #47382 

**Release note**:

```release-note-none
```
2017-06-26 08:55:30 -07:00
Kubernetes Submit Queue
d07a0200c7 Merge pull request #47656 from jsafrane/speed-up-volume-integration
Automatic merge from submit-queue

Speed up PV integration tests

Integration tests are ~20 seconds shorter. Before (with #47645 applied):
```
--- PASS: TestPodDeletionWithDswp (3.70s)
--- PASS: TestPodUpdateWithWithADC (3.61s)
--- PASS: TestPodUpdateWithKeepTerminatedPodVolumes (3.59s)
--- PASS: TestPodAddedByDswp (5.69s)
--- PASS: TestPersistentVolumeRecycler (2.68s)
--- PASS: TestPersistentVolumeDeleter (12.61s)
--- PASS: TestPersistentVolumeBindRace (12.53s)
--- PASS: TestPersistentVolumeClaimLabelSelector (12.52s)
--- PASS: TestPersistentVolumeClaimLabelSelectorMatchExpressions (2.62s)
--- PASS: TestPersistentVolumeMultiPVs (3.98s)
--- PASS: TestPersistentVolumeMultiPVsPVCs (3.67s)
--- PASS: TestPersistentVolumeControllerStartup (7.41s)
--- PASS: TestPersistentVolumeProvisionMultiPVCs (4.85s)
--- PASS: TestPersistentVolumeMultiPVsDiffAccessModes (2.71s)
ok  	k8s.io/kubernetes/test/integration/volume	82.304s
```

After:
```
--- PASS: TestPodDeletionWithDswp (3.70s)
--- PASS: TestPodUpdateWithWithADC (3.61s)
--- PASS: TestPodUpdateWithKeepTerminatedPodVolumes (3.63s)
--- PASS: TestPodAddedByDswp (5.73s)
--- PASS: TestPersistentVolumeRecycler (3.53s)
--- PASS: TestPersistentVolumeDeleter (4.56s)
--- PASS: TestPersistentVolumeBindRace (3.52s)
--- PASS: TestPersistentVolumeClaimLabelSelector (3.52s)
--- PASS: TestPersistentVolumeClaimLabelSelectorMatchExpressions (3.49s)
--- PASS: TestPersistentVolumeMultiPVs (3.07s)
--- PASS: TestPersistentVolumeMultiPVsPVCs (3.94s)
--- PASS: TestPersistentVolumeControllerStartup (7.64s)
--- PASS: TestPersistentVolumeProvisionMultiPVCs (6.25s)
--- PASS: TestPersistentVolumeMultiPVsDiffAccessModes (3.55s)
ok  	k8s.io/kubernetes/test/integration/volume	59.945s
```

/assign @gnufied 
@kubernetes/sig-storage-pr-reviews 

Release note
```release-note
NONE
```
2017-06-26 08:54:09 -07:00
Kubernetes Submit Queue
49b83c6cb4 Merge pull request #46425 from wanghaoran1988/move_workload_e2e_to_own_package
Automatic merge from submit-queue

Move the workload e2e tests to it's own package

**What this PR does / why we need it**:

Move the workload e2e tests to it's own package

**Release note**:
```
None
```
2017-06-26 05:25:49 -07:00
Kubernetes Submit Queue
e725c8fca7 Merge pull request #47961 from piosz/heapster-1.4
Automatic merge from submit-queue (batch tested with PRs 47961, 46276)

Bumped Heapster to v1.4.0-beta.0

Heapster release candidate for Kubernetes 1.7

cc @dchen1107 @caesarxuchao
2017-06-26 02:11:52 -07:00
Haoran Wang
76251ea782 Move the workload e2e tests to it's own package 2017-06-26 09:56:20 +08:00
Klaus Ma
0f0e104061 Checked whether balanced Pods were created. 2017-06-26 09:06:28 +08:00
Kubernetes Submit Queue
d39115a938 Merge pull request #48006 from bsalamat/e2e_test_cleanup
Automatic merge from submit-queue (batch tested with PRs 47650, 47936, 47939, 47986, 48006)

Remove e2e test that checked scheduler priority function for spreading of ReplicationController pods

**What this PR does / why we need it**:
Remove e2e test that checked scheduler priority function for spreading of ReplicationController pods.
The test is flaky in our test clusters, but passes in our local clusters. It looks like our e2e test clusters are not deterministic enough for checking all of the scheduler priority functions. Besides, this functionality is checked extensively by our unit tests.

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #47769

**Special notes for your reviewer**:

**Release note**:

```release-note
NONE
```
2017-06-24 05:15:46 -07:00
Kubernetes Submit Queue
3da95b6ec3 Merge pull request #47986 from MrHohn/e2e-esipp-change-port
Automatic merge from submit-queue (batch tested with PRs 47650, 47936, 47939, 47986, 48006)

[esipp-e2e] Change service port to avoid collision

**What this PR does / why we need it**: As https://github.com/kubernetes/kubernetes/issues/47745#issuecomment-310750499 indicates, changing service port in test to avoid collision. 

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #47745

**Special notes for your reviewer**:

**Release note**:

```release-note
NONE
```
2017-06-24 05:15:44 -07:00
Kubernetes Submit Queue
156ff3a72d Merge pull request #43637 from wongma7/framework
Automatic merge from submit-queue

Move e2e fromManifest funcs to manifest package

fixes https://github.com/kubernetes/kubernetes/issues/37007 again

The goal here is to remove e2e/framework's dependence on e2e/generated so that external projects can import e2e/framework without issue. I reorganize & try to make all the 'fromManifest' funcs consistent (to all use generated, return err) to prevent any reintroduction of generated into framework

```release-note
NONE
```
2017-06-23 21:32:23 -07:00
Kubernetes Submit Queue
68a05ac74b Merge pull request #47469 from crimsonfaith91/created
Automatic merge from submit-queue (batch tested with PRs 47993, 47892, 47591, 47469, 47845)

deprecate created-by annotation for cronjob

**What this PR does / why we need it**: This PR deprecates created-by annotation for cronjob. This is needed as we now have ControllerRef.

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: xref #44407

**Special notes for your reviewer**: I will create 3 PRs to fix the issue as the annotation is used in various parts of the codebase: cronjob, pod drain, and e2e test framework. This is the first PR. Other PRs can be found here: #47471, #47475

**Release note**:

```release-note
```
2017-06-23 18:05:51 -07:00
Kubernetes Submit Queue
14edc46c2e Merge pull request #47892 from ajitak/npd-config
Automatic merge from submit-queue (batch tested with PRs 47993, 47892, 47591, 47469, 47845)

Bump up npd version to v0.4.1

```
Bump up npd version to v0.4.1
```

Fixes #47219
2017-06-23 18:05:46 -07:00
Bobby (Babak) Salamat
6b949a1a6a Remove e2e test that checked scheduler priority function for ReplicationController spreading 2017-06-23 16:44:32 -07:00
Kubernetes Submit Queue
80a5842623 Merge pull request #47883 from bsalamat/e2e_test_cleanup
Automatic merge from submit-queue (batch tested with PRs 47883, 47179, 46966, 47982, 47945)

Remove e2e test for least requested prioirty function

**What this PR does / why we need it**:
Deletes a flaky e2e test for "least requested" priority function of the scheduler. This priority function is tested more extensively in our unit tests.

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #47769 

**Special notes for your reviewer**:

**Release note**:

```release-note
NONE
```
2017-06-23 15:48:35 -07:00
Matthew Wong
76a72f6248 Move e2e fromManifest funcs to manifest package 2017-06-23 18:47:42 -04:00
Zihong Zheng
2cb9f5d83d [esipp-e2e] Change service port to avoid collision 2017-06-23 14:14:30 -07:00
Piotr Szczesniak
8d5d85e126 Fixed Monitoring e2e test 2017-06-23 21:43:17 +02:00
Jun Xiang Tee
ae4df9f573 deprecate created-by annotation for cronjob 2017-06-23 11:05:40 -07:00
Kubernetes Submit Queue
afc4506c33 Merge pull request #47957 from aleksandra-malinowska/autoscaling-anti-affinity-test-fix
Automatic merge from submit-queue

Fix test name

This test wasn't run by any job due to incorrect name.
2017-06-23 10:39:14 -07:00
Kubernetes Submit Queue
068e361936 Merge pull request #47875 from crassirostris/event-exporter-e2e-test
Automatic merge from submit-queue

Implement e2e test for Stackdriver event exporter

Fixes https://github.com/kubernetes/kubernetes/issues/47963

/cc @piosz
2017-06-23 10:38:48 -07:00
Mikhail Vyatskov
f11d8b9daa Implement e2e test for Stackdriver event exporter 2017-06-23 16:50:12 +02:00
Maciej Pytel
5e0520a4a0 Add more pdbs in autoscaling e2e 2017-06-23 13:37:59 +02:00
Kubernetes Submit Queue
4d748f5e03 Merge pull request #47475 from crimsonfaith91/e2e
Automatic merge from submit-queue (batch tested with PRs 47915, 47856, 44086, 47575, 47475)

deprecate created-by annotation for e2e test framework

**What this PR does / why we need it**: This PR deprecates created-by annotation for e2e test framework. This is needed as we now have ControllerRef.

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: xref #44407

**Special notes for your reviewer**: This is the third PR for deprecating created-by annotation. Other PRs can be found here: #47469, #47471 

**Release note**:

```release-note
```
2017-06-23 04:06:28 -07:00
Aleksandra Malinowska
a12669accb Fix test name 2017-06-23 12:29:40 +02:00
Kubernetes Submit Queue
d7c569eda7 Merge pull request #46527 from perotinus/rssyncconversiontwo
Automatic merge from submit-queue (batch tested with PRs 47403, 46646, 46906, 46527, 46792)

[Federation] Convert the ReplicaSet controller to a sync controller.

See #45563 for previous discussion: that PR was split into two, with this one containing the actual conversion work and that one containing the implementation of the scheduling methods in the sync controller.

**Release note**:
```release-note
NONE
```
2017-06-23 02:59:29 -07:00
Kubernetes Submit Queue
95a5810209 Merge pull request #47403 from zhangxiaoyu-zidif/e2e-delete-meaningless-check
Automatic merge from submit-queue (batch tested with PRs 47403, 46646, 46906, 46527, 46792)

E2E:Delete unecessary check

**What this PR does / why we need it**:
Delete meaningless check
The deleted check is redundant.

**Release note**:

```release-note
NONE
```
2017-06-23 02:59:23 -07:00
Kubernetes Submit Queue
7ecb6cc238 Merge pull request #47095 from MrHohn/fix-e2e-firewall-multizone
Automatic merge from submit-queue (batch tested with PRs 47922, 47195, 47241, 47095, 47401)

Fix firewall e2e test for multizone CIs

Fixes #46894.

- Getting node/master tags from config instead of instance.
- Don't assume instance always run in default zone.

/assign @freehan 

**Release note**:

```release-note
NONE
```
2017-06-22 21:33:31 -07:00
Kubernetes Submit Queue
467705be00 Merge pull request #47195 from dims/bind-cadvisor-on-kubelet-interface
Automatic merge from submit-queue (batch tested with PRs 47922, 47195, 47241, 47095, 47401)

Run cAdvisor on the same interface as kubelet

**What this PR does / why we need it**:

cAdvisor currently binds to all interfaces. Currently the only
solution is to use iptables to block access to the port. We
are better off making cAdvisor to bind to the interface that
kubelet uses for better security.

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #

Fixes #11710

**Special notes for your reviewer**:

**Release note**:

```release-note
cAdvisor binds only to the interface that kubelet is running on instead of all interfaces.
```
2017-06-22 21:33:27 -07:00
Jonathan MacMillan
16943f6f30 [Federation] Convert the ReplicaSet controller to a sync controller. 2017-06-22 17:47:43 -07:00
Ajit Kumar
caff16c678 Bump up npd version to v0.4.1 2017-06-22 13:13:50 -07:00
Chao Xu
60604f8818 run hack/update-all 2017-06-22 11:31:03 -07:00
Chao Xu
f2d3220a11 run root-rewrite-import-client-go-api-types 2017-06-22 11:30:59 -07:00
Chao Xu
cde4772928 run ./root-rewrite-all-other-apis.sh, then run make all, pkg/... compiles 2017-06-22 11:30:52 -07:00
Chao Xu
f4989a45a5 run root-rewrite-v1-..., compile 2017-06-22 10:25:57 -07:00
Aleksandra Malinowska
53ab8fd86d Create respecting pod anti affinity test 2017-06-22 14:09:54 +02:00
Kubernetes Submit Queue
c3dd67dc21 Merge pull request #47876 from mwielgus/remove-initial-resources-e2e
Automatic merge from submit-queue

Remove initial resources e2e

This is just a cleanup PR. We won't continue initial resources effort. It is dropped in favor of VPA developed in kubernetes/autoscaler. 

cc: @piosz @jszczepkowski @kgrygiel
2017-06-22 02:20:48 -07:00
Kubernetes Submit Queue
96716d3eb4 Merge pull request #47857 from deads2k/agg-35-routing
Automatic merge from submit-queue (batch tested with PRs 47878, 47503, 47857)

restore working aggregator and avoid duplicate informers

Fixes https://github.com/kubernetes/kubernetes/issues/47866

This runs the informer all the way through and makes sure its started.

@lavalamp ptal
@kubernetes/sig-api-machinery-bugs
2017-06-21 21:12:18 -07:00
Bobby (Babak) Salamat
1e5afd722c Remove e2e test for least requested prioirty function 2017-06-21 17:14:23 -07:00
Kubernetes Submit Queue
d86c267112 Merge pull request #46099 from shashidharatd/revert-45798-federated-service
Automatic merge from submit-queue (batch tested with PRs 47851, 47824, 47858, 46099)

Revert "[Federation] Fix federated service reconcilation issue due to addition of External…"

Reverts kubernetes/kubernetes#45798

Reverting the temporary fix as the problem is fixed in #45869.
with that fix federation also can default ExternalTrafficLocalOnly if not set.

Issue: #45812

cc @MrHohn @madhusudancs @kubernetes/sig-federation-bugs
2017-06-21 15:21:19 -07:00
Kubernetes Submit Queue
03014f486c Merge pull request #47824 from mbohlool/revert2
Automatic merge from submit-queue (batch tested with PRs 47851, 47824, 47858, 46099)

Revert 44714 manually

#44714 broke backward compatibility for old swagger spec that kubectl still uses. The decision on #47448 was to revert this change but the change was not automatically revertible. Here I semi-manually remove all references to UnixUserID and UnixGroupID and updated generated files accordingly.

Please wait for tests to pass then review that as there may still be tests that are failing.

Fixes #47448

Adding release note just because the original PR has a release note. If possible, we should remove both release notes as they cancel each other.

**Release note**: (removed by caesarxuchao)

UnixUserID and UnixGroupID is reverted back as int64 to keep backward compatibility.
2017-06-21 15:21:14 -07:00
Marcin Wielgus
5537088493 Update BUILD after initial resources e2e deletion 2017-06-21 23:39:36 +02:00
Marcin Wielgus
40a6fe56b0 Remove initial resources e2e 2017-06-21 23:28:36 +02:00
deads2k
f525c0815e restore working aggregator and avoid duplicate informers 2017-06-21 15:14:59 -04:00
Kubernetes Submit Queue
0f0e017ade Merge pull request #45473 from karataliu/AzureInternalLoadBalancerE2E
Automatic merge from submit-queue

Add E2E tests for Azure internal loadbalancer support, fix an issue for public IP resource deletion.

**What this PR does / why we need it**:

- Add E2E tests for Azure internal loadbalancer support: https://github.com/kubernetes/kubernetes/pull/43510
- Fix an issue that public IP resource not get deleted when switching from external loadbalancer to internal static loadbalancer.

**Special notes for your reviewer**:

1.  Add new Azure resource tag to Public IP resources to indicate kubernetes managed resources.
   Currently we determine whether the public IP resource should be deleted by looking at LoadBalancerIp property on spec. In the scenario 'Switching from external loadbalancer to internal loadbalancer with static IP', that value might have been updated for internal loadbalancer. So here we're to add an explicit tag for kubernetes managed resources.

2. Merge cleanupPublicIP logic into cleanupLoadBalancer

**Release note**:
NONE

CC @brendandburns @colemickens
2017-06-21 11:41:22 -07:00
mbohlool
70c4fe7f4f update generated files 2017-06-21 04:09:08 -07:00
mbohlool
c91a12d205 Remove all references to types.UnixUserID and types.UnixGroupID 2017-06-21 04:09:07 -07:00
Kubernetes Submit Queue
a3501fb994 Merge pull request #47790 from MaciekPytel/fix_autoscaler_flaky_e2e_3
Automatic merge from submit-queue

Further reduce cluster-autoscaler e2e flakiness

Ref: https://github.com/kubernetes/autoscaler/issues/89

Add pdbs for additional kube-system pod, move adding
pdbs to separate function, as it will need to be reused
in new tests we're working on (ex. scale to 0).
2017-06-21 03:21:12 -07:00
Kubernetes Submit Queue
4fff2fbad3 Merge pull request #47332 from wasylkowski/e2e-autoscaling-timer
Automatic merge from submit-queue

Added an e2e test timing HPA + CA scaling up from 1 to 8 pods and from 3 to >=4 clusters

**What this PR does / why we need it**:

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #46847 

**Special notes for your reviewer**:

**Release note**:

```release-note
NONE
```
2017-06-20 10:06:34 -07:00
Maciej Pytel
7072a50aea Further reduce cluster-autoscaler e2e flakiness
Add pdbs for additional kube-system pod, move adding
pdbs to separate function, as it will need to be reused
in new tests we're working on.
2017-06-20 17:26:00 +02:00
Kubernetes Submit Queue
5e8f44f195 Merge pull request #47256 from aleksandra-malinowska/autoscaling-system-pods-test
Automatic merge from submit-queue

Add new e2e test for cluster size autoscaler (evicting system pods)

This test verifies that cluster autoscaler drains nodes with system pods running if they have a PDB.
2017-06-20 04:09:23 -07:00