Commit Graph

201 Commits (aa485ce82c3336b0aa70e27e558d2ce12340a4dd)

Author SHA1 Message Date
gmarek 1aff115f44 Make logging function injectable in RCConfig 2016-10-11 16:46:33 +02:00
deads2k 48ac4d549d change e2e scale to avoid adapter 2016-09-26 07:58:01 -04:00
Wojciech Tyczynski 767bc90a20 Fix removing RCs in density test 2016-09-16 13:13:02 +02:00
Mike Danese a765d59932 move informer and controller to pkg/client/cache
Signed-off-by: Mike Danese <mikedanese@google.com>
2016-09-15 12:50:08 -07:00
gmarek a6e8789194 Relax the CPU constraint for L7 lb 2016-09-12 10:42:44 +02:00
gmarek 4051a2f8ea Increase the threshold for apiserver CPU for 500 node kubemark 2016-09-05 09:48:25 +02:00
Kubernetes Submit Queue 8a44cb5c7b Merge pull request #31683 from gmarek/kubemark-5
Automatic merge from submit-queue

Tune kubemark-5 resource constraints

Ref. #31105
2016-08-30 01:35:38 -07:00
gmarek 6ab61d7a97 Tune kubemark-5 resource constraints 2016-08-30 09:34:47 +02:00
Wojciech Tyczynski c106646d32 Tweak resource constraints in scalability tests 2016-08-26 10:19:45 +02:00
Kubernetes Submit Queue 1e7545393f Merge pull request #30979 from m1093782566/fix-unnecessary-delete-rc-requests
Automatic merge from submit-queue

[e2e density test] Fix unnecessary Delete RC requests when not running latency test

As the following code block
https://github.com/kubernetes/kubernetes/blob/master/test/e2e/density.go#L666-L670
shows, after running each density test case, it will attempt to delete "additional replication controllers" even though there is **no additional replication controller**.

When we are not running latency test, API Server will return "404 error code". So, I propose to move the above code block inside thedetermine statementsif `itArg.runLatencyTest{ }` , looks like:

```
if itArg.runLatencyTest {
  ...
  for i := 1; i <= nodeCount; i++ {
    name := additionalPodsPrefix + "-" + strconv.Itoa(i)
    c.ReplicationControllers(ns).Delete(name, nil)
  }
}
```
In this way, removing RC will be executed only if we set `itArg.runLatencyTest` to be `true`. It can avoid post some necessary requests to API Server.

Issuse is #30977
2016-08-22 07:00:52 -07:00
Wojciech Tyczynski 943860b3c7 Tweak resource consumption in density test 2016-08-22 10:03:45 +02:00
m1093782566 09668823a9 fix unnecessary delete rc requests
Change-Id: I5e7813acebd0607c049d521cce559b96708be1a2
2016-08-19 16:14:50 +08:00
m1093782566 47e0d4dc89 fix e2e test pause image hard code
Change-Id: I30ee8d5da96b6de42cf7d308776dcb9606d67dfb
2016-08-19 12:11:28 +08:00
gmarek d3d5f88880 Make sure that provider is set when we create resource constraints in the density test 2016-08-18 13:41:17 +02:00
Wojciech Tyczynski 715f4f0978 Merge pull request #30856 from wojtek-t/fix_density_part_1
Fix l7 constraints in density test
2016-08-18 12:12:29 +02:00
Wojciech Tyczynski bd6c5f4f57 Fix l7 constraints in density test 2016-08-18 11:30:03 +02:00
gmarek 59b7a05e15 Fix resource constraint checking in Density test 2016-08-17 14:12:41 +02:00
Chao Xu 3688a78769 let load and density e2e tests use GC if it's on 2016-08-11 11:48:20 -07:00
Kubernetes Submit Queue e7d01097dc Merge pull request #29971 from caesarxuchao/fix-kubectl-rolling-update-with-gc
Automatic merge from submit-queue

[GarbageCollector] Fix kubectl rolling-update to work with GC

This changes the order of the [Rename()](https://github.com/kubernetes/kubernetes/blob/master/pkg/kubectl/rolling_updater.go#L532) function. After the change, Rename() first deletes the old RC and orphans its pods, then creates the new RC, which will then have a chance to adopt the orphaned pods.

This also fixes the "should support rolling-update to same image" [test](https://github.com/kubernetes/kubernetes/blob/master/test/e2e/kubectl.go#L915) when the garbage collector is on.

Here is the detailed explanation on why the test would have failed:
`kubectl rolling-update` will [rename](https://github.com/kubernetes/kubernetes/blob/master/pkg/kubectl/rolling_updater.go#L532-L546) the RC. It first creates the an identical RC (including spec.selectors) with the new name, then it deletes the existing RC. When GC is turned on, the newly created RC cannot adopt the existing pod, because it has a controllerRef pointing to the exising RC, so the new RC will create new pods and expect to see the creation. However, the new RC and the old RC have the same selector, so sometimes the old RC, instead of the new RC, has its [expectation lowered](https://github.com/kubernetes/kubernetes/blob/master/pkg/controller/replication/replication_controller.go#L346-L362), the new RC's expectation will stuck forever. The e2e test then times out when executing `kubectl delete newRC`, because there is the new RC will not scale down as its expectation is not fulfilled.

A side-note, we should fix [rm.getPodController()](https://github.com/kubernetes/kubernetes/blob/master/pkg/controller/replication/replication_controller.go#L346) to respect pod's controllerref, that will prevent similar bugs.

Also note that an old version `kubectl rolling-update` will not work with the GC. We cannot fix that.
2016-08-04 20:48:10 -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
Chao Xu 96c84303bd fix kubectl rolling-update when GC is enabled 2016-08-03 15:57:03 -07:00
Harry Zhang c495397cae Refactor uuid into its own pkg 2016-07-30 00:07:02 -04:00
Matt T. Proud 656ea8f98f test/e2e: plug time.Ticker resource leak.
This commit ensures that `logPodStartupStatus` does not leak
running `time.Ticker` instances.  Upon termination of the consuming
routine, we stop the ticker.
2016-07-18 16:26:42 +02:00
David McMahon ef0c9f0c5b Remove "All rights reserved" from all the headers. 2016-06-29 17:47:36 -07:00
k8s-merge-robot 61fb527801 Merge pull request #26631 from gmarek/constraints
Automatic merge from submit-queue

Revert revert of adding resource constraints for master components in density tests

The problem was the time when resource constraints were generated. It turns out that the provider is not set there. This version should work.

cc @roberthbailey @alex-mohr
2016-06-02 05:15:55 -07:00
Wojciech Tyczynski 2a5883fc3a Wait for all nodes to be schedulable in scalability tests 2016-06-02 10:20:59 +02:00
gmarek 8a05258070 Revert "Merge pull request #26356 from kubernetes/revert-25752-resource-limits"
This reverts commit 60c2ab4995, reversing
changes made to 965e8dce12.
2016-06-01 15:09:32 +02:00
k8s-merge-robot 04bdd37bc4 Merge pull request #25972 from luxas/remove_arch_constants
Automatic merge from submit-queue

Use pause image depending on the server's platform when testing

Removed all pause image constant strings, now the pause image is chosen by arch. Part of the effort of making e2e arch-agnostic.

The pause image name and version is also now only in two places, and it's documented to bump both
Also removed "amd64" constants in the code. Such constants should be replaced by `runtime.GOARCH` or by looking up the server platform

Fixes: #22876 and #15140
Makes it easier for: #25730
Related: #17981

This is for `v1.3`
@ixdy @thockin @vishh @kubernetes/sig-testing @andyzheng0831 @pensu
2016-05-28 04:48:59 -07:00
Lucas Käldström 79ca1911e1 Removed all pause image constant strings, now the pause image is chosen by arch. Part of the effort of making e2e arch-agnostic 2016-05-26 19:16:43 +03:00
Wojciech Tyczynski a2011cdf42 Revert "Add resource limits for master resources in density test" 2016-05-26 16:35:48 +02:00
k8s-merge-robot c58d0453dc Merge pull request #25752 from gmarek/resource-limits
Automatic merge from submit-queue

Add resource limits for master resources in density test

cc @jlowdermilk @fabioy
2016-05-26 04:25:44 -07:00
Wojciech Tyczynski eaf9e5f596 Increase expected fluent CPU limit 2016-05-25 08:16:01 +02:00
gmarek 8684b8ea72 Add resource limits for master resources in density test 2016-05-19 15:32:21 +02:00
Daniel Smith 0bda969cd5 Merge pull request #25261 from jayunit100/readySched
Support GetReadySchedulable pods for all node selecting tests.
2016-05-18 17:24:09 -07:00
Jay Vyas 326b213231 Consolidate node selecting tests to only use Schedulable + Running
nodes.
2016-05-06 10:32:12 -04:00
Muhammed Uluyol f3690e2d5e
build/pause: write in C
Builds statically against glibc. References to the old pause
image have been updated.
2016-05-04 21:45:52 -04:00
k8s-merge-robot 0057931f4e Merge pull request #24434 from gmarek/services
Automatic merge from submit-queue

Create multiple RCs in NC - prerequisite for adding services

WIP because I need to make the logs readable again.
2016-04-23 15:19:36 -07:00
gmarek d344c2e32b Create multiple RCs in NC - prerequisite for adding services 2016-04-21 17:20:05 +02:00
goltermann 3fa6c6f6d9 Enable vet 2016-04-20 09:48:24 -07:00
Tim St. Clair b0d3f32e88 Update test/e2e for test/e2e/framework refactoring 2016-04-13 10:50:17 -07:00
Tim St. Clair a55b4f2e77 Move test/e2e framework & utility code to test/e2e/framework 2016-04-13 10:50:17 -07:00
gmarek be3a5d83bb Make total pod throughput in density test more readable 2016-04-01 17:48:55 +02:00
Jay Vyas 9a71dfe753 KubeDescribe implementation with verify into after-build/ scripts. 2016-03-15 22:29:21 -04:00
Wojciech Tyczynski b47018e366 Schedule ReplicationControllers in pod-startup-time test. 2016-03-15 09:56:46 +01:00
k8s-merge-robot 168711b3cd Merge pull request #22281 from wojtek-t/increase_qps_limits
Auto commit by PR queue bot
2016-03-05 21:27:57 -08:00
gmarek 43a8f04193 Add a comment explaining the necessary scale for Density test 2016-03-04 11:05:20 +01:00
gmarek 030f5d073d Relax thresholds for cluster saturation time 2016-03-04 09:07:52 +01:00
Wojciech Tyczynski 2ba342797d Explicitly set request for latency pods 2016-03-03 08:29:47 +01:00
Wojciech Tyczynski c47aa368bd Increase QPS limits in integration and e2e tests 2016-03-01 16:49:38 +01:00
k8s-merge-robot 9e5d1b61fa Merge pull request #21882 from gmarek/throttle
Auto commit by PR queue bot
2016-03-01 06:38:06 -08:00