Commit Graph

1228 Commits (49921ada74a7727f2943a5d86e4e433e6d76f96b)

Author SHA1 Message Date
Robert Bailey 4f856b595d Merge pull request #9165 from smarterclayton/graceful
Enable graceful deletion using reconciliation loops in the Kubelet without TTL
2015-08-18 10:01:40 -07:00
Clayton Coleman adc97bf936 Conversions 2015-08-18 09:08:45 -04:00
Clayton Coleman 89f1f3b1b8 Alter graceful deletion to not use TTL
Avoid TTL by deleting pods immediately when they aren't
scheduled, and letting the Kubelet delete them otherwise.

Ensure the Kubelet uses pod.Spec.TerminationGracePeriodSeconds
when no pod.DeletionGracePeriodSeconds is available.
2015-08-18 09:08:43 -04:00
Clayton Coleman b842a7dd15 Revert "Revert "Gracefully delete pods from the Kubelet""
This reverts commit 98115facfd.
2015-08-18 08:57:02 -04:00
Wojciech Tyczynski d9d7f6d964 Reduce logs verbosity in scalability tests 2015-08-18 11:14:14 +02:00
Prashanth Balasubramanian 83f1212e0b Confirms daemons restart and do sane things in an e2e test 2015-08-17 20:39:57 -07:00
Eric Paris ca310ffdac Sort output from hack/verify-flag-underscore.py
So we get some consistency and readability to the exceptions.txt
2015-08-17 11:32:18 -07:00
Eric Paris 99dc464c9b verify-flags-underscore.py: update how to add to exclusion list 2015-08-17 11:19:32 -07:00
Eric Paris b432f036db verify-flags-underscore.py: ignore juju variables 2015-08-17 11:19:32 -07:00
Eric Paris f3fd2e1028 verify-flags-underscore.py: ignore if trailed by :
These are often yaml definitions, and thus not usages of kubernetes
flags.  Not necessarily always, but usually.
2015-08-17 11:19:32 -07:00
Eric Paris d1ba0bb692 verify-flags-underscore.py: ignore salt grains 2015-08-17 11:19:31 -07:00
Eric Paris d7f9477a94 verify-flags-underscore.py: exclude salt variable which look like flags 2015-08-17 11:19:31 -07:00
Eric Paris 7f96f9312b verify-flags-underscore.py: ignore 'flags' that look like bash variables
We ignore bad 'flags' if they are proceeded by $ or { as these are
likely bash variables and not in fact bad flag usage.
2015-08-17 11:19:31 -07:00
Eric Paris 055d648ed7 Change log_flush_frequency to use -
Since we have the rename, we don't need our declaration to look like
glog's.
2015-08-17 11:19:31 -07:00
Eric Paris c560f41a4b Update docs which were incorrectly using _ in flag names
Each of these was verified that the actual flag declaration correctly
used -
2015-08-17 11:18:43 -07:00
Eric Paris 68716c7712 verify-flags-underscore.py: Remove contrib/ exceptions
As contrib was removed.
2015-08-17 11:08:24 -07:00
Marek Grabowski d310eeaca9 Merge pull request #12783 from piosz/autoscaling_e2e
Enabled Autoscaling e2e test for Jenkins flaky job
2015-08-17 11:23:28 +02:00
Piotr Szczesniak b6f1c9d989 Enabled Autoscaling e2e test for Jenkins flaky job 2015-08-17 10:54:33 +02:00
Mike Danese fa40ced8f8 move contrib/for-tests to test/images 2015-08-14 20:42:46 -07:00
Brendan Burns 9cc5156b69 Merge pull request #12685 from eparis/goimports
Remove dependancy on external goimports binary
2015-08-14 20:29:00 -07:00
Brendan Burns 8e05da1316 Merge pull request #12696 from eparis/less-dash-false-positive
Fewer false positives in verify-flag-underscore.sh
2015-08-14 16:27:08 -07:00
Brendan Burns d7763a1b2b Merge pull request #12557 from eparis/boilerplate-multiple-file-types
Speed up pre-commit boilerplate by only checking changed files
2015-08-14 16:26:00 -07:00
Eric Paris 823392e4e3 New test and commit hook to make sure changes to Godeps are reproducable
It is a pretty slow test (it downloads fresh) all of kube's Godeps, so only
run it when needed in pre-commit hook.

This also means that random changes to other non-kube repositories could
cause travis/shippable to just randomly stop working for all PRs which touch
Godeps after that moment (even though no changes have been made to Godeps by
us).  Examples would be things like other repos completely disappearing. Or
even the directory we include disappearing in master in the remote
project (even though the directory may exist at the commit we care
about) This is a bugwin godep, but it is a problem we have seen happen
with kube Godeps.
2015-08-14 14:21:43 -04:00
Eric Paris 56f6ad0c01 Specifically exclude some flag definitions in verify-flag-underscore.sh
We know there are some flags (declared with an _) which we wish to
ignore. These flags are used by container definitions, e2e, etc. By
explicitly ignoring those flags we can cut the amount of noise in the
whitelist.
2015-08-14 14:11:02 -04:00
Eric Paris 8a91d86783 Speed up pre-commit boilerplate by only checking changed files
Although the boilerplate checker was very fast it can be faster. With
this change we can hand the boilerplate a list of files which need to be
checked or give it no files. If given no files it will run all files in
the repo. Before you had to explicitly tell the boiler checker the
'extention' of the the files.  In this case we let the checker figure it
out and load the headers as needed.

Doing the whole repo takes about 0.4 seconds. Doing a single go file
takes < .04 seconds.
2015-08-14 10:19:33 -04:00
Piotr Szczesniak fb44d5e045 Enabled Autoscaling e2e test for cpu utilization 2015-08-14 11:50:19 +02:00
Eric Paris 30d34d0e59 Reduce false positives with verify-flag-underscore.sh by updating regex
Check to make sure there is not an alphanumeric character immeditely
before or after the 'flag'.  It there is an alphanumeric character then
this is obviously not actually the flag we care about.  For example if
the project declares a flag "valid-name" but the regex finds something
like "invalid_name" we should not match.  Clearly this "invalid_name" is
not actually a wrong usage of the "valid-name" flag.
2015-08-13 21:06:39 -04:00
Eric Paris 8f3577e5d2 Do not use goimports in hack/update-generated-conversions.sh
Stop depending on an external binary needing to be on people's system.
Use the code directly.
2015-08-13 19:37:01 -04:00
Eric Paris 00929ed7d8 Convert generated-deep-copies to use golang.org/x/tools/imports
I don't like users to need an external binary which can easily get out
of sync. Just include the tools.
2015-08-13 19:36:57 -04:00
Eric Paris c6a5b43df1 Update flag-dash whitelist
PRs older than the verifier landed in HEAD.  So update the whitelist
2015-08-13 17:43:00 -04:00
Brendan Burns db39e5c45d Merge pull request #12667 from preillyme/issues_12666
Remove the "www" and "third-party ui" directories from kubernetes repo
2015-08-13 14:09:11 -07:00
Brendan Burns d8a60d2271 Merge pull request #12623 from thockin/proxy-ipt
Auto commit by PR queue bot
2015-08-13 13:10:39 -07:00
Brendan Burns 15281a5e01 Merge pull request #12388 from feihujiang/kubectlPatchAcceptFileNameParam
Auto commit by PR queue bot
2015-08-13 11:49:41 -07:00
Patrick Reilly bf02aaa97d remove other cluster addon references for container build 2015-08-13 10:23:55 -07:00
Tim Hockin 776132e1ae Make kube-proxy iptables sync period configurable 2015-08-13 09:53:32 -07:00
Ruddarraju, Uday Kumar Raju 937db3f70d Keystone authentication plugin 2015-08-13 09:46:30 -07:00
Wojciech Tyczynski 77f431649b Merge pull request #12462 from gmarek/cadvisor2
Add e2e test monitoring current container resource usage with updated constants.
2015-08-13 10:19:19 +02:00
feihujiang 1e10a7cc59 Kubectl scale command accepts a filename param 2015-08-13 14:26:25 +08:00
feihujiang 0110f3176b Kubectl patch command accepts a filename param 2015-08-13 11:13:31 +08:00
Brendan Burns 99b8df1812 Add field based sorting to the kubectl command line. 2015-08-12 16:51:36 -07:00
CJ Cullen 0f27307f82 Merge pull request #12609 from mikedanese/rewrite-shell
updating all references to GoogleCloudPlatform in .sh scripts
2015-08-12 16:00:43 -07:00
Mike Danese f061875840 updating all references in .sh scripts 2015-08-12 14:41:59 -07:00
Eric Paris f54098fe00 Verify all flag usage does not use _
This works by defining two 'static' lists in hack. The first is the list
of all flags in the project which use a `-` or an `_` in their name. All
files being processed by verify-flags-underscore.py (or all files in the
repo if no filename arguments are given) will be searched for flag
declaration using a simple regex. Its not super smart. If a flag is
found which is not in the static list it will complain/reject the commit
until a human adds it to the list. If we do not keep a static list of
flags it takes >.2 seconds to find them 'all' at runtime. Since this is
run in pre-commit saving every part of a second helps.

After it finds all of the flags it runs all of the arguments (or all
files in repo if no arguments) looking for usage of those flags which
includes an `_`. There are lots of places where these are false
positives. For example we have a flag named oom-adj-score but the kernel
calls it oom_adj_score. To handle this we keep a second 'whitelist' of
lines which are allowed to use these flag names with an `_`.

Running the entire git repo looking for flags in every golang file and
looking in every single file for bad usage takes about 8.75 seconds.

Running it in the precommit hook where we only check things that changed
takes about .06 seconds.
2015-08-12 16:17:02 -04:00
Eric Paris ce6f2c8e4b Do not build genconversion and gendeepcopy twice
The hack/after-build/verify-* functions were using the hack/update-*
functions.  Which means that if you call hack/verify-* you will do the
build twice. Stop it.
2015-08-12 10:05:48 -04:00
Piotr Szczesniak 81fb0f9582 Merge pull request #12527 from piosz/jenkins-autoscaling
Actually changed monitoring level on Jenkins
2015-08-12 11:23:12 +02:00
Piotr Szczesniak c91950f01c Merge pull request #12566 from uluyol/cleanup-convcopy
Add missing argument to function call and don't set GOPATH directly
2015-08-12 09:36:10 +02:00
Piotr Szczesniak 1df0267f4a Merge pull request #12551 from eparis/underscore-to-dash
Update code and docs to use - in flag names instead of _
2015-08-12 07:16:31 +02:00
Muhammed Uluyol eba19cc96a Add missing argument to function call and don't set GOPATH directly 2015-08-11 21:54:37 -07:00
Muhammed Uluyol 567bb15432 Generate conversions/deep-copies for experimental. Currently we make (and register) duplicate functions but this is benign. 2015-08-11 17:12:24 -07:00
Eric Paris 5aa495cdad Update code to use - in flag names instead of _ 2015-08-11 16:31:52 -04:00
CJ Cullen 9fdd793555 Merge pull request #11841 from eparis/commit-speed
Speedups for precommit hooks
2015-08-11 12:58:58 -07:00
Eric Paris bf9c3cd6ec Use cp instead of rsync in hack/verify-gendocs.sh
Instead of calling rsync over and over and over and over and over and
over and over and over and over (and probably over) use one `cp`

Before:
real	0m5.247s
user	0m2.294s
sys	0m1.300s

After:
real	0m2.260s
user	0m2.230s
sys	0m0.936s
2015-08-11 14:21:57 -04:00
Eric Paris 3b7c39656c Split hack/{verify,update}-* files so we don't always go build
Right now some of the hack/* tools use `go run` and build almost every
time. There are some which expect you to have already run `go install`.
And in all cases the pre-commit hook, which runs a full build wouldn't
want to do either, since it just built!

This creates a new hack/after-build/ directory and has the scripts which
REQUIRE that the binary already be built. It doesn't test and complain.
It just fails miserably. Users should not be in this directory. Users
should just use hack/verify-* which will just do the build and then call
the "after-build" version. The pre-commit hook or anything which KNOWS
the binaries have been built can use the fast version.
2015-08-11 14:20:46 -04:00
Eric Paris 985fa94ebd Move description.sh from hack/ to hooks/
Just to get everything in one place...
2015-08-11 14:20:38 -04:00
Eric Paris 302cb7dc7f Move boilerplate from hooks/ to hack/
Hooks seems like they should be hooks. Not sure hack makes a lot more
sense, but it has more stuff already.
2015-08-11 14:20:38 -04:00
Eric Paris 5aeebb4886 hack/lib/etcd: remove unused local var
It isn't used anywhere, why have it?
2015-08-11 14:20:38 -04:00
Eric Paris 055d008634 Update swagger to start etcd with known values
We were setting etcd values, but we started etcd before we set them.
Thankfully we were just setting them to etcd defaults, so it all worked
2015-08-11 14:20:38 -04:00
Piotr Szczesniak 5e58004131 Actually changed monitoring level on Jenkins 2015-08-11 15:25:36 +02:00
Piotr Szczesniak 63a88dd24a Changed monitoring level to googleinfluxdb in kubernetes-e2e-gce Jenkins job. 2015-08-11 10:07:48 +02:00
gmarek bc8565d55e Add e2e test monitoring current container resource usage with updated
constants.

This reverts commit 1f7ea4f08e.
2015-08-11 09:16:14 +02:00
Alex Robinson f7cab137df Merge pull request #12019 from gmarek/soak
Move initialization of Jenkins variables for GCE soak tests to hack/jenkins/e2e.sh
2015-08-10 16:19:57 -07:00
Alex Robinson 59c0fbb6a9 Merge pull request #12444 from thockin/nix-cmd-kubernetes
Remove old 'kubernetes' cmd
2015-08-10 16:19:35 -07:00
Alex Robinson 462fed62b7 Merge pull request #12189 from feihujiang/describeKindNoArguments
kubectl describe <kind> work with no arguments
2015-08-10 11:05:34 -07:00
feihujiang 73b694e6c1 kubectl describe <kind> work with no arguments
Add describe_resource_assert to test describe <kind> with no object arg
2015-08-10 14:08:38 +08:00
Tim Hockin bef7c99619 Remove old 'kubernetes' cmd 2015-08-08 20:39:57 -07:00
Veres Lajos 9f77e49109 typofix - https://github.com/vlajos/misspell_fixer 2015-08-08 22:31:48 +01:00
Satnam Singh 1f7ea4f08e Revert "Add e2e test monitoring current container resource usage" 2015-08-07 22:40:15 -07:00
Satnam Singh 991775e77b Merge pull request #10724 from eparis/convert-underscore-to-dash
Convert underscore to dash
2015-08-07 16:40:05 -07:00
Eric Paris 7715290bbd Convert hack/ from --flag_name= to --flag-name= 2015-08-07 11:00:13 -04:00
gmarek 1c22fd39ff Add e2e test monitoring current container resource usage.wq 2015-08-07 10:27:00 +02:00
Dawn Chen 9af5d8bd23 Merge pull request #12305 from GoogleCloudPlatform/puller
Exit if hub isn't installed, add verbiage to message.
2015-08-06 16:08:30 -07:00
Max Forbes acef505cfd Implement 'Nodes Network' test for GKE; add optional verbose SSH. 2015-08-06 14:25:53 -07:00
Wojciech Tyczynski e366f2c597 Merge pull request #12206 from derekwaynecarr/move_doc
LimitRange documentation should be under admin
2015-08-06 13:43:58 +02:00
Wojciech Tyczynski 871ca7a05f Merge pull request #12320 from mikedanese/rewrite-pr-issue-links
rewrite all links to issues and PRs to k8s links
2015-08-06 09:55:44 +02:00
Mike Danese 8326697055 rewrite all links to prs to k8s links 2015-08-05 21:11:11 -07:00
Alex Crawford 79f14822a9 Plumb linker flags through from the Makefile 2015-08-05 20:42:54 -07:00
Mike Danese fb02b33e99 fix build 2015-08-05 17:52:56 -07:00
Dawn Chen a90075d7eb Merge pull request #12008 from smarterclayton/fix_mac
mktemp on OSX behaves differently than Linux
2015-08-05 15:17:59 -07:00
Zach Loafman fccca8fa5d Exit if hub isn't installed, add verbiage to message. 2015-08-05 14:26:14 -07:00
Dawn Chen 9bc54481e0 Merge pull request #12233 from brendandburns/velocity
Add automated pull request creation.
2015-08-05 12:02:25 -07:00
derekwaynecarr 577f2fa0a5 LimitRange documentation should be under admin 2015-08-05 10:26:52 -04:00
Brendan Burns 70d843176b Add automated pull request creation. 2015-08-04 21:32:03 -07:00
Alex Mohr e92c63be74 Merge pull request #12172 from GoogleCloudPlatform/revert-12106-demote
Auto commit by PR queue bot
2015-08-04 18:48:49 -07:00
Eric Paris 08d8bea61d Support 1.5 in verify-gofmt 2015-08-04 19:59:19 -04:00
Alex Mohr 2d81775a2d Merge pull request #11151 from gmarek/max_pods_e2e
Auto commit by PR queue bot
2015-08-04 15:11:13 -07:00
gmarek 61d3b250ee Move initialization of Jenkins variables for soak tests to hack/jenkins/e2e.sh 2015-08-04 10:18:52 +02:00
Mike Danese 8e48431ca5 Revert "demote to flaky tests from parallel e2e" 2015-08-03 15:32:04 -07:00
Alex Mohr 2a397671bb Merge pull request #12060 from jlowdermilk/jenkins-default-version
Allow gke to use server-default version for jenkins runs
2015-08-03 13:22:07 -07:00
Mike Danese b56edd1355 Merge pull request #11727 from ZJU-SEL/build-nonstatic-hyperkube
build hyperkube as a nonstatic binary
2015-07-31 15:46:04 -07:00
Mike Danese cf4cb1a6a3 Merge pull request #10474 from kargakis/scale-multiple-controllers
Enable scaling multiple controllers
2015-07-31 14:51:43 -07:00
Mike Danese e376a094af demote to flaky service tests from parallel e2e 2015-07-31 13:25:15 -07:00
Mike Danese 7c47d6bdbd Merge pull request #12009 from smarterclayton/fix_cmd_config
hack/test-cmd.sh can fail when ~/.kube/config exists
2015-07-31 10:25:35 -07:00
Mike Danese 0269e2baad Merge pull request #11941 from GoogleCloudPlatform/enact_version_md
Enacting versioning.md
2015-07-31 10:17:22 -07:00
kargakis 0e796ca09a Enable scaling multiple controllers 2015-07-31 10:24:59 +02:00
Jeff Lowdermilk 2707fab181 Allow gke to use server-default version for jenkins runs 2015-07-30 18:32:57 -07:00
Clayton Coleman 0f9fdcafea Revert "Revert "Improve conversion to support multiple packages""
This reverts commit 94a387d5d1.
2015-07-30 15:15:14 -04:00
Mike Danese 94a387d5d1 Revert "Improve conversion to support multiple packages" 2015-07-30 10:51:40 -07:00
Mike Danese 1a613c43fe Merge pull request #9971 from smarterclayton/make_conversion_more_flexible
Improve conversion to support multiple packages
2015-07-30 10:05:10 -07:00
Mike Danese 0ae48c449e Merge pull request #11927 from wojtek-t/remove_shell_services
Remove shell services test
2015-07-30 09:56:38 -07:00
Wojciech Tyczynski 2c5c1931e5 Revert "Implement 'Nodes Network' test for GKE" 2015-07-30 15:29:01 +02:00