Commit Graph

1055 Commits (18ede140ac4fce456f88c49a7254ddc9a97655af)

Author SHA1 Message Date
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 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
Erick Fejta ffe611569d Set GOOGLE_APPLICATION_CREDENTIALS after activating 2016-08-03 23:24:31 -07:00
Kubernetes Submit Queue b917ca0937 Merge pull request #29864 from wonderfly/correct-k8s-version
Automatic merge from submit-queue

e2e-runner: Fix the logic of pulling GCI builtin version of k8s

Apparently setting JENKINS_PUBLISHED_VERSION didn't work because it only worked
with `ci/latest`, `release/latest-1.3` or the like, whereas we wanted to pull
specific versions like `release/v1.3.3`. Added a function
`fetch_gci_version_tars` instead.

@spxtr Can you review? Hopefully this should get our QA tests passing.
2016-08-03 18:14:56 -07:00
Jeff Grafton 42d2aca7bb Download go-bindata in unit/verify/node e2e Jenkins builds 2016-08-03 13:23:41 -07:00
Erick Fejta 548780f05a Revert "Revert "Drop support for --gce-service-account, require activated creds"" 2016-08-02 20:34:02 -07:00
Kubernetes Submit Queue 31ff5d423d Merge pull request #29892 from david-mcmahon/jenkins2
Automatic merge from submit-queue

Clone kubernetes/release into something other than /release due to conflict with /release.

It looks like $WORKSPACE is the root of the kubernetes clone.  I think ideally we'd move that down a level and allow subdirectories of different repos.  I think this may become the norm as we split things up further as well.  What do you think and how should we proceed here?  @ixdy @spxtr
2016-08-02 16:24:07 -07:00
David McMahon eceef16b9f Clone kubernetes/release into something other than /release due to conflict. 2016-08-02 13:02:39 -07:00
k8s-merge-robot ca7ad6896b Merge pull request #29185 from soltysh/make_verify
Automatic merge from submit-queue

Fix make verify

@thockin this fixes `make verify` which was previously failing due to undefined `BRANCH` and `VERBOSE` variables, ptal
2016-08-02 01:53:44 -07:00
Daniel Wang dee12cfb60 e2e-runner: Fix the logic of pulling GCI builtin version of k8s
Apparently setting JENKINS_PUBLISHED_VERSION didn't work because it only worked
with `ci/latest`, `release/latest-1.3` or the like, whereas we wanted to pull
specific versions like `release/v1.3.3`. Added a function
`fetch_gci_version_tars` instead.
2016-08-01 14:31:04 -07:00
Daniel Wang e4cfb91bd9 e2e-runner: Remove quotes from jq output
Recent GCI e2e test failures suggest that the image name was double quoted when
passed to gcloud. E.g.,

```
09:59:51  - Invalid value for field
'resource.disks[0].initializeParams.sourceImage':
'https://www.googleapis.com/compute/v1/projects/container-vm-image-staging/global/images/"gci-base-53-8530-36-0"'.
The referenced image resource cannot be found.
```

The `-r`, or `--raw-output` option strips quotes from jq's output. With this
change, the output looks like:

```
+ export KUBE_GCE_MASTER_IMAGE=gci-base-54-8650-0-0  # Not quoted
```
2016-07-29 10:31:42 -07:00
Daniel Wang 081dfbbb73 Add support for GCI QA jobs
GCI QA jobs will run tests using GCI daily builds, and the kubernetes built into
the images. All QA jobs will set the `JENKINS_USE_GCI_VERSION` env var.
2016-07-26 13:33:45 -07:00
Maciej Szulik 2a2b46a105 Fix make verify 2016-07-25 15:34:00 +02:00
Wojciech Tyczynski 7ef2e946dc Fail Kubemark if tests fail with error. 2016-07-22 11:52:54 +02:00
Ryan Hitchman 0cb96b3614 Revert "Drop support for --gce-service-account, require activated creds" 2016-07-19 17:41:39 -07:00
k8s-merge-robot 8cf40ebe25 Merge pull request #29127 from jfrazelle/update-go-1.6.3
Automatic merge from submit-queue

update go to 1.6.3

<!--
Checklist for submitting a Pull Request

Please remove this comment block before submitting.

1. Please read our [contributor guidelines](https://github.com/kubernetes/kubernetes/blob/master/CONTRIBUTING.md).
2. See our [developer guide](https://github.com/kubernetes/kubernetes/blob/master/docs/devel/development.md).
3. If you want this PR to automatically close an issue when it is merged,
   add `fixes #<issue number>` or `fixes #<issue number>, fixes #<issue number>`
   to close multiple issues (see: https://github.com/blog/1506-closing-issues-via-pull-requests).
4. 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
updated go version to 1.6.3
```
2016-07-19 06:17:58 -07:00
k8s-merge-robot cbda93e23e Merge pull request #27898 from lavalamp/fixv
Automatic merge from submit-queue

Detect flakes in PR builder e2e runs

Won't be mergable until https://github.com/onsi/ginkgo/pull/261 is agreed upon and merged.

Tossing a PR here to get the e2e test to run on it.
2016-07-18 16:17:02 -07:00
Jess Frazelle a333cf429a update go to 1.6.3
Signed-off-by: Jess Frazelle <me@jessfraz.com>
2016-07-18 16:29:46 -04:00
k8s-merge-robot d08a722647 Merge pull request #28802 from fejta/service
Automatic merge from submit-queue

Drop support for --gce-service-account, require activated creds

Now that `gcloud auth activate-service-account` is in remove support in the test framework for default service accounts -- testing GCE/GKE now requires prior gcloud activation.
2016-07-18 12:08:03 -07:00
Daniel Smith e91c5d048b run e2e with flake detection 2016-07-15 23:58:29 -07:00
k8s-merge-robot 36c07cc554 Merge pull request #27961 from maisem/os-distribution
Automatic merge from submit-queue

Splitting OS_DISTRIBUTION into NODE_OS_DISTRIBUTION and MASTER_OS_DISTRIBUTION

fixes #26183 
[![Analytics](https://kubernetes-site.appspot.com/UA-36037335-10/GitHub/.github/PULL_REQUEST_TEMPLATE.md?pixel)]()
2016-07-15 14:11:47 -07:00
Maisem Ali 97f3f80833 Splitting OS_DISTRIBUTION into NODE_OS_DISTRIBUTION and
MASTER_OS_DISTRIBUTION.
2016-07-15 12:02:31 -07:00
k8s-merge-robot 4030afdfb1 Merge pull request #28922 from david-mcmahon/release-push
Automatic merge from submit-queue

Add RELEASE_INFRA_PUSH related code to support pushes from kubernetes/release.

ref #16529
2016-07-14 19:55:56 -07:00
David McMahon ae18034920 Add RELEASE_INFRA_PUSH related code to support pushes from kubernetes/release. 2016-07-14 11:46:59 -07:00
Daniel Wang f5d711a56c Update the image family used for GCI canary test images 2016-07-13 14:27:18 -07:00
Tim Hockin 1bd3918c15 Overhaul deps on generator tools
This forces a regeneration if the generation tools are rebuilt.

Also clean up Makefile a bit.
2016-07-12 21:52:54 -07:00
Tim Hockin faeef5c4ae Use make as the main build tool
This allows us to start building real dependencies into Makefile.

Leave old hack/* scripts in place but advise to use 'make'.  There are a few
rules that call things like 'go run' or 'build/*' that I left as-is for now.
2016-07-12 21:52:00 -07:00
Erick Fejta fe8578db80 Drop support for --gce-service-account, require activated creds 2016-07-11 14:47:04 -07:00
Erick Fejta 41a23d6521 Do not depend on metadata server for auth 2016-07-11 13:16:34 -07:00
Erick Fejta 3d30caf80c Inject service-account.json into test container 2016-07-11 09:27:15 -07:00
k8s-merge-robot 6eae13e0c4 Merge pull request #28133 from freehan/e2ejunit
Automatic merge from submit-queue

add junit recording in e2e runner

2nd part of #27825  

Injected junit recorder at a few places.
2016-07-09 01:55:07 -07:00
Erick Fejta 4ea125f9eb Only activate service account when file exists 2016-07-09 00:16:20 -07:00
k8s-merge-robot 3961631b0f Merge pull request #28634 from fejta/auth
Automatic merge from submit-queue

Activate a service account if this is defined

For https://github.com/kubernetes/test-infra/issues/266 and https://github.com/kubernetes/kubernetes/issues/28612

This will activate a service account if the service account file exists.
2016-07-08 23:17:45 -07:00
Erick Fejta 8c8c7d4ef8 Activate a service account if this is defined 2016-07-07 13:35:10 -07:00
Jess Frazelle 5e03b743d6
Cleanup hack/ Dockerfiles
Signed-off-by: Jess Frazelle <jessfraz@google.com>
2016-07-06 11:03:42 -07:00
k8s-merge-robot d8a54c7a5c Merge pull request #28237 from ixdy/kubekins-aws
Automatic merge from submit-queue

Install pip in kubekins test image and install AWS cli when needed

Follow-on to https://github.com/kubernetes/test-infra/pull/66. This fixes AWS hopefully.

I haven't pushed the image yet.

cc @fejta
2016-06-30 22:07:51 -07:00
k8s-merge-robot f2ddd60eb9 Merge pull request #26755 from david-mcmahon/fix-headers
Automatic merge from submit-queue

Remove "All rights reserved" from all the headers.

cc @thockin @zmerlynn @brendanburns
2016-06-29 18:46:07 -07:00
David McMahon ef0c9f0c5b Remove "All rights reserved" from all the headers. 2016-06-29 17:47:36 -07:00
Minhan Xia a5c1e02e0e add junit recording in e2e runner 2016-06-29 16:46:04 -07:00
k8s-merge-robot 5f4aaaa1db Merge pull request #28246 from zmerlynn/revert-revert-28193
Automatic merge from submit-queue

Revert Revert of #28193

Bring back #28193. We caught a break in https://github.com/kubernetes/test-infra/issues/240 and discovered the previous issue, fixed in https://github.com/kubernetes/test-infra/pull/241 and https://github.com/kubernetes/test-infra/pull/244, so I have a pretty good handle on what was causing the previous bringup issues (and it wasn't #28193). By the time this merges, we'll have good signal on GKE in several concurrent GKE jobs.

This reverts commit ee1d480333.

[![Analytics](https://kubernetes-site.appspot.com/UA-36037335-10/GitHub/.github/PULL_REQUEST_TEMPLATE.md?pixel)]()
2016-06-29 16:25:31 -07:00
k8s-merge-robot 8621326a7a Merge pull request #28191 from lavalamp/red-dot-city
Automatic merge from submit-queue

unstable -> failed

@fejta and @ixdy
2016-06-29 15:35:08 -07:00
Zach Loafman 2d6c632de2 Revert "Revert "Merge pull request #28193 from zmerlynn/pull-ci-elsewhere""
Bring back #28193. We caught a break in
https://github.com/kubernetes/test-infra/issues/240 and discovered the
previous issue, fixed in
https://github.com/kubernetes/test-infra/pull/241 and
https://github.com/kubernetes/test-infra/pull/244, so I have a pretty
good handle on what was causing the previous bringup issues (and it
wasn't #28193). By the time this merges, we'll have good signal on GKE
in the `kubernetes-e2e-gke-updown` job.

This reverts commit ee1d480333.
2016-06-29 15:10:24 -07:00
Jeff Grafton 9c7da9c116 Install pip in kubekins test image and install AWS cli when needed 2016-06-29 13:15:14 -07:00
Zach Loafman ee1d480333 Revert "Merge pull request #28193 from zmerlynn/pull-ci-elsewhere"
This reverts commit d965b4719c, reversing
changes made to 08a28e5123.
2016-06-29 07:55:53 -07:00
k8s-merge-robot 2baf9b0f27 Merge pull request #27798 from ixdy/kubekins-docker
Automatic merge from submit-queue

Include static docker binary in kubekins-test image

Fixes kubernetes/test-infra/issues/47.

I haven't pushed this image yet, so I expect CI to fail for now.
2016-06-28 23:42:24 -07:00
Daniel Smith b754695adb unstable -> failed (for presubmit tests) 2016-06-28 20:29:49 -07:00
Zach Loafman 92485326b1 Change references to gs://kubernetes-release/ci
Change over to gs://kubernetes-release-dev/ci. This should be all the
places we reference gs://kubernetes-release/ci or
https://storage.googleapis.com/kubernetes-release/ci. I'm happy to be
wrong.

Follow-on to #28172
2016-06-28 16:45:12 -07:00
Jeff Grafton 1040fac4b3 Download static docker binary into kubekins test image
Also rework apt-get lines to conform to Docker best practices and add
dnsutils.

Lastly, use kubekins-test:go1.6.2-docker1.9.1-rev1 on Jenkins
2016-06-27 14:27:12 -07:00
k8s-merge-robot ff7c280200 Merge pull request #27163 from ixdy/curl-flakes
Automatic merge from submit-queue

Pass --keepalive-time 2 to curl to fix unexpected EOF flakes

I hope this fixes #22967.
2016-06-26 13:35:03 -07:00
Daniel Wang 93189dd569 e2e-runner: Improve the logic of detecting Docker releases
Apparently it didn't handle GitHub prereleases well. Also switch from
`JENKINS_GCI_IMAGE_TYPE` to `JENKINS_GCI_IMAGE_FAMILY` following up #27083.
2016-06-20 17:51:20 -07:00